-
Notifications
You must be signed in to change notification settings - Fork 289
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
Bastion: replacing tipsy with bootstrap 3 tooltips. #3416
Conversation
@@ -63,14 +63,14 @@ <h2 class="fl" translate>System {{ system.name }}</h2> | |||
</nav> | |||
|
|||
<div class="flash_hud" ng-show="saveSuccess"> | |||
<a class="control clickable" title="Close" ng-click="saveSuccess = false"></a> | |||
<a class="control clickable" tooltip="Close" ng-click="saveSuccess = false"></a> |
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.
Update to translated please.
@ehelms updated. |
@@ -5,7 +5,7 @@ | |||
<a class="menu-item-link organization-name clickable" | |||
ng-click="toggleVisibility()" | |||
ng-class="{'active-item': visible}" | |||
title="{{ currentOrganization.name }}"> | |||
tooltip="{{ currentOrganization.name }}"> |
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 is a bit of a strange tooltip. I realize the title was there before, but unless we are cutting off the text via ellipsis (which I think we are avoiding) this looks off to me.
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.
Shall I just remove this one?
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.
+1
@ehelms updated again. |
What do you think about trying to do a helptip icon with tooltip instead of showing a greyed out remove button? |
@ehelms updated again. |
ACK |
Bastion: replacing tipsy with bootstrap 3 tooltips.
No description provided.