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

[Visualize] Badges on OSS inactive cards appear with the external link icon #85559

Closed
stratoula opened this issue Dec 10, 2020 · 4 comments · Fixed by #85580
Closed

[Visualize] Badges on OSS inactive cards appear with the external link icon #85559

stratoula opened this issue Dec 10, 2020 · 4 comments · Fixed by #85580
Labels
Feature:Vis Editor Visualization editor issues Team:Platform-Design Team Label for Kibana Design Team. Support the Analyze group of plugins. Team:Visualizations Visualization editors, elastic-charts and infrastructure

Comments

@stratoula
Copy link
Contributor

Kibana version:
master

Describe the bug:
It's is not a bug but is a look and feel issue. When the new visualization wizard has merged the badges of the inactive cards on OSS were redirecting to external links but the icon was not depicted.

But I think that there was a change on EUI that every link that has target=_blank should default to true. This change has caused the badges to appear with an icon.

Before:
Screenshot 2020-12-10 at 2 54 48 PM

Now:
Screenshot 2020-12-10 at 2 52 58 PM

@stratoula stratoula added Team:Visualizations Visualization editors, elastic-charts and infrastructure Feature:Vis Editor Visualization editor issues labels Dec 10, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app (Team:KibanaApp)

@stratoula
Copy link
Contributor Author

stratoula commented Dec 10, 2020

@elastic/kibana-design My dear design team I need your opinion on that. I can see two paths:

  • add the external=false but I don't think that this is correct here as they redirect to an external link actually
  • hide this icon with css
    .visNewVisDialog__groupsCardLink .euiLink__externalIcon { display: none; }

Or maybe another idea 🤔 Wdyt?

@stratoula stratoula added the Team:Platform-Design Team Label for Kibana Design Team. Support the Analyze group of plugins. label Dec 10, 2020
@elizabetdev
Copy link
Contributor

Hi @stratoula,
IMO we should just add external=false to the EuiLink. This prop only removes the icon but the link continues to have the target="_blank".

There's actually an issue with the way the EuiBadge is handling the links and we need to solve it in EUI side: elastic/eui#4190.

So I'll try to prioritize it. 🙌

@stratoula
Copy link
Contributor Author

Thank you dear! Will create a PR right now!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Vis Editor Visualization editor issues Team:Platform-Design Team Label for Kibana Design Team. Support the Analyze group of plugins. Team:Visualizations Visualization editors, elastic-charts and infrastructure
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants