Bug 475009

Summary: Add keyboard navigation to posts with multiple images
Product: [Applications] Tokodon Reporter: Justin Zobel <justin.zobel>
Component: generalAssignee: Unassigned bugs mailing-list <unassigned-bugs>
Status: RESOLVED FIXED    
Severity: wishlist CC: carl, josh
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed In: 23.12.0
Sentry Crash Report:

Description Justin Zobel 2023-09-29 02:19:16 UTC
When you click on an image, you get the > and < arrows to navigate by clicking on them to see the next picture.

It would be great to add keyboard navigation to this, so users could switch between images using the left and right arrow keys on their keyboard.
Comment 1 Joshua Goins 2023-09-29 02:33:59 UTC
I raced here when I got the e-mail :D  I actually fixed this recently! It turns out, the fullscreen image viewer already has these keyboard shortcuts but we weren't focusing it properly, see https://invent.kde.org/network/tokodon/-/merge_requests/365. Feel free to re-open if you already have this commit and find it doesn't work.
Comment 2 Justin Zobel 2023-09-29 03:21:08 UTC
Ah thanks I am only on 23.08.1 so I'll check it out again when 23.12.0 hits!

On 29/9/23 12:03, Joshua Goins wrote:
> https://bugs.kde.org/show_bug.cgi?id=475009
>
> Joshua Goins <josh@redstrate.com> changed:
>
>             What    |Removed                     |Added
> ----------------------------------------------------------------------------
>     Version Fixed In|                            |23.12.0
>               Status|REPORTED                    |RESOLVED
>           Resolution|---                         |FIXED
>
> --- Comment #1 from Joshua Goins <josh@redstrate.com> ---
> I raced here when I got the e-mail :D  I actually fixed this recently! It turns
> out, the fullscreen image viewer already has these keyboard shortcuts but we
> weren't focusing it properly, see
> https://invent.kde.org/network/tokodon/-/merge_requests/365. Feel free to
> re-open if you already have this commit and find it doesn't work.
>