-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Untangling Emails: UI and email forward #98681
Comments
More context: p9Jlb4-eCp-p2 |
Highlighting some related issues: |
Hey @crisbusquets, we at Vertex will be working on this. I grabbed this issue that @arunsathiya posted to track engineering work |
Great, @escapemanuele ! Aaaaand we're working together again 🙏 |
Hey @crisbusquets, @alshakero already worked on something that you can try here #98837! ![]() |
Hi @alshakero , @escapemanuele and @crisbusquets Here’s some initial feedback I’ve put together so far testing #98837 : View forwards(I'm compartmentalizing my feedback in two different comments. After this one I'll add another comment with the Add forwards feedback.)
I’ve included some quick mockups to illustrate this feedback. ✨ These are just rough sketches, but I hope they help bring the ideas to life! Let me know if you have any questions or thoughts—I’m happy to tweak further if needed. 😊 ![]() ![]() @alshakero To reference our Slack conversation, my first instinct is that we could solve this table display with the dataViews component. I’m going to spend some time exploring this approach and will share mockups soon. I know you mentioned concerns about showing nesting, so I’ll need a few hours to think through it. |
Thank you @auareyou, we'll check the feedback and address it ✌ |
This looks great, @auareyou ! Thanks for on this :) I'm leaning towards Option 2 here, perhaps without the arrow. It’s easier to scan and read the table when all the information is aligned in a row. From what I understand, if there are multiple 'To' emails, they will appear one below the other, correct? I guess this isn't relevant now (we need to migrate this to Core), but just a heads up that we have a similar UI in Calypso when a user can invite someone to their site ( ![]() ![]() About this:
I think this will make the UI unnecessarily complex. Let’s first focus on guiding users to complete the task at hand with the |
@auareyou Beautiful work. Thank you!
I just read this part "they will appear one below the other, correct?". This would be jarring IMO. Because each email can need verification, has a resend button, and a remove button. How would that look if they're were stacked in one row? Unless you mean each item will be in a row? |
Love it! |
Context: p58i-jdB-p2
What
Ensure that the email view is using the Core components, and it supports email forward to multiple emails.
Why
Currently, the UI looks like this, and it doesn't support multiple emails (Support Docs).
The text was updated successfully, but these errors were encountered: