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

Cursor jumps around while typing #57

Closed
nikkiwd opened this issue Aug 26, 2023 · 1 comment · Fixed by #77
Closed

Cursor jumps around while typing #57

nikkiwd opened this issue Aug 26, 2023 · 1 comment · Fixed by #77

Comments

@nikkiwd
Copy link

nikkiwd commented Aug 26, 2023

I'm not able to write queries in the editor because the cursor frequently jumps around while I'm typing.

One example that I can reproduce in multiple Chrome-based browsers:

What happens: Every key that is pressed causes the cursor to move down a line: Up arrow returns the cursor to the same position, down arrow goes down two lines, ctrl, shift, alt, caps lock move the cursor down one line, shift+down moves the cursor down three lines and doesn't select the text, letter keys insert the letter and then move the cursor down one line.

@hannahbast
Copy link
Member

@nikkiwd Sorry for the late response. The problem occurs when the SELECT is not in all uppercase. This is a bug and we will try to fix it. But at least there is an easy workaround.

hannahbast added a commit that referenced this issue Feb 11, 2024
…is vein (#77)

Some of the useful automatic features of the query editor went rogue under certain conditions. Three things are now fixed or at least better:

1. When the query did not contain `SELECT` in all upper case and one of the existing prefixes was typed, each cursor movement would be followed by a reversal of that movement, which made the UI unusable. This is now fixed. In particular, this fixes #57.

2. So far, the query was "cleaned up" (blank lines removed, whitespace replaced by single spaces, etc.) after *every* cursor movement, which was not always desired. Now the query is only cleaned up in this way when the `TAB` key is pressed (which jumps between the different positions in the placeholder). In particular, this fixes #63.

3. Some REGEXes were only looking for a fully uppercase version of SPARQL keywords like `WHERE`, etc. Now they are case-insensitive.
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

Successfully merging a pull request may close this issue.

2 participants