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

Hover to zoom characters in Property window #35

Open
ddacw opened this issue Jan 1, 2018 · 5 comments
Open

Hover to zoom characters in Property window #35

ddacw opened this issue Jan 1, 2018 · 5 comments

Comments

@ddacw
Copy link
Contributor

ddacw commented Jan 1, 2018

We can change character size now with the split handler, but it will shrink the character table. Instead, we can use some alternatives, such as hover-to-zoom.

@ddacw ddacw changed the title Hover to zoom characters in roperty windows" Hover to zoom characters in Property window Jan 1, 2018
@humdingerb
Copy link
Member

Is zooming really needed?
Personally, I don't even see the need in the split view. The default width of the left strip of the window is OK (better if the the font name etc weren't truncated). The selected character is nice and big and making it huge by reducing the list of all characters to the right doesn't make much sense to me...

@ddacw
Copy link
Contributor Author

ddacw commented Jan 1, 2018

I think in most cases, it isn't necessary. Though fonts do contain some faults, and only with zooming can we detect them. Here's an example (inconsolata, Google Fonts):
untitled-1

@humdingerb
Copy link
Member

It's probably not the usual use-case for FontBoy, but OK...
The split view is still the wrong means for zooming. Maybe the mouse wheel on the preview could zoom in/out where the mouse pointer is.

@janus2
Copy link
Member

janus2 commented Jan 1, 2018

I think FontBoy more as a catalog to consult the installed fonts... I don't thinks need to advanced functionality... I don't think it should be used to inspect glitches in fonts.

@humdingerb
Copy link
Member

No, but zooming in/out of that preview would be a nice intuitive feature that doesn't need additional widgets to clutter the GUI. If anyone wants to implement that it's OK. Otherwise the issue just sits in this tracker... :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants