#234 Menu icon size patch

open
nobody
bugfix (78)
5
2013-11-25
2011-11-07
Wilco
No

The configuration variable MenuIconSize is only defined in src/themable.h but not used in the code. Therefore, setting it to a value does not do anyting. Icewm uses smallIconSize for the menu icon size.

This patch adds proper behavior by using MenuIconSize for the size of the menu icons. I've tested this at two systems and it works correctly as far as I can tell.

Discussion

  • Marcus Moeller

    Marcus Moeller - 2012-04-10

    Sadly the menu icons are not rendered correctly using this patch. They look blurry, even if the original size of the icons is selected.

     
  • Wilco

    Wilco - 2012-07-27

    Updated patch with proper scaling

     
  • Wilco

    Wilco - 2012-07-27

    Yes, you're absolutely right. I've created a new patch with proper image scaling.

     
  • miner

    miner - 2013-11-12

    Dear Wilco:
    This patch make the menu icon size setting works well, but it also enlarge the height of window menu(Alt+Space).
    Would you make a new patch that can enable the menu icon size setting and keep normal height of window menu at the same time?
    Thank you so much.
    Best Regards,
    Miner

     
  • Wilco

    Wilco - 2013-11-19

    Hello Miner,

    This patch does not alter the height or properties of the taskbar. But I'm not quite sure what you mean with "window menu".

    Can you provide a screenshot with description of your problem?

     
  • Wilco

    Wilco - 2013-11-25

    Thanks for the screenshot, however to fix this in a generic way would require some serious rewriting of the menu code. So this is something the Icewm author has to fix.

     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:

JavaScript is required for this form.





No, thanks