-
Notifications
You must be signed in to change notification settings - Fork 233
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
new dm : page to start a new dm conversation #1252
base: main
Are you sure you want to change the base?
new dm : page to start a new dm conversation #1252
Conversation
Implemented the UI for the "No DMs yet" state on the RecentDmConversationsPage according to the design provided. Fixes part of zulip#127
…onversation Implemented the search functionality in the dm page which searches from within the recent conversations. Implemented starting a new dm with other user 1-to-1 or in a group both. the UI is similar to that in the figma design
1666ddf
to
1a5cf9f
Compare
Thanks for the contribution. Before we can review this, it will need tests. See our README: From a quick look at the screenshots, the UI doesn't appear to match the design in Figma. Please take a close look at the details of padding, spacing, color, and other visual elements, and try to exactly match what's in Figma. If there are aspects you don't match or that you're unsure about, please flag them explicitly in the PR. |
The main thing this issue is calling for is to implement the UX design we have that's expressed in Figma, not to design a new UX. There might be details we revise after seeing an implementation. But the starting point should be to implement the design accurately.
If there's something you're running into in the implementation that feels more complex than it should be, the #mobile-dev-help channel is a good place to ask. You'll need to give more details of what it is you're doing and what you find is a challenge with it, and then we can discuss solutions.
This would be a good question to ask in the #mobile-design channel. See this section of our docs: |
This intents to add new dm functionality as mentioned in #127 & #385
First the search functionality within the recent dm's, here the search icon in the design is in the appbar, but I think having a separate search bar would be much better and looks very obvious for the users. other than that the margin for the new DM button is properly set by me also with the case of whether it should be sticked or moved up with the keyboard would be taken care by in the next change,
I have used wrap with SingleChildScrollView with chip for displaying the selected users for staring a dm, an addition here is that there is a small cross button also in the chip when pressed will deselect that user, then after adding all desired users there is a Next button on the appbar when pressed will take it to their messageList.
kindly suggest the required changes and review my suggestion of search box and cross button is selected user chip.