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

Commit 6508ba2

Browse files
authored
Merge pull request #388 from ckeditor/t/ckedtor5/920b
Fix: The action should be executed upon the first click on a tooltip-enabled UI in iOS. Closes ckeditor/ckeditor5#920.
2 parents 714ef21 + fe9fb6d commit 6508ba2

File tree

1 file changed

+20
-0
lines changed

1 file changed

+20
-0
lines changed

theme/components/tooltip/mixins/_tooltip.css

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,26 @@
1010
@define-mixin ck-tooltip_enabled {
1111
& .ck-tooltip {
1212
display: block;
13+
14+
/*
15+
* Don't display tooltips in devices which don't support :hover.
16+
* In fact, it's all about iOS, which forces user to click UI elements twice to execute
17+
* the primary action, when tooltips are enabled.
18+
*
19+
* Q: OK, but why not the following query?
20+
*
21+
* @media (hover) {
22+
* display: block;
23+
* }
24+
*
25+
* A: Because FF does not support it and it would completely disable tooltips
26+
* in that browser.
27+
*
28+
* More in https://github.com/ckeditor/ckeditor5/issues/920.
29+
*/
30+
@media (hover:none) {
31+
display: none;
32+
}
1333
}
1434
}
1535

0 commit comments

Comments
 (0)