elastic / kibana

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

[Meta] Analyst app layout functionality for PDF/PNG Reporting #191532

Open tsullivan opened 2 months ago

tsullivan commented 2 months ago

This is a meta issue to collect issues and enhancements that pertain to how the Dashboard/Visualize/Canvas apps generate params for a report job, and how they behaves when a screenshot report is running.

elasticmachine commented 2 months ago

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

elasticmachine commented 2 months ago

Pinging @elastic/kibana-presentation (Team:Presentation)

teresaalvarezsoler commented 1 month ago

hey @tsullivan is this one going to be done by the shared UX team? It has been requested multiple times and I'm happy to remove it from our backlog but just to be sure https://github.com/elastic/kibana/issues/17605. Thanks.

tsullivan commented 1 month ago

@teresaalvarezsoler that issue should be done by the presentation team. It pertains to the dimensions of the area that is captured in the screenshot, which is under the control of the dashboard application.

From https://www.elastic.co/guide/en/kibana/current/reporting-integration.html#_how_applications_make_themselves_screenshot_capable:

The preserve layout takes a screenshot of the element with the data-shared-items-container attribute.

That documentation helps explain that to address the issue, the data-shared-items-container element of the dashboard needs to include the filters area of the UI.

There is also a way to customize the UI for the screenshotting view, by hiding things or rendering things in a "read" state instead of a editable state. Let me know if I can help with that further.

teresaalvarezsoler commented 1 month ago

Ok, then I will unlink it from here and link it to our Usability enhancements issue (same issue can't have two parents in GH)