-
Notifications
You must be signed in to change notification settings - Fork 4.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
UI: [VAULT-17041] Client count card #22323
UI: [VAULT-17041] Client count card #22323
Conversation
Build Results: |
@@ -50,6 +50,7 @@ | |||
@route={{backend.backendLink}} | |||
@model={{backend.id}} | |||
class="has-text-weight-semibold is-no-underline" | |||
target="_blank" |
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.
Just confirming that we indeed want these links to always open in a new tab? My understanding is you only do that if you're linking off your website. Ex: we were linking to documentation.
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'll reconfirm with design and get back to you! The reason why we added this is that the breadcrumbs don't match the behavior of the link. Going to remove the new tab links for now.
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.
Question about the use of target=_blank but everything else looks good.
Description
TODO's in different PR: