Skip to content
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

[meta] iFrame embedding inconsitencies #93200

Closed
Dosant opened this issue Mar 2, 2021 · 7 comments
Closed

[meta] iFrame embedding inconsitencies #93200

Dosant opened this issue Mar 2, 2021 · 7 comments
Labels
Feature:Dashboard Dashboard related features Feature:Embedding Embedding content via iFrame impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. loe:small Small Level of Effort Meta Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas

Comments

@Dosant
Copy link
Contributor

Dosant commented Mar 2, 2021

This is an umbrella issue to discuss and work through various and very similar iFrame embedding clientside bugs like:

A lot of the bugs are:

  1. Land in a dashboard inside an iframe
  2. find a way to navigate away from a dashboard (e.g. drilldown or edit visualization),
  3. the initial embedding options are lost.

It would make sense to tackle these issues in batch and since not only dashboard app supports embedding and there is a way to navigate from a dashboard to other apps while you in embed mode the effort should probably be split between @elastic/kibana-presentation and @elastic/kibana-app-services where are some of services/utilities/documentation on embedding would become a part of share plugin.

Some common themes that we will need to work through:

  1. When you land into a dashboard with specific view topbar options, make sure those options are persistent when navigating between apps: If you navigate to create maps from iFrame - top nav becomes invisible and you get locked in maps app #89208 [Drilldowns] Dashboard to dashboard drilldown is not maintaining iframe options #77243
  2. How to handle navigation to apps that don't explicitly support embedding (e.g. Maps If you navigate to create maps from iFrame - top nav becomes invisible and you get locked in maps app #89208, visualize/edit Empty dashboard iframe should not give the access for editing to user #56252).
    2.1. One option could be to block all navigation outside of the dashboard when embedded
  3. Multiple issues around filters / uiActions when embed 😮 could be related to URL based state management Embedded iframe dashboard filtering does not work #91590
  4. Problems with short URL and embedding Short url: Embed flag is not respected #72830, Embed Snapshot with Short URL, ?embed=true not working to remove header and nav drawer #50107 (should a solution be as part of URL services?
@Dosant Dosant added Feature:Dashboard Dashboard related features Meta Feature:Embedding Embedding content via iFrame Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas Team:AppServices labels Mar 2, 2021
@Dosant
Copy link
Contributor Author

Dosant commented Mar 2, 2021

This issue is created as part of @elastic/kibana-app-services fix-it-week mostly to group embedding related bugs together.
It doesn't look like these are high-impact issues that require immediate attention, but it would be nice to have this meta for future discussions and for linking more consideration in case something else comes up until we prioritize these embedding issues.

@voidel
Copy link

voidel commented Apr 9, 2021

I disagree with the sentiment that these are not high-impact issues. Slightly confused how these have slipped under the radar on release as these have rendered embedded dashboards wholly unusable.

@ThomThomson
Copy link
Contributor

@voidel, some of these issues are indeed marked high-impact, and have been fixed for 7.12.1 (#94471, #94467, #94465). With these fixed, exported dashboards should be usable again.

The remainder of the issues linked here are relatively low priority, fortunately, they are on our radar to be fixed.

Which issues are affecting your use case the most?

@voidel
Copy link

voidel commented Apr 9, 2021

Hey @ThomThomson, #94465 and #94467 are the real killers at the moment as they have manifested themselves in our testing environment, rendering our application broken there and leaving us unable to deploy into production until they are fixed. Glad to hear they are fixed and an update is coming, however.

Perhaps we were too eager with updating!

@AndePa01
Copy link

Hello, I hope it is okay to bring up an issue that is affecting us that I just created that is also embedded iFrame related.

#100300

Thank you!

@massifazio
Copy link

I have opened this one which seems to be related: https://github.com/elastic/sdh-kibana/issues/1615

@exalate-issue-sync exalate-issue-sync bot added impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. loe:small Small Level of Effort labels Jun 21, 2021
@ppisljar
Copy link
Member

ppisljar commented Aug 9, 2022

Thank you for contributing to this issue, however, we are closing this issue due to inactivity as part of a backlog grooming effort. If you believe this feature/bug should still be considered, please reopen with a comment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Dashboard Dashboard related features Feature:Embedding Embedding content via iFrame impact:low Addressing this issue will have a low level of impact on the quality/strength of our product. loe:small Small Level of Effort Meta Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas
Projects
None yet
Development

No branches or pull requests

6 participants