Bug 511295 - Back button in PageRow is always disabled when LayoutMirroring is active
Summary: Back button in PageRow is always disabled when LayoutMirroring is active
Status: RESOLVED FIXED
Alias: None
Product: frameworks-kirigami
Classification: Frameworks and Libraries
Component: general (other bugs)
Version First Reported In: 6.19.0
Platform: Arch Linux Linux
: NOR major
Target Milestone: Not decided
Assignee: kdelibs bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-10-28 23:17 UTC by Youssef Al-Bor3y
Modified: 2025-12-14 04:38 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In: 6.21
Sentry Crash Report:


Attachments
a video shows the problem (491.78 KB, video/mp4)
2025-10-28 23:17 UTC, Youssef Al-Bor3y
Details
an image shows the corruption (119.78 KB, image/png)
2025-12-14 04:06 UTC, Youssef Al-Bor3y
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Youssef Al-Bor3y 2025-10-28 23:17:10 UTC
Created attachment 186272 [details]
a video shows the problem

SUMMARY
the back button is disabled, This problem not only exists in KDE Connect but in all KDE application even in "get new" window (the window that let me download things from KDE store)

STEPS TO REPRODUCE
1. Change system language to Arabic
2. Open KDE Connect Application
3. Go to "Plugins Settings"
4. Try to click on the back button

OBSERVED RESULT
The back button is disabled

EXPECTED RESULT
going to previous page

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: Arch Linux
KDE Plasma Version: 6.5.0
KDE Frameworks Version: 6.5.0
Qt Version: 6.10.0

ADDITIONAL INFORMATION
This issue exists even before this version but i don't know for how long
Comment 1 cwo 2025-11-13 20:40:51 UTC
Thanks for the bug report! I can reproduce it, and I think I know what's wrong.
Comment 2 Bug Janitor Service 2025-11-13 21:10:29 UTC
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/kirigami/-/merge_requests/1962
Comment 3 Youssef Al-Bor3y 2025-11-14 13:55:30 UTC
(In reply to cwo from comment #1)
> Thanks for the bug report! I can reproduce it, and I think I know what's
> wrong.

Thanks!
Comment 4 cwo 2025-11-14 14:20:06 UTC
(In reply to Al-Bor3y from comment #3)
> (In reply to cwo from comment #1)
> > Thanks for the bug report! I can reproduce it, and I think I know what's
> > wrong.
> 
> Thanks!

FWIW, the keyboard shortcut Alt+Left works even with the bugged back button, so you can use that in the meantime if you get stuck until the bug is fixed and the fix released.
Comment 5 cwo 2025-11-18 20:02:22 UTC
Git commit 848cfe32058718d30056c45833de540b186e3ec9 by Christoph Wolk.
Committed on 18/11/2025 at 16:17.
Pushed by cwo into branch 'master'.

globaltoolbar/NavigationButtons: fix Back button in RtL

The Back button tests whether it should be enabled using the pageStack's
contentX. In LtR, this works, but in RtL it does not, as the top-most
page is also the left-most and therefore has an x-axis position of 0. As
a consequence, the Back button is always disabled in RtL.

Instead, we make this test conditional on LayoutMirroring, keeping the
current method for LtR, and testing whether the position is at 0 and the
total contentWidth of the page stack is wider than the pageStack (and
therefore some parts of the stack must be outside the visible area).
FIXED-IN: 6.21

M  +5    -1    src/controls/private/globaltoolbar/NavigationButtons.qml

https://invent.kde.org/frameworks/kirigami/-/commit/848cfe32058718d30056c45833de540b186e3ec9
Comment 6 Youssef Al-Bor3y 2025-12-14 04:04:15 UTC
(In reply to cwo from comment #5)
> Git commit 848cfe32058718d30056c45833de540b186e3ec9 by Christoph Wolk.
> Committed on 18/11/2025 at 16:17.
> Pushed by cwo into branch 'master'.
> 
> globaltoolbar/NavigationButtons: fix Back button in RtL
> 
> The Back button tests whether it should be enabled using the pageStack's
> contentX. In LtR, this works, but in RtL it does not, as the top-most
> page is also the left-most and therefore has an x-axis position of 0. As
> a consequence, the Back button is always disabled in RtL.
> 
> Instead, we make this test conditional on LayoutMirroring, keeping the
> current method for LtR, and testing whether the position is at 0 and the
> total contentWidth of the page stack is wider than the pageStack (and
> therefore some parts of the stack must be outside the visible area).
> FIXED-IN: 6.21
> 
> M  +5    -1    src/controls/private/globaltoolbar/NavigationButtons.qml
> 
> https://invent.kde.org/frameworks/kirigami/-/commit/
> 848cfe32058718d30056c45833de540b186e3ec9

it's fixed but now the animation stopped working and in kde connect the layout is corrupted
Comment 7 Youssef Al-Bor3y 2025-12-14 04:06:59 UTC
Created attachment 187620 [details]
an image shows the corruption
Comment 8 Youssef Al-Bor3y 2025-12-14 04:09:58 UTC
(In reply to Al-Bor3y from comment #6)
> (In reply to cwo from comment #5)
> > Git commit 848cfe32058718d30056c45833de540b186e3ec9 by Christoph Wolk.
> > Committed on 18/11/2025 at 16:17.
> > Pushed by cwo into branch 'master'.
> > 
> > globaltoolbar/NavigationButtons: fix Back button in RtL
> > 
> > The Back button tests whether it should be enabled using the pageStack's
> > contentX. In LtR, this works, but in RtL it does not, as the top-most
> > page is also the left-most and therefore has an x-axis position of 0. As
> > a consequence, the Back button is always disabled in RtL.
> > 
> > Instead, we make this test conditional on LayoutMirroring, keeping the
> > current method for LtR, and testing whether the position is at 0 and the
> > total contentWidth of the page stack is wider than the pageStack (and
> > therefore some parts of the stack must be outside the visible area).
> > FIXED-IN: 6.21
> > 
> > M  +5    -1    src/controls/private/globaltoolbar/NavigationButtons.qml
> > 
> > https://invent.kde.org/frameworks/kirigami/-/commit/
> > 848cfe32058718d30056c45833de540b186e3ec9
> 
> it's fixed but now the animation stopped working and in kde connect the
> layout is corrupted

i don't know if i should report it as a new bug or not
Comment 9 cwo 2025-12-14 04:15:05 UTC
Separate issue, likely already fixed with https://invent.kde.org/frameworks/kirigami/-/merge_requests/1990
Comment 10 Youssef Al-Bor3y 2025-12-14 04:38:29 UTC
(In reply to cwo from comment #9)
> Separate issue, likely already fixed with
> https://invent.kde.org/frameworks/kirigami/-/merge_requests/1990

Ok, I'll create a new ticket for the animation issue