elastic / kibana

Your window into the Elastic Stack
https://www.elastic.co/products/kibana
Other
19.64k stars 8.23k forks source link

Side Nav Serverless Requirements for April Integration Checkpoint #153945

Closed petrklapka closed 1 year ago

petrklapka commented 1 year ago

The side navigation menu component is to be modified to meet the following needs in support of the Serverless Initiative.

1. Left nav

Appearance

Root Level Menu Item Behavior

The root menu items defined above shall have the following behavior

Additional Programmatic Functionality

This section defines additional behaviors and functionality required at a programmatic level that are not covered in the Appearance and Root Level Menu Behavior sections.

Out of scope for April Checkpoint

(This is a general consensus as of writing; not set in stone)

elasticmachine commented 1 year ago

Pinging @elastic/appex-sharedux (Team:SharedUX)

tsullivan commented 1 year ago

There was an ask from the Security Solution team (@YulNaumenko) that the nav component package provide a way for solutions to import Platform navigation items, with the intention of placing them at-will within the Solution area and hiding them from the Platform area.

Example of how it would be used: Observability wants to show a link to Logs Anomalies from ML in the nav under Observability > Logs > Anomalies. The Anomalies link is in the Platform area of the nav under Machine Learning. Observability should hide the anomalies link under Machine Learning, and "import" that link so it can be showed in their solution area.

rayafratkina commented 1 year ago

@tsullivan the capability to import individual items from the Platform area is something we need to have for MVP, but it's not a requirement for the first checkpoint. Should we start another issue with the remaining requirements so we can track them without adding scope to the initial work?

tsullivan commented 1 year ago

@rayafratkina I have created new issues to track progress, and each can be moved around or changed in priority as needed. The capability in question in tracked here: https://github.com/elastic/kibana/issues/154486 and we can have a separate discussion there about it.

yuliacech commented 1 year ago

I believe I was pinged by mistake, probably meant for @YulNaumenko

tsullivan commented 1 year ago

I've updated 20/35 of the checklist items to reflect the state of implementation in this (currently Draft) PR: https://github.com/elastic/kibana/pull/152510

petrklapka commented 1 year ago

The MVP Left Nav implementation for the April Integration checkpoint was delivered as a POC, then subsequently merged into main. Chrome polish tasks that remain are being tracked as separate issues.