Skip to content
This repository has been archived by the owner on Jun 26, 2020. It is now read-only.

Commit

Permalink
Merge pull request #388 from ckeditor/t/ckedtor5/920b
Browse files Browse the repository at this point in the history
Fix: The action should be executed upon the first click on a tooltip-enabled UI in iOS. Closes ckeditor/ckeditor5#920.
  • Loading branch information
Reinmar committed May 14, 2018
2 parents 714ef21 + fe9fb6d commit 6508ba2
Showing 1 changed file with 20 additions and 0 deletions.
20 changes: 20 additions & 0 deletions theme/components/tooltip/mixins/_tooltip.css
Expand Up @@ -10,6 +10,26 @@
@define-mixin ck-tooltip_enabled {
& .ck-tooltip {
display: block;

/*
* Don't display tooltips in devices which don't support :hover.
* In fact, it's all about iOS, which forces user to click UI elements twice to execute
* the primary action, when tooltips are enabled.
*
* Q: OK, but why not the following query?
*
* @media (hover) {
* display: block;
* }
*
* A: Because FF does not support it and it would completely disable tooltips
* in that browser.
*
* More in https://github.com/ckeditor/ckeditor5/issues/920.
*/
@media (hover:none) {
display: none;
}
}
}

Expand Down

0 comments on commit 6508ba2

Please sign in to comment.