App: [$250] [Group Chats] [Polish] Move "Leave" button into a row of the Report Details page
As discussed with @shawnborton and @JmillsExpensify we want to update the design of the Report Details page with the following changes:
- Move the “Share” row into the
ChatDetailsQuickActionsBar - Move the “Leave” button out of this component and into it’s own row here.
Result:
Upwork Automation - Do Not Edit
- Upwork Job URL: https://www.upwork.com/jobs/~01b00ddeb3014a078a
- Upwork Job ID: 1779985331229118464
- Last Price Increase: 2024-04-22
- Automatic offers:
- DylanDylann | Contributor | 0
- ZhenjaHorbach | Contributor | 0
About this issue
- Original URL
- State: open
- Created 3 months ago
- Comments: 44 (34 by maintainers)
Here is a better mockup to reference:
PR will be ready tomorrow
I think this conversation could be taking a bit of an unproductive turn. I have the following thoughts:
as you can see after the second proposal was posted I stopped editing my proposal Although I still had ideas on how to improve it
And in my original proposal I already provided links to components that need to be changes The remaining changes were related to technical implementation Which in most cases is optional
But anyway Maybe I’m wrong and I did something bad So let’s wait for the decision
And thank you for your opinion
@DylanDylann another thing you should consider please: @ZhenjaHorbach broke the contributions rules.
as you can see on the screenshots, he edited his proposal many times without any mentions. And this is against our contribution rules.
We discussed this on Slack many times and this behavior should be stopped.
This is probably just a mistake from our Figma files. We want leave to always be a row at the bottom of the option rows, it would never be a button in the horizontal button stack.
@dragnoir we should confirm requirements clearly before updating the proposal to save effort 😄
@dragnoir same as the other issue, but posting here for posterity:
Figma link
@DylanDylann
I think we can ignore the avatar for the purposes of this issue. We’re focusing on the “Leave” + “Share” actions in this ticket.
We did this largely to focus the introduction of this behavior and allow testing it for Group Chats in isolation. But I think ideally we would take this opportunity to standardize the UI for this. What do you think @JmillsExpensify?
Since the “Leave” action is the only one with a behavior specific to Group Chats - we should be able to show this for Rooms as well. But we can investigate as part of this issue to make sure that assumption is correct.
Taking over as C+ role