-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Fix go-to-layer dialog in gcode preview #4816
Conversation
<msgid "Jump to layer"> probably became obsolete as a result of this change since it is no longer found in the sources. However, I've left them in place since I don't know how to properly modify .po files. |
That was my local modifications causing this issue. Change present in this PR is good. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shift-G works on my machine. Can you share a bit more info?
It seems only the dialog title is changed in your commit
"Jump to layer" dialog is not shown on Linux machine and English language selected. Moreover: after pressing "Shift+G" or selecting "Jump to layer" from a context menu of layer slider, new context menu does not shown on right click on the layer slider. Initial debugging shown that ``ImGui::OpenPopup()``` first checks if there are open modal dialog for a given window ID and fails if there are no one. In my case, window ID calculated to the next values:
This might be related related to the current locale set in Orca: most of the languages has the same strings for "Jump to layer" and "Jump to Layer". Locales most likely affected:
I'm not sure why I'm getting default locale strings instead of EN ones: my settings are set to "English" language and "Other" login region. |
Thanks for the explaination. Looks good |
gcode-preview: fix go-to-layer dialog
gcode-preview: fix go-to-layer dialog
gcode-preview: fix go-to-layer dialog
"Jump to Layer" popup is not shown neither by Shift+G nor by pressing context menu.
This change fixes that bad behavior.