Index: Source/core/css/fullscreen.css |
diff --git a/Source/core/css/fullscreen.css b/Source/core/css/fullscreen.css |
index bb9c79e0eae427ab0fa76dd0fc2fa00796f0c063..a53ede6057f4c7b74e48eaf5adde36ede88af2ab 100644 |
--- a/Source/core/css/fullscreen.css |
+++ b/Source/core/css/fullscreen.css |
@@ -1,10 +1,52 @@ |
+/* |
+ * User-agent level style sheet defaults for the Fullscreen API |
+ * https://fullscreen.spec.whatwg.org/#user-agent-level-style-sheet-defaults |
+ * |
+ * Note that the prefixed :-webkit-full-screen pseudo-class is used instead of |
+ * :fullscreen. crbug.com/402378 |
Julien - ping for review
2015/03/04 01:07:38
That's not a really actionable (btw the rest of th
philipj_slow
2015/03/04 07:15:28
I'll convert everything to FIXMEs instead.
|
+ */ |
+ |
+:not(:root):-webkit-full-screen { |
+ position: fixed !important; |
+ top: 0 !important; |
+ right: 0 !important; |
+ bottom: 0 !important; |
+ left: 0 !important; |
+ margin: 0 !important; |
+ box-sizing: border-box !important; |
+ min-width: 0 !important; |
+ max-width: none !important; |
+ min-height: 0 !important; |
+ max-height: none !important; |
+ width: 100% !important; |
+ height: 100% !important; |
+ object-fit: contain !important; |
+} |
+ |
+iframe:-webkit-full-screen { |
+ border: none; |
+} |
+ |
+/* |
+ * The flex and transform rules will likely be added to the spec: |
+ * https://www.w3.org/Bugs/Public/show_bug.cgi?id=27931 |
+ * Bug history in WebKit and Blink: |
+ * https://bugs.webkit.org/show_bug.cgi?id=58291 |
+ * https://bugs.webkit.org/show_bug.cgi?id=60140 |
+ * https://bugs.webkit.org/show_bug.cgi?id=62463 |
+ * https://crbug.com/356282 |
Julien - ping for review
2015/03/04 01:07:38
The (apparently) semi-random selection of bugs you
philipj_slow
2015/03/04 07:15:28
These were the bugs references when adding/tweakin
|
+ */ |
:-webkit-full-screen { |
- background-color: white; |
Julien - ping for review
2015/03/04 01:07:38
I don't agree with this change nor the justificati
philipj_slow
2015/03/04 07:15:28
OK, I'll revert the background changes.
|
- z-index: 2147483647 !important; |
+ flex: none !important; |
Julien - ping for review
2015/03/04 01:07:38
I would have liked a focused explanation on why it
philipj_slow
2015/03/04 07:15:28
Sorry, I should have declared my ignorance on this
Julien - ping for review
2015/03/04 17:50:22
It was probably not needed before actually but we
philipj_slow
2015/03/05 04:05:06
I've played around a bit with flexbox and the Full
|
+ transform: none !important; |
} |
Julien - ping for review
2015/03/04 01:07:38
I would love to keep the separation between spec a
philipj_slow
2015/03/04 07:15:28
Done.
|
-:root:-webkit-full-screen-ancestor { |
- overflow: hidden !important; |
+/* |
+ * Implementing Fullscreen using top layer would remove the need for |
+ * z-index and :-webkit-full-screen-ancestor rules. crbug.com/240576 |
+ */ |
+:-webkit-full-screen { |
+ z-index: 2147483647 !important; |
} |
:-webkit-full-screen-ancestor:not(iframe) { |
@@ -21,40 +63,9 @@ |
-webkit-transform-style: flat !important; |
} |
-video:-webkit-full-screen, audio:-webkit-full-screen { |
- background-color: transparent !important; |
- position: relative !important; |
- left: 0 !important; |
- top: 0 ! important; |
- margin: 0 !important; |
- min-width: 0 !important; |
- max-width: none !important; |
- min-height: 0 !important; |
- max-height: none !important; |
- width: 100% !important; |
- height: 100% !important; |
- flex: 1 !important; |
- display: block !important; |
- transform: none !important; |
-} |
- |
-img:-webkit-full-screen { |
- width: auto; |
- height: 100%; |
- max-width: 100%; |
-} |
- |
-iframe:-webkit-full-screen { |
- margin: 0 !important; |
- padding: 0 !important; |
- border: 0 !important; |
- position: fixed !important; |
- min-width: 0 !important; |
- max-width: none !important; |
- min-height: 0 !important; |
- max-height: none !important; |
- width: 100% !important; |
- height: 100% !important; |
- left: 0 !important; |
- top: 0 !important; |
+/* |
+ * Prevents video from overflowing the viewport on Android. crbug.com/441890 |
+ */ |
+:-webkit-full-screen-ancestor { |
+ overflow: hidden !important; |
} |