Bug 1220302

Summary:	MComix smart scroll up doesn't work on wayland
Product:	[openSUSE] openSUSE Tumbleweed	Reporter:	piotrus3g <piotrus3g>
Component:	X11 Applications	Assignee:	E-mail List <screening-team-bugs>
Status:	NEW ---	QA Contact:	E-mail List <qa-bugs>
Severity:	Normal
Priority:	P5 - None	CC:	comes, ftake, gcomes.obs, jengelh, piotrus3g
Version:	Current
Target Milestone:	---
Hardware:	x86-64
OS:	openSUSE Tumbleweed
Whiteboard:
Found By:	---	Services Priority:
Business Priority:		Blocker:	---
Marketing QA Status:	---	IT Deployment:	---

Description piotrus3g 2024-02-25 13:24:38 UTC

Operating System: openSUSE Tumbleweed 20240223
KDE Plasma Version: 5.27.10
KDE Frameworks Version: 5.115.0
Qt Version: 5.15.12
Kernel Version: 6.7.5-1-default (64-bit)
Graphics Platform: Wayland
Processors: 12 × AMD Ryzen 5 5625U with Radeon Graphics
Memory: 13,5 GiB of RAM
Graphics Processor: AMD Radeon Graphics
Manufacturer: LENOVO
Product Name: 82SG
System Version: IdeaPad 5 15ABA7

Issue description:
When on wayland, smart scroll in MComix doesn't work as expected, as it's only possible to smart scroll down, but not up.

Observed behavior:
Default keyboard shortcut for smart scroll up (shift + space) does the same thing as smart scroll down (space).

Expected behavior:
Smart scroll down scrolls down, smart scroll up scrolls up.

Additional information:
On X11 smart scrolling works as expected. Other keyboard shortcuts that also use the shift key work properly on wayland and X11.

Comment 1 Giacomo Comes 2024-03-30 05:10:44 UTC

Sorry for the late replay.
I tested smart scroll up (shift + space) and it works with both X11 and Wayland
on current tumbleweed (plasma6).
Do you still have the issue?

Comment 2 piotrus3g 2024-04-01 21:13:59 UTC

(In reply to Giacomo Comes from comment #1)
> Sorry for the late replay.
> I tested smart scroll up (shift + space) and it works with both X11 and
> Wayland
> on current tumbleweed (plasma6).
> Do you still have the issue?

It still doesn't work on wayland plasma 6 with the default shift + space. What I found out though is that when I set it to something else (I set it to ctrl + space for testing), it works properly.

Comment 3 Jan Engelhardt 2024-04-23 08:46:15 UTC

I tested

* Xorg/xfce > weston[X11 backend] > mcomix[Wayland backend]
* fbcon > weston[libdrm backend] > mcomix[Wayland backend]

in both cases, Space goes forward, and Ctrl-Space goes backward as expected.

Versions: Tumbleweed 20240404(not pure/mixed with 20240419), weston-13.0.0, libinput-1.25.0, libxkbcommon-1.7.0, mcomix-2.3.0

Comment 4 Jan Engelhardt 2024-04-23 08:46:41 UTC

...shift-space that is.

Comment 5 Giacomo Comes 2024-04-23 12:25:30 UTC

(In reply to piotrus3g from comment #2)
> It still doesn't work on wayland plasma 6 with the default shift + space.
> What I found out though is that when I set it to something else (I set it to
> ctrl + space for testing), it works properly.

That sounds more like the combination shift + space is intercepted by some other
application and never reaches mcomix.

You could try a fresh installation (only kde role + mcomix) and test the issue.
If you don't want to wipe your current installation you could use a spare 
PC, a spare hard disk or a virtual machine.
If, in the fresh installation, you don't see the problem, then the issue you 
observe is most probably due to your current setup.

Comment 6 Takashi Iwai 2024-04-23 13:54:36 UTC

Isn't it IBus or such input engine that catches that key combo?

Comment 7 Giacomo Comes 2024-04-23 17:05:57 UTC

(In reply to Takashi Iwai from comment #6)
> Isn't it IBus or such input engine that catches that key combo?

I haven't dealt with IBus in a long time.
After enabling it, I verified that it does not interfere with mcomix
(shift + space still works).
Also while in the past the combination shift + space probably was used to
to select the next input method, nowadays such combination is super + space.

Comment 8 Takashi Iwai 2024-04-24 09:04:12 UTC

Then I guess it's rather an issue in Wayland implementation, not an issue of the application side.  Tossed back.

Comment 9 Jan Engelhardt 2024-04-24 09:37:37 UTC

The ibus theory is easy to test: just stop ibus.