Skip to content
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

Change in sqrt behavior #1975

Closed
mgreminger opened this issue May 22, 2023 · 1 comment
Closed

Change in sqrt behavior #1975

mgreminger opened this issue May 22, 2023 · 1 comment

Comments

@mgreminger
Copy link
Contributor

mgreminger commented May 22, 2023

Description

Sometime after 0.93.0, the \sqrt{} command behavior changed so that keyboard navigation moves the cursor to the degree location of the root symbol (the location where you would enter 3 to get a cubic root) when attempting to navigate out from under the square root symbol using the right arrow key (the same thing happens with the Tab key). The change in behavior also impacts attempting to delete the square root using the backspace command when the cursor is located after the square root (the backspace navigates to the degree location instead of deleting the square root). The second pass of deleting works since deleting the degree location once seems to remove the ability to navigate to the degree location subsequently.

See screen capture video:

Screen.Recording.2023-05-21.at.9.05.38.PM.mov

Same behavior occurs when inserting a square root using the virtual keyboard

Expected Behavior

When inserting a square root, I would like to have the option of not having the cursor navigate to the degree location of the square root since my app doesn't support changing the square root degree and this behavior would be confusing to users. It's especially confusing that the right arrow moves the cursor to the left and that the backspace doesn't work to delete the square root.

Environment

Math

MathLive version
Latest version on github db9099d

Operating System
macOS and Linux

Browser
Safari and Firefox

@mgreminger
Copy link
Contributor Author

Thanks for the fix!

mgreminger added a commit to mgreminger/mathlive that referenced this issue May 23, 2023
Adds regression tests for arnog#1975, arnog#1978, and arnog#1981

Also skip webkit iframe test in CI only since this test fails most of the time in CI even though it passes locally
arnog pushed a commit that referenced this issue May 23, 2023
* tests: add regression tests

Adds regression tests for #1975, #1978, and #1981

Also skip webkit iframe test in CI only since this test fails most of the time in CI even though it passes locally

* tests: add text mode serialization regression test
helio3197 pushed a commit to helio3197/mathlive that referenced this issue May 26, 2023
helio3197 pushed a commit to helio3197/mathlive that referenced this issue May 26, 2023
* tests: add regression tests

Adds regression tests for arnog#1975, arnog#1978, and arnog#1981

Also skip webkit iframe test in CI only since this test fails most of the time in CI even though it passes locally

* tests: add text mode serialization regression test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant