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

Fixes #22138 - fix tooltip triggering #5148

Closed
wants to merge 1 commit into from

Conversation

amirfefer
Copy link
Member

Tooltip is not removed after click and requires antoher click in order to remove it.
This may be annoying especially on dropdown / buttons etc...
hosts - edited

@theforeman-bot
Copy link
Member

There were the following issues with the commit message:

  • 0841bad must be in the format fixes #redmine_number - brief description

If you don't have a ticket number, please create an issue in Redmine.

More guidelines are available in Coding Standards or on the Foreman wiki.


This message was auto-generated by Foreman's prprocessor

@amirfefer amirfefer changed the title Fixes 22138 - fix tooltip triggering Fixes #22138 - fix tooltip triggering Jan 3, 2018
@ohadlevy
Copy link
Member

ohadlevy commented Jan 3, 2018

@tbrisker mind having a look?

@@ -52,7 +52,7 @@ export function activateTooltips(elParam = 'body') {
el
.find('*[title]')
.not('*[rel]')
.tooltip({ container: 'body' });
.tooltip({ container: 'body', trigger: 'hover' });
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why only on this and not on other tooltip activation methods? might make sense to clean up a bit an unify them

@theforeman-bot
Copy link
Member

Thank you for your contribution, @amirfefer! This PR has been inactive for 6 months, closing for now.
Feel free to reopen when you return to it. This is an automated process.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants