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.

Sign up for GitHub

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

Jump to bottom

Rapid key repeating causes cursor to move to beginning of the input box (Chrome on Linux) #44

Closed
refset opened this issue Jul 14, 2024 · 9 comments
Closed

Rapid key repeating causes cursor to move to beginning of the input box (Chrome on Linux) #44

refset opened this issue Jul 14, 2024 · 9 comments

Comments

@refset
Copy link
Contributor

refset commented Jul 14, 2024

No description provided.

@Akeboshiwind
Copy link
Collaborator

I've had this too on safari

johantonelli added a commit that referenced this issue Dec 6, 2024
@johantonelli
Re-frame improvements fixes #44
da27f68
johantonelli added a commit that referenced this issue Dec 6, 2024
@johantonelli
Re-frame improvements fixes #44
e52cc22
johantonelli added a commit that referenced this issue Dec 6, 2024
@johantonelli
Re-frame improvements fixes #44
b1e9d5d
johantonelli added a commit that referenced this issue Dec 9, 2024
@johantonelli
Re-frame improvements fixes #44
4f8d42a
johantonelli added a commit that referenced this issue Dec 10, 2024
@johantonelli
Beta use pgwire for sql
d43d183 
fixes #42
fixes #19

Handle vector row results

Move data manipulation to server

Remove beta button

Add beta banner

Add more logging

Handle todos

Add handler tests

Refactor

Reorg file structure

Stop shadowing clojure.core/list

Refactor

Re-frame improvements fixes #44

Add cljs run test

Tidy up

Fix query not updating

Fix nested data formats

Stay backwards compatible for docs

Remove confusing system time abstraction

Tidy up

Fix mobile footer and unstick non-mobile header

Standardise where we update-url on editor updates

Add update url interval

Make cljs config more config

Fix cljs tests
johantonelli added a commit that referenced this issue Dec 10, 2024
@johantonelli
Beta use pgwire for sql
217e751 
fixes #42
fixes #19

Handle vector row results

Move data manipulation to server

Remove beta button

Add beta banner

Add more logging

Handle todos

Add handler tests

Refactor

Reorg file structure

Stop shadowing clojure.core/list

Refactor

Re-frame improvements fixes #44

Add cljs run test

Tidy up

Fix query not updating

Fix nested data formats

Stay backwards compatible for docs

Remove confusing system time abstraction

Tidy up

Fix mobile footer and unstick non-mobile header

Standardise where we update-url on editor updates

Add update url interval

Make cljs config more config

Fix cljs tests
@refset
Copy link
Contributor Author

refset commented Dec 11, 2024

Noting that it seems this can still be triggered (e.g. hold down a key in the middle of some text for long enough...) but should happen sufficiently rarely now that we don't need to worry about it.

@refset refset reopened this Jan 24, 2025
@mpisanko
Copy link
Contributor

cannot reproduce.
tried chrome 122 on arch.
safari 18.2
firefox (both linux and macos)

@mpisanko
Copy link
Contributor

updated chrome to 132 on linux.
needed to absolutely keep banging the keyboard to manage to reproduce it.
seemed to jump to the beginning of first line.
but i really had to be doing it stupid fast - no way could i do it by typing something.

@mpisanko
Copy link
Contributor

now that i noticed how to - i can repro on firefox on macos, but again - same 'technique' - bang a couple keys like there's no tomorrow.

@Akeboshiwind
Copy link
Collaborator

I’ve definitely had this just with regular typing, I’m not the fastest typer either.

@Akeboshiwind
Copy link
Collaborator

Might be worth checking if this is an upstream issue in CodeMirror

@mpisanko
Copy link
Contributor

mpisanko commented Jan 27, 2025 •
edited
Loading

yeah, was just looking:
uiwjs/react-codemirror#694
and less related
codemirror/dev#1473

@mpisanko mpisanko mentioned this issue Jan 28, 2025
Closed
@mpisanko
Copy link
Contributor

actually - seems like there's a 'know phenomenon' of cursor jumps in react controlled inputs
looking at that currently - to see if that can fix the problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@refset @mpisanko @Akeboshiwind