-
Notifications
You must be signed in to change notification settings - Fork 32
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
UI for lineMetricsVerticalLayout #2054
Comments
Do we need to add top and bottom side bearings for proper compatibility as well? |
vertTypoAscender: default -> (head.unitsPerEm)/2 vertTypoDescender: default -> -(head.unitsPerEm)/2 But what is |
If we add UI for |
Do we need a glyph 'hight' similar to 'width' for CJK characters? |
Would it make sense to use the |
Do we need 'Zones' for |
This whole topic requires a lot more thinking then originally expected. As this is not part of Version 1.0 and the time till Version 1.0 is very limited, the best solution for now, might be to add |
In a call today, together with @justvanrossum, we agreed on it, that all the questions above are too much to handle now. BUT one possible next step would be to convert the font source 'Line Metrics' section into a ui-list with add and remove buttons (in a separate branch and PR). The add-button opens a dialog where we can select possible options like A question, we have not talked about: What happens if we delete one of the default line metrics like 'Baseline' or 'Ascender'? |
@justvanrossum I think this requires more and thorough consideration of how we really want to deal with it. There are too many open questions: How would a UI look like? Does it look like "Line metrics" or like "Custom Data". Is it always there, like "Guidelines", even though we don't really need it always? In general I would think it should look like "Line metrics", but we don't set these values always (which differs from "Line Metrics"), and we don't have an add or remove concept for that UI, yet.
Originally posted by @ollimeier in #2039 (comment)
The text was updated successfully, but these errors were encountered: