-
Notifications
You must be signed in to change notification settings - Fork 21
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
Select next/previous glyph #1706
Conversation
The order of the glyphMap does not really have a meaning, and is more or less undefined, so what this ends up doing is mostly accidental, even if it may happen to work somewhat. It may depend on how the backend prepares the list. It is notably not the same order as what you see in the search list, which is sorted according to certain criteria, also taking the code point(s) into account. So, as it stands, I am not in favor of this feature, and it requires more thought and design to properly define what it should do in the first place. |
Some notes from todays meeting:
Should the sorting be equal to the 'Glyph Search' sidebar?
Please check if the shortcuts commandKey + arrowLeft/Right work for Firefox and Safari also. |
@justvanrossum I checked the shortcuts commandKey + arrowLeft/Right in Chrome, Firefox and Safari and it works in all browsers. |
@justvanrossum I have set this PR to 'draft', as I am not sure about the last commit. But I figured out a solution to go through the glyph-order of 'Glyph Search'-sidebar + only the glyphs which are relevant, when the search has been used. Here a short demo: Screen.Recording.2024-10-10.at.12.36.14.mp4 |
18cbc0f
to
a0a66e6
Compare
…ectPreviousNextGlyph
d66987e
to
cfb4e7c
Compare
Fixes #1703