Bug 1220302 - MComix smart scroll up doesn't work on wayland
Summary: MComix smart scroll up doesn't work on wayland
Status: NEW
Alias: None
Product: openSUSE Tumbleweed
Classification: openSUSE
Component: X11 Applications (show other bugs)
Version: Current
Hardware: x86-64 openSUSE Tumbleweed
: P5 - None : Normal (vote)
Target Milestone: ---
Assignee: E-mail List
QA Contact: E-mail List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-02-25 13:24 UTC by piotrus3g
Modified: 2024-05-21 07:56 UTC (History)
5 users (show)

See Also:
Found By: ---
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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.