Migrate from GitHub to SourceForge with this tool. Check out all of SourceForge's recent improvements.
Close

#458 lxpanel load icon from icon theme first

closed-accepted
nobody
None
5
2014-01-27
2013-12-10
No

Hi,

Some icons of menu items will try to load the icon files under /usr/share/pixmaps when the icon name is in absolute path, but the icon failed to be scaled, so that these items looks ugly comparing to other items. Many icon themes have already prepared icons for different applications, I think it should be better to use these icons from icon theme.

I check the existing bugs, the #3614886, #3568913, #3411996 may be related.

Please kindly check the screenshots before and after the patch is applied.

Best Regards,
SHiNE

Discussion

  • SHiNE CsyFeK

    SHiNE CsyFeK - 2013-12-10

    debian, lxpanel-0.5.12-3-amd64

     
  • SHiNE CsyFeK

    SHiNE CsyFeK - 2013-12-10

    patch applied

     
  • SHiNE CsyFeK

    SHiNE CsyFeK - 2013-12-10

    Debug code has not been removed, if the code can be accepted, please clean the g_print()..

     
  • Henry Gebhardt

    Henry Gebhardt - 2014-01-27
    • status: open --> closed-accepted
     
  • Henry Gebhardt

    Henry Gebhardt - 2014-01-27

    Thanks, committed to master branch.

     

Log in to post a comment.