-
Notifications
You must be signed in to change notification settings - Fork 177
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Change : Room Preview #4250
Change : Room Preview #4250
Conversation
📱 Scan the QR code below to install the build (arm64 only) for this PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some things I found. Since @ganfra is away at the moment I'll take over the PR and try fixing those issues.
@@ -80,7 +79,6 @@ fun JoinRoomView( | |||
Box( | |||
modifier = modifier.fillMaxSize(), | |||
) { | |||
LightGradientBackground() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
...joinroom/impl/src/main/kotlin/io/element/android/features/joinroom/impl/JoinRoomPresenter.kt
Outdated
Show resolved
Hide resolved
...es/joinroom/impl/src/main/kotlin/io/element/android/features/joinroom/impl/JoinRoomEvents.kt
Outdated
Show resolved
Hide resolved
ecd45d1
to
dabf209
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving this after a review. I also made some changes to be able to get the banning user's name and the reason for the ban so we have feature parity with iOS.
dabf209
to
12043e1
Compare
…eview screen for banned rooms. Some internal refactoring was done too: - Remove RoomInfo.isPublic to only use JoinRule. - Also take into account restricted access rooms for previews.
…nt user and the sender of its `m.room.member` state event in the room.
12043e1
to
9b9e7b1
Compare
|
Content
This PR takes care to update the
JoinRoom
screen aka Room Preview, it includes :Membership.Banned
orJoinRule.Restricted
There are still a bunch of small things to fix, but I'll iterate again later.
Motivation and context
Closes #4141
Closes #4142
Figma link : https://www.figma.com/design/7TqjqdMBaPpm3IavKsMYu6/Ask-to-join-(Knocking)?node-id=4797-42334&m=dev
Screenshots / GIFs
See recorded screenshots.
Tests
Tested devices
Checklist