-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[User Experience] Proposal for design polish and improvements to the view #78261
Comments
Pinging @elastic/uptime (Team:uptime) |
Thanks for this @formgeist ! We've been doing bits and pieces of this so some of these may already be covered. Let me address your points: 1 - This value is being removed so moot point now |
Thanks @drewpost - I've commented back on the ones that you had questions for. Happy that the rest was useful...
Could it be "Slowest long task duration"?
Alright, perhaps assumed that the Elastic.co website was a good use-case. I just though it was strange to see the Frontend display 0.88 s instead of 880 ms. Just thought the format would be more dynamic and based on typical thresholds, instead of a hard set value e.g. seconds.
So I'm propose to flip the map and pie charts so the relevant content is vertically aligned. Right now, the page duration visualizations are across from each other and the page views are across from each other. I understand they're in different sections, but I didn't think flipping the map and pie charts would confuse anyone, and the benefit would be to have the page view information together vertically. Before After
Yeah, I think that can work too. Probably still need a panel title and then a selector. Will the metric aggregation only be for those metrics displayed or also for the Core web vitals underneath? Because then it might be better to have the aggregation selector by the section title "User experience" - thoughts? |
Thanks @formgeist 2 - I like the idea but I think we need another synonym for duration. Slow implies speed and it really isn't a speed thing. It's more about the duration that task was running. 5 - so for the front end, if it's below 1s it shows ms but above, uses the 3.4 sec type format? I'd be happy with that. 7/8 - ahhhh, that makes perfect sense and I like the balance. I'd say go for it. 11 - Good point. It will apply to the user experience metrics only. The Core Vitals are basically mini distributions already so wouldn't be impacted. @shahzad31 over to you for how you'd like to handle this (ie a single ticket, etc) |
@drewpost Small question; for the Core web vitals distribution definitions, we're pretty close to the Google definitions, but "Average" is "Needs improvement". Was there a decision not to align there? |
@formgeist Great catch. It should match Google's definition 100% - we missed that one |
OK, I'll add this to the list above ^ |
converted bullet points to checkboxes to mark them as complete as we work on them. I will pick few them in ongoing PR's for rest might do a single pr. |
@shahzad31 sounds good |
@formgeist reduced a text size a bit from your figma file for needs improvement change |
My understanding is that #79824 will resolve items 11, 14 and 17. @shahzad31, you mentioned that (15) can't be done (make |
@paulb-elastic @shahzad31 I think we have since tweaked the headings a bit, so the section titles are not needed at this time. I'd consider 15 not needed anymore. |
Closing, since all points have been addressed |
Summary
I've been meaning to summarize some suggestions for design polish and improvements for the new User Experience - in an effort to round off the app before release.
I've put together a Figma mock that exemplifies a lot of the suggested improvements but also has some clarifying questions around what's being displayed. Feel free to comment back in the Figma file. I've listed them below as well, for those not familiar with Figma or have trouble viewing them.
→ Figma review file
- [ ] 2. "Longest task duration"?- [ ] 15. Increase section title (euiTitle
default size)takes
tois
for the mouse over description on CLS onlycc @drewpost @shahzad31 @elastic/observability-design
The text was updated successfully, but these errors were encountered: