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
Style header group-chats #7269
Style header group-chats #7269
Conversation
Pull Request Checklist
|
Jenkins BuildsClick to see older builds
|
@errorists I am working on styling group chats https://www.figma.com/file/aS1ct66VQ6V0cio7vSqS8UoG/Chat?node-id=375%3A3842
|
@cammellos Yes, delete the group chat and return to home screen. |
(defn number-of-participants-disclaimer [number-of-participants-available] | ||
[react/view {:style styles/number-of-participants-disclaimer} | ||
[react/text (if (> number-of-participants-available | ||
0) |
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.
pos? maybe?
3764cb5
to
5d71fc7
Compare
@cammellos would you mind to rebase it to current develop? |
5d71fc7
to
a6103ee
Compare
@churik done, sorry for the delay, is building now |
Jenkins BuildsClick to see older builds (79)
|
@cammellos so far: 1. Group chat owner (creator) is counted as 1/10 group chat members. In design, owner is not counted as 1/10 possible members. Also there is no search bar on New Group Chat screen. 2. Create New Chat button is hidden under keyboard (IOS) 3. Can't scroll on chat name screen (Android 7.0, LG v20) If I try to scroll with open keyboard, I can see only 6 members - others are hidden under keyboard.4. "Cancel" button is missing on chat name screen 5. "Next" button is overlapping with checkbox on Members screen 6. No design changes on "Group info screen" 7. Can't add members to group chat - there is no button to proceed
8. No changes in menus |
@churik thanks for testing, sorry I should have mentioned which parts were implemented and which were not.
thanks! |
a6103ee
to
3bc546f
Compare
@churik Thanks! |
@cammellos sorry for taking so long, tough question actually but I propose a 0/9. If we went with a 1/10 then somewhere in the UI we'd have to include the user creating a group to make it clear that he's taking one of the spots. |
100% of end-end tests have passed
Passed tests (1) |
@cammellos thanks for quick fixes! |
@cammellos #7269 (comment) is fixed in the last build, so PR can be moved to |
@churik awesome, I will do, thanks! |
100% of end-end tests have passed
Passed tests (58)Click to expand
|
@cammellos after upgrade from 0.9.32 for all group chats created before upgrade I (as an admin) instead of group chat history see the screen where I invited myself to join group chat: UPD: same behavior on nightly, but here "Decline" option is added. Is it possible to prevent displaying this screen for such cases? |
If I'm not mistaken the toolbar changes are local to group chat in this PR? I was thinking of doing the toolbar revamp according to the UI components described here https://www.figma.com/file/cb4p8AxLtTF3q1L6JYDnKN15/Index?node-id=794%3A0
|
@yenda not sure I understand, correct me if I am wrong, this changes only group-chats, not any other kind of chats, is that what you were asking? |
2f62170
to
4990da8
Compare
@churik it should be fixed now, I have also changed the upgrade behavior so that users won't have to join existing chats. If chat has been created in nightly or 0.9.32 (anything before this PR), on upgrade to this PR they won't have to join the chat. It does not matter who creates the chat (which version), it only matters which version of the app was running when the app was created on the device. (not sure if the explanation is clear). thanks |
@cammellos sorry for the confusion I was just curious if you were already on the implementation of top bar components, but it seems that here you just made a local update to fit the group chat design. What I am saying is that the top bar you implemented will soon be the same all accross the app according to designs here https://www.figma.com/file/cb4p8AxLtTF3q1L6JYDnKN15/Index?node-id=794%3A0 I will work on that after discussing with design team tomorrow. I doesn't matter for that PR
|
4990da8
to
2ce7cae
Compare
Note: after upgrade you will see system messages that you automatically joined to chat (if it was created on 0.9.32) Thanks @cammellos, great work here! |
2ce7cae
to
40303f5
Compare
@churik thanks for testing! |
Signed-off-by: Andrea Maria Piana <andrea.maria.piana@gmail.com>
40303f5
to
52ae2c2
Compare
Style group chats according to https://www.figma.com/file/aS1ct66VQ6V0cio7vSqS8UoG/Chat?node-id=375%3A3842
Styles group chats according to designs.
What's missing
Currently no bold on username on chat notifications
Search bar
Group info styling
status: ready