-
Notifications
You must be signed in to change notification settings - Fork 27
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
Font overview phase 1 #1849
Font overview phase 1 #1849
Conversation
cf8c6cd
to
b815bb7
Compare
TODOs and Questions:
|
Little demo of the current stage: fontra-font-overview-sneak-preview.mp4 |
Next steps:
Question:
|
Minor bug: double clicking the |
c12e8c7
to
e75682d
Compare
c12e8c7
to
033e7fa
Compare
033e7fa
to
a22e41d
Compare
…aking the whole glyph-cell inline-block
… code points > 0xFFFF
I made a small change to glyph-cell.js, that fixes the lazy loading: d510469. You should notice a vast improvement in performance, as only glyphs that are in view will be loaded. One small bug that I don't quite understand: in Egitto, double-clicking on the "tulip" glyph does not open the right glyph. Maybe the source data contains ambiguous code points. Ohh, the code point is |
By default, I think the default source should be selected. You can easily get it with the We should store the selected source identifier in the URL fragment. |
…sing handleRemoteClose and handleRemoteError (similar to font-info))
…he navigation panel
…ion; don't fail updating selected state if the cell hasn't been rendered yet
Part of #102
This implements a very basic font overview. It needs some followups for refinement, but the core functionality is there and mergable.