#458 lxpanel load icon from icon theme first

closed-accepted
nobody
None
5
2014-01-27
2013-12-10
SHiNE CsyFeK
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

     
    Attachments
  • SHiNE CsyFeK
    SHiNE CsyFeK
    2013-12-10

    patch applied

     
    Attachments
  • 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.