Uh oh!
There was an error while loading. Please reload this page.
Fix corrupted icons in second column - #1088
Conversation
When using skin with a different second column icon size, all of the icons in it would become blank. Because m_bTwoColumns is set after AddStandardItems, it is still false in the column break check. As a workaround, use the value it's assigned later in InitWindowInternal instead. FixOpen-Shell#980
AppVeyorBot
commented
Aug 10, 2022
blackcrack
commented
Aug 10, 2022
Thank you :))))) |
Ibuprophen
left a comment
There was a problem hiding this comment.
Very good. Thanks a Bunch! 👍
Ibuprophen
commented
Aug 11, 2022
Thank you @among-us-official! 👍 ~Ibuprophen |
CTVCAM8
commented
Aug 13, 2022
It works. Wonderful. This is similar to #561 Thank you, and thanks to other coders who have donated their time fixing skinned scrollbars in Win10, |
Ibuprophen
commented
Aug 13, 2022
Also fixes #561. ~Ibuprophen |
amymor
commented
Aug 14, 2022
thank you all ❤️ , it would be greet. finally i can delete classic shell 4.3.0 😂. |
Ibuprophen
commented
Aug 14, 2022
@amymor, I believe that I may have (hopefully) done it correctly this time around. 👍 https://github.com/Open-Shell/Open-Shell-Menu/releases I'm a bit rusty regarding a push on new releases... LOL! ~Ibuprophen |
bonzibudd
commented
Oct 11, 2022
among-us-official
commented
Oct 12, 2022
Hey @bonzibudd, I've looked into this, it seems this behavior is caused by turning on 'No icons in second column' (Main_no_icons2 in skin file). I'm not sure if this is unintended behavior. I can add a check in the code to prevent this, but perhaps the skin itself could be updated. In any case, what I should do? Also, the tiles are still clickable even if you can't see them. Not sure why the highlight doesn't appear. |
bonzibudd
commented
Oct 12, 2022
@among-us-official I did notice this, but in 4.4.170 and older, the Sub-items option would override the setting for no icons, which is why I can have items like the shutdown button coexisting with the tiles. If there is a way you could restore this behavior, I would appreciate it. |



When using skin with a different second column icon size, all of the
icons in it would become blank.
Because m_bTwoColumns is set after AddStandardItems, it is still false
in the column break check. As a workaround, use the value it's assigned
later in InitWindowInternal instead.
Fix#980