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
Crash when using mouse's forward/backward navigation keys #1628
Comments
Thanks for reporting @ZelimDamian. This can be reproduced by pressing the browser back or forwards button in chrome (not in firefox) - this is because it reports a mouse button event which cannot be interpreted - https://developer.mozilla.org/en-US/docs/Web/API/MouseEvent/buttons#value. To fix, the back and forward buttons could be added to the ModifierKeys struct. |
@0HyperCube I can't reproduce this in Chrome. I was wondering how you came to the conclusion that this issue report arises from the browser forward/backward keys, if OP didn't fill out the bug report with any details like reproduction steps? |
I can trivially reproduce in chrome. Perhaps you should try pressing the forwards button with the viewport selected in a new document? |
@Keavon Sorry for the poor details in the issue report but there weren't any other steps really originally. I just opened the window and clicked on New document and error showed up. Now that I'm testing it it doesn't happen when just opening a new document but clicking on the Back/Forward buttons on my mouse (even when there is no history to navigate) causes an error. |
Ok, yeah, I can get it to happen with the mouse button. I misinterpreted @0HyperCube's first comment thinking he meant the back/forward buttons in the browser's UI. I can reproduce this trivially with the mouse buttons. I'll update the issue details with that in mind, and I agree with Hypercube's point about adding those buttons which will be necessary for #706. |
@skoriop please comment here so I can assign you to this issue. |
sure! |
Steps To Reproduce
Describe precisely how the crash occurred, step by step, starting with a new editor window.
Browser and OS
Chrome 121, Windows 10 or 11
Stack Trace
Copied from the crash dialog in the Graphite editor:
The text was updated successfully, but these errors were encountered: