~mil/sxmo-devel

This thread contains a patchset. You're looking at the original emails, but you may wish to use the patch review UI. Review patch
4 4

[PATCH sxmo-utils] sxmo_hook_inputhandler.sh: firefox - removed right hand side actions as it conflicts with the scrollbar.

Details
Message ID
<20241212022637.25056-1-hazardchem@disroot.org>
DKIM signature
pass
Download raw message
Patch: +7 -0
Signed-off-by: hazardchem <hazardchem@disroot.org>
---
 configs/default_hooks/sxmo_hook_inputhandler.sh | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/configs/default_hooks/sxmo_hook_inputhandler.sh b/configs/default_hooks/sxmo_hook_inputhandler.sh
index a443e1a0..c5c2d184 100755
--- a/configs/default_hooks/sxmo_hook_inputhandler.sh
+++ b/configs/default_hooks/sxmo_hook_inputhandler.sh
@@ -140,6 +140,13 @@ case "$WMCLASS" in
				;;
		esac
		;;
	*"firefox"*)
		case "$ACTION" in
			# you're probably dragging the scrollbar
			*"uprightedge") exit 0 ;;
			*"downrightedge") exit 0 ;;
		esac
		;;
	*"foot"*|*"st"*|*"vte"*|"terminal") # Terminals
		case "$WMCLASS" in # Handle programs without touch support
			*"st"*)
-- 
2.45.2

[sxmo-utils/patches/test.yml] build success

builds.sr.ht <builds@sr.ht>
Details
Message ID
<D69D82Y24332.1F4SSGBWU6RD0@fra02>
In-Reply-To
<20241212022637.25056-1-hazardchem@disroot.org> (view parent)
DKIM signature
missing
Download raw message
sxmo-utils/patches/test.yml: SUCCESS in 24s

[sxmo_hook_inputhandler.sh: firefox - removed right hand side actions as it conflicts with the scrollbar.][0] from [hazardchem][1]

[0]: https://lists.sr.ht/~mil/sxmo-devel/patches/56387
[1]: hazardchem@disroot.org

✓ #1388091 SUCCESS sxmo-utils/patches/test.yml https://builds.sr.ht/~mil/job/1388091
Details
Message ID
<D7JU5CPQQ3JU.3V55XSFIYTAPT@momi.ca>
In-Reply-To
<20241212022637.25056-1-hazardchem@disroot.org> (view parent)
Sender timestamp
1738661151
DKIM signature
pass
Download raw message
On Wed Dec 11, 2024 at 6:26 PM PST, hazardchem wrote:
> Signed-off-by: hazardchem <hazardchem@disroot.org>
> ---
>  configs/default_hooks/sxmo_hook_inputhandler.sh | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/configs/default_hooks/sxmo_hook_inputhandler.sh b/configs/default_hooks/sxmo_hook_inputhandler.sh
> index a443e1a0..c5c2d184 100755
> --- a/configs/default_hooks/sxmo_hook_inputhandler.sh
> +++ b/configs/default_hooks/sxmo_hook_inputhandler.sh
> @@ -140,6 +140,13 @@ case "$WMCLASS" in
>  				;;
>  		esac
>  		;;
> +	*"firefox"*)
> +		case "$ACTION" in
> +			# you're probably dragging the scrollbar
> +			*"uprightedge") exit 0 ;;
> +			*"downrightedge") exit 0 ;;
> +		esac
> +		;;
>  	*"foot"*|*"st"*|*"vte"*|"terminal") # Terminals
>  		case "$WMCLASS" in # Handle programs without touch support
>  			*"st"*)

I dont use the scrollbar when Im using firefox...
I usually swipe up and down in the middle of the screen.

Infact, I would prefer disabling the scrollbar on sxmo via the firefox
mobile config on alpine.

I dont think we should have different gestures behaviour on the right
side of the screen when firefox is in focus, that will be confusing for
users - what if I really wanted to lower my volume while using firefox?
Details
Message ID
<Z6KKqaES440C0Dbj@oneplus-enchilada>
In-Reply-To
<D7JU5CPQQ3JU.3V55XSFIYTAPT@momi.ca> (view parent)
Sender timestamp
1738741577
DKIM signature
pass
Download raw message
> I dont use the scrollbar when Im using firefox...
> I usually swipe up and down in the middle of the screen.
> 
> Infact, I would prefer disabling the scrollbar on sxmo via the firefox
> mobile config on alpine.

Scrollbar is very useful if you are viewing a large page and want to jump back
to a specific location.

> I dont think we should have different gestures behaviour on the right
> side of the screen when firefox is in focus, that will be confusing for
> users - what if I really wanted to lower my volume while using firefox?

Right hand side input is up and down arrow keys, which was causing jarring
behaviour when using scrolling or scrollbar from the right hand side.

Volume controls are on the left hand side which are unaffected by this patch.
Details
Message ID
<D7R4IYNBLMH8.3OEFB92Q97X03@willowbarraco.fr>
In-Reply-To
<20241212022637.25056-1-hazardchem@disroot.org> (view parent)
Sender timestamp
1739433748
DKIM signature
pass
Download raw message
We already disable this specific gesture for some other programs. Also,
I agree that the Firefox scrollbar is valuable. Those right gesture
by default only trigger some Up or Down keys.

Thanks!

To git@git.sr.ht:~mil/sxmo-utils
   b9821b93..b1ed8c8a  master -> master
Reply to thread Export thread (mbox)