Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

left panel: reverted menu icon scale to match item font size #9966

Merged

Conversation

h0lg
Copy link
Contributor

@h0lg h0lg commented Apr 17, 2022

Fixes #9964

Screenshots (Before / After)

image image

I agree that the maintainer squash merge this PR (if the commit message is clear).

✒️ I contribute this code under The Developer Certificate of Origin.

@ghost ghost assigned h0lg Apr 17, 2022
@h0lg h0lg changed the title reverted icon scale to match menu item font size; fixes #9964 reverted icon scale to match menu item font size Apr 17, 2022
@h0lg h0lg changed the title reverted icon scale to match menu item font size left panel: reverted menu icon scale to match item font size Apr 17, 2022
@RussKie RussKie added the 📭 needs: author feedback More info/confirmation awaited from OP; issues typically get closed after 30 days of inactivity label Apr 18, 2022
@ghost ghost removed the 📭 needs: author feedback More info/confirmation awaited from OP; issues typically get closed after 30 days of inactivity label Apr 19, 2022
@h0lg h0lg force-pushed the revert-left-panel-menu-icon-scale branch from c944365 to 7b0e532 Compare April 22, 2022 17:28
@h0lg
Copy link
Contributor Author

h0lg commented Apr 22, 2022

Just rebased on the master.

@ghost ghost added the 📭 needs: author feedback More info/confirmation awaited from OP; issues typically get closed after 30 days of inactivity label Apr 27, 2022
@h0lg h0lg force-pushed the revert-left-panel-menu-icon-scale branch from 7b0e532 to 3bcef7d Compare April 27, 2022 18:17
@ghost ghost removed the 📭 needs: author feedback More info/confirmation awaited from OP; issues typically get closed after 30 days of inactivity label Apr 27, 2022
@h0lg
Copy link
Contributor Author

h0lg commented Apr 27, 2022

@RussKie I've reverted the ImageScalingSize of both the leftPanelToolStrip and menuMain to 16x16 and the designer-generated lines vanish on save. My forms designer seems to be opinionated about the sizes though and resets them to 20x20 upon re-opening. That must be how those lines slipped in. And you were right, 16x16 seem to render just the same icon size as before. But so does 18x18. In other words, the difference between 18x18 and 16x16 is not visible to me.

@pmiossec
Copy link
Member

My forms designer seems to be opinionated about the sizes though and resets them to 20x20 upon re-opening.

@h0lg I think that the reason is that you have a display scaling different than 100%. And in this case, every time you have to use the winforms designer, you should launch VS with the /noscale option. See https://github.com/gitextensions/gitextensions/wiki/Guidelines%3A-UI-design

@h0lg h0lg requested a review from RussKie April 29, 2022 09:56
@RussKie
Copy link
Member

RussKie commented Apr 29, 2022

See gitextensions/gitextensions/wiki/Guidelines%3A-UI-design

Unfortunately, this guide is only true for .NET Framework, and it is not suited for .NET. The goldbar has been removed because the out-of-process designer doesn't work in hdpi scenarios. That said, the goldbar will be back in VS 17.3p1, and /noscale should be working for .NET apps as well. Until then, the designer should be run on monitors at 100% scale factor.

@RussKie RussKie merged commit f3e5c63 into gitextensions:master Apr 29, 2022
@ghost ghost added this to the vNext milestone Apr 29, 2022
@RussKie
Copy link
Member

RussKie commented Apr 29, 2022

Thank you

@h0lg h0lg deleted the revert-left-panel-menu-icon-scale branch October 23, 2022 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Left Panel: incorrect icons size
3 participants