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

improve language swipe cycling #656

Closed
wants to merge 4 commits into from

Conversation

codokie
Copy link
Contributor

@codokie codokie commented Apr 6, 2024

I've added a setting to adjust the sensitivity of the language swipe/slide gesture distance sensitivity.
The current step value does not work well on some devices like S22+.
Here is the current behavior:

Click to expand
High.mp4

And with the new default Medium option selected:

Click to expand
medium.mp4
fixes #255 and #614

@RHJihan
Copy link
Contributor

RHJihan commented Apr 6, 2024

Sensitivity 20 is very low for vertical sliding. I am not sure about this approach. Related issue: #255

@RHJihan
Copy link
Contributor

RHJihan commented Apr 6, 2024

Low sensitivity kind of fixes the flickering problem. However, it requires more travel distance and, so, more time to type. This is a drawback. To address this trade-off, perhaps we could consider other options, like the ones mentioned in #255, such as switching only one language per swipe or stopping the cycle at the end of the enabled language.

This PR is a positive step forward. I'm just curious if there might be an even more optimal solution to this flickering issue.

I'm curious if @nach0 has any thoughts on this approach.

@Helium314
Copy link
Owner

Just some thoughts, haven't read up on the old issues yet. (so some parts below might be outdated)

If sensitivity is too high for swipe to be usable, does it also apply to cursor movement?
My reasoning was that if a user is able to place the cursor with character accuracy, they should be able to to select a language with quarter that accuracy.
If the general sensitivity is too high, maybe the setting could adjust pointerStep instead?

In my opinion a slider is preferable over a bunch of fixed values, because it allows more fine-grained tuning and requires fewer strings, so less work for translators.

@RHJihan
Copy link
Contributor

RHJihan commented Apr 6, 2024

Should I pursue both solutions?

I am unsure about which approach would be better. Or implementing both will be a good decision? I think it will be better than the current situation. However, I use only two subtypes, I cannot give a final though on this.
But I don't think anyone would be unhappy if one language change per swipe is a tweakable option.

replaced options with slider
renamed "sensitivity" with "distance"
@codokie codokie changed the title language swipe sensitivity options improve language swipe cycling Apr 6, 2024
@codokie codokie closed this Apr 6, 2024
@codokie codokie deleted the lang_swipe_sensitivity branch April 6, 2024 22:10
@codokie codokie restored the lang_swipe_sensitivity branch April 6, 2024 22:10
@codokie codokie reopened this Apr 6, 2024
@RHJihan
Copy link
Contributor

RHJihan commented Apr 7, 2024

works perfectly. loved it

@RHJihan
Copy link
Contributor

RHJihan commented Oct 10, 2024

@Helium314, just wanted to kindly check in on the status of this PR. I know you're busy, but this PR would be a huge update for me. I’d really appreciate it if it could be reviewed when you have a moment. Thank you!

@codokie
Copy link
Contributor Author

codokie commented Jan 23, 2025

Closed due to @Helium314 unwillingness to work with me.

#1019 (comment)

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 this pull request may close these issues.

Refinement of Space bar language slide Mechanism for Improved User Interaction
3 participants