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

#1004 fixed tooltip not disappearing on mobile #1083

Merged
merged 2 commits into from
Jun 19, 2018

Conversation

nathanbacon
Copy link
Contributor

@nathanbacon nathanbacon commented Jun 19, 2018

I tested this on iOS and the proper behavior is observed. I hope clearing the show timeout is on the right track. (#1004 )

@ntucakovic
Copy link

Just tested on Android's latest Chrome and on Mac OS' Chrome device emulator. Works as expected.

@TheSharpieOne
Copy link
Member

This breaks the "Tooltip Disable Autohide" feature which allows users to select the text within the tooltip. Now, when clicking to select the text, it will close the tooltip (on desktop).

Copy link
Member

@TheSharpieOne TheSharpieOne left a comment

Choose a reason for hiding this comment

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

This will also implement #1079 which was declined in #1080 since it's not what bootstrap itself does... but since this fixes a higher priority / real bug I'd rather have a defect of closing when clicking outside than it not closing for mobile uses.

@TheSharpieOne TheSharpieOne merged commit e6a1313 into reactstrap:master Jun 19, 2018
juanmaguitar added a commit to juanmaguitar/reactstrap that referenced this pull request Jun 25, 2018
…p-innerRef

* 'master' of github.com:reactstrap/reactstrap:
  fix(tooltip): fixed tooltip not disappearing on mobile (reactstrap#1083)
@juanmaguitar juanmaguitar mentioned this pull request Jun 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants