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

FontSize of controls #3593

Open
kabjar opened this issue Dec 6, 2017 · 4 comments

Comments

Projects
None yet
3 participants
@kabjar
Copy link

commented Dec 6, 2017

First: good job for all who participate on this project!

Is it possible somehow centrally change font size of all RD controls? Could it be part of settings?
I would prefer 10px (i.e. similar to VBA editor)...

Thanks.

@Vogel612

This comment has been minimized.

Copy link
Member

commented Dec 6, 2017

FWIW we should respect the font-size coming from OS customizations. As such we shouldn't expose that setting directly as a Rubberduck setting, but instead grab that information from our host app (or even the OS)

See also #3585

@kabjar

This comment has been minimized.

Copy link
Author

commented Dec 6, 2017

And grabbing that information from host app (VBA editor) is already implemented? Is it OK, that my VBA (Excel) editor appears with all controls in 10px, but Rubberduck controls are 12px height at the same time?

@Vogel612

This comment has been minimized.

Copy link
Member

commented Dec 6, 2017

That is because I didn't express what I meant fully.... That functionality is currently not implemented. That is why this is labeled [enhancement] and not [bug]. If it were already implemented, but there was a difference between the VBE and Rubberduck, that'd be a bug.

Do note that RD UI is not striving to look like the VBE, quite the opposite. We should respect accessibility and OS-Wide definitions though, and control / font size is one of those...

@bodywithoutorgans

This comment has been minimized.

Copy link

commented Jan 25, 2019

Not only font size, but also font family and font smoothing settings should come from OS settings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.