-
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
[DOCS] Empty dashboard screen #55727
Conversation
Pinging @elastic/kibana-docs (Team:Docs) |
docs/user/dashboard.asciidoc
Outdated
@@ -137,26 +152,21 @@ tools. To create a short URL, you must have write access to {kib}. | |||
=== Import and export dashboards | |||
|
|||
To import and export dashboards, go to *Management > Saved Objects*. For details, | |||
see <<managing-saved-objects, Managing saved objects>>. | |||
see <<managing-saved-objects, Managing saved objects>>. | |||
|
|||
[float] | |||
[[viewing-detailed-information]] | |||
=== Inspect and edit elements |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This section feels like it belongs on a separate page. Also, it needs more information about why you would want to inspect and edit an element. Maybe a separate PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm moving this section up. I inspect/edit elements as I'm arranging them on the dashboard, and I definitely do this before I save the dashboard.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
* [DOCS] AdEmpty dashboard screen redesign * Review comments f from Gail * Comments from Gaiail pt 2
* [DOCS] AdEmpty dashboard screen redesign * Review comments f from Gail * Comments from Gaiail pt 2
Summary
Added the docs for:
[skip ci]
You can view the docs here.
Checklist
Use
strikethroughsto remove checklist items you don't feel are applicable to this PR.[ ] This was checked for cross-browser compatibility, including a check against IE11[ ] Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support[ ] Unit or functional tests were updated or added to match the most common scenarios[ ] This was checked for keyboard-only and screenreader accessibilityFor maintainers
[ ] This was checked for breaking API changes and was labeled appropriately[ ] This includes a feature addition or change that requires a release note and was labeled appropriately