Page MenuHomePhorge
Differential D84

Make JoinRoomPage auto-closable
ClosedPublic
Actions

Authored by tusooa on Jun 30 2024, 8:28 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Sep 5, 6:21 AM
Unknown Object (File)
Tue, Sep 3, 8:10 PM
Unknown Object (File)
Tue, Sep 3, 8:10 PM
Unknown Object (File)
Tue, Sep 3, 8:09 PM
Unknown Object (File)
Mon, Sep 2, 10:33 PM
Unknown Object (File)
Mon, Sep 2, 3:38 PM
Unknown Object (File)
Sun, Sep 1, 5:28 AM
Unknown Object (File)
Thu, Aug 29, 9:38 AM
View All 86 Files
Subscribers
None

Details

Reviewers
nannanko
Group Reviewers
O1: the Kazv Project
Commits
rK035c218fda35: Make JoinRoomPage auto-closable
Summary

We missed JoinRoomPage in https://iron.lily-is.land/D23 because it is using Kirigami.Page (not Kirigami.ScrollablePage). This commit makes it behave as the other pages.

Type: fix

Test Plan

Verify that you can close a JoinRoomPage by clicking the left arrow on the top-left.

Diff Detail

Repository
rK kazv
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tusooa created this revision.Jun 30 2024, 8:28 AM
Owners added a reviewer: O1: the Kazv Project.Jun 30 2024, 8:28 AM
Harbormaster returned this revision to the author for changes because remote builds failed.Jun 30 2024, 8:44 AM
Harbormaster failed remote builds in B122: Diff 195!
tusooa updated this revision to Diff 196.Jun 30 2024, 8:47 AM

Fix tests under KF6

Harbormaster completed remote builds in B123: Diff 196.Jun 30 2024, 9:03 AM
tusooa requested review of this revision.Jun 30 2024, 9:03 AM
nannanko accepted this revision.Jul 3 2024, 6:10 PM
This revision is now accepted and ready to land.Jul 3 2024, 6:10 PM
Closed by commit rK035c218fda35: Make JoinRoomPage auto-closable. · Explain WhyJul 3 2024, 6:39 PM
This revision was automatically updated to reflect the committed changes.
tusooa added a commit: rK035c218fda35: Make JoinRoomPage auto-closable.

Revision Contents
Changeset List

PathSizePackages
src/
contents/
ui/
2 linesO1: the Kazv Project
tests/
quick-tests/
6 linesO1: the Kazv Project

Diff 221

View Options

src/contents/ui/JoinRoomPage.qml

Loading...
View Options

src/tests/quick-tests/tst_JoinRoomPage.qml

Loading...