-
Notifications
You must be signed in to change notification settings - Fork 109
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
Add an empty state to the room list #2342
Add an empty state to the room list #2342
Conversation
- Make `RoomListDataSource.allRooms` a `SharedFlow` so we can know when we don't have a value yet. - Map its output in `RoomListPresenter` to `AsyncData`. - Display the new empty state when the room list has loaded and has no items.
…RoomSummaryFactory.createFakeList` functions static ones
- Instead of an extra item, use `contentPadding`. - Make sure each item has an unique id.
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
📱 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.
LGTM, thanks. I have one question, that I have also asked on Figma.
Not blocking the 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.
I would remove the FAB in the case of an empty list, here the added button has the same effect.
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.
I made the same exact suggestion to @amshakal 😅 . She said it was to also make the meaning of the FAB easier for the user to understand or something along those lines. I'm ok with either leaving it or removing it, it shouldn't be much work.
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.
But yes, it's probably something we can handle in some other PR.
Type of change
Content
RoomListDataSource.allRooms
aSharedFlow
so we can know when we don't have a value yet.RoomListPresenter
toAsyncData
.contentPadding
for the room list bottom padding instead of an extra item.Motivation and context
Closes #2330.
Screenshots / GIFs
Tests
Tested devices
Checklist