Bug 498116 - Image view is irresponsive on mobile
Summary: Image view is irresponsive on mobile
Status: REPORTED
Alias: None
Product: Koko
Classification: Applications
Component: General (show other bugs)
Version: 24.12.0
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Unassigned bugs mailing-list
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-01-01 10:23 UTC by Anders Lund
Modified: 2025-01-02 15:37 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Anders Lund 2025-01-01 10:23:40 UTC
***
If you're not sure this is actually a bug, instead post about it at https://discuss.kde.org

If you're reporting a crash, attach a backtrace with debug symbols; see https://community.kde.org/Guidelines_and_HOWTOs/Debugging/How_to_create_useful_crash_reports

Please remove this comment after reading and before submitting - thanks!
***

SUMMARY
On mobile, tapping the image view has no effect, preventing from hiding the UI elements in top and bottom.


STEPS TO REPRODUCE
1. Open koko (aka Photos) on a phone or tablet (using QT_QUICK_CONTROLS_MOBILE=! on a pc/laptop won't work unless you have a touch screen, mouse click click is still registered but produces some qml error oupput)
2. open an image
3. tap the image

OBSERVED RESULT
nothing happens

EXPECTED RESULT
ui (toolbars) would show/hide

SOFTWARE/OS VERSIONS
Windows: 
macOS: 
(available in the Info Center app, or by running `kinfo` in a terminal window)
Linux/KDE Plasma: postmarket OS
KDE Plasma Version: 6.2.80
KDE Frameworks Version: 6.8 
Qt Version: 6.8.1

ADDITIONAL INFORMATION

I'm willing to try and take this on. Likely, it is a question of using a taphandler insted of onClicked, though I did not dive that much into the code yet.
I'd also like to add support for switching to full screen mode on mobile, which currently requires to display the keyboard and press 'F', and then it's hard to get out of. I suggest using longpress for that. It' my opinion that fullscreen should be default for image view on mobile devices.
Comment 1 Anders Lund 2025-01-02 15:37:35 UTC
I have tested some more. Using my touch sceen, both on pc and mobile, I can navigate - drag or tap the round buttons (on pc), but tapping in image is not registered. I even compiled koko and tried adding a console.log message in the onclicked function in BaseImageDelegate.qml. So the tap must be lost somewhere else in the stack - but where? Running in mobile mode on PC, clicking still works.
The result is that using touch, you can't zoom image to full size by double tapping, nor can you hide the toolbars.