-
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
[APM] Consistent Observability header and navigation redesign #40236
Comments
Pinging @elastic/apm-ui |
@elastic/observability-design I've updated the description with the latest design proposals for the headers (overview and service views). Let me know what you think, I'm trying to put this together for a header change in APM over the next release cycle. |
I think this is great stuff. I'm not 100% sure why the add data button is green and the gear icon is dark but as long as it is consistent with other part of observability I'm fine with that. I also think we should iterate on the filters topic in the near future but it works for know. |
@katrin-freihofner The "green" is just our secondary button color, so it's only to create some separation from the other blue links and buttons (refresh button i.e.). But that said, it's also not the most important option all of the time, so maybe it should be made more subtle. It's just about striking the right balance. But it should be the same style, copy and position for all Observability apps (as you also underlined). Filters are certainly a whole topic on its own 😅 |
The first step is to create a shared header bar component (App icon, navigation and actions). This will be addressed in #48113 |
@formgeist This looks ready to get started. I know there are talks about a shared header component but mostly I think this is a matter of aligning on the same layout. Therefore I don't think we need to wait for such a component to be available. |
@sqren I agree - I'll update the designs to reflect the final consistent header designs that were made by @katrin-freihofner and I think we move this into implementation. When the shared component is available, we can always look at letting it replace our own implementation for consistency. |
Sounds good 👍 |
I've updated the screens and tasks in the description. As mentioned, the Logs UI team are already setting out to build a shared component among Observability (#48113) |
@formgeist Is this still relevant or should it be closed? |
Things have deviated since creating this issue and we have some navigation consistency projects underway, so I'll close this and create a new issue once those navigation designs are decided upon. |
Summary
Based on https://github.com/elastic/observability-dev/issues/150 and some other discussions in the design team around how to manage the general layout and reduce vertical space, we would like to look at how we might condense the APM header to make better use of the space.
Related shared component currently in development #48113
Design solution
APM overview header
condensed
Tabs navigation to the very top for switching between Services and Traces.Service view
The text was updated successfully, but these errors were encountered: