Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(26)

Unified Diff: third_party/WebKit/LayoutTests/fast/css/sticky/sticky-clip-rel-child-expected.html

Issue 2020103002: Fix sticky constraints and update sticky layer positions recursively after scroll. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix bugs with overflow scrollers in sticky position constraints, add unit tests, and test clipped b… Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/fast/css/sticky/sticky-clip-rel-child-expected.html
diff --git a/third_party/WebKit/LayoutTests/fast/css/sticky/sticky-clip-rel-child-expected.html b/third_party/WebKit/LayoutTests/fast/css/sticky/sticky-clip-rel-child-expected.html
new file mode 100644
index 0000000000000000000000000000000000000000..2299799a653078dc0c7be88446057788aa4dce8a
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/fast/css/sticky/sticky-clip-rel-child-expected.html
@@ -0,0 +1,51 @@
+<!DOCTYPE html>
+<style>
+body {
+ height: 2000px;
+ margin: 0;
+}
+.sticky {
+ background: red;
+ height: 100px;
+ width: 100px;
+ overflow: hidden;
+ position: relative;
+ top: 100px;
+}
+.relative {
+ background: green;
+ height: 100px;
+ width: 100px;
+ position: relative;
+}
+#scroller {
+ left: 200px;
+ top: 100px;
+ height: 100px;
+ width: 120px;
+ overflow: auto;
+ position: absolute;
+}
+.spacer {
+ height: 1000px;
+}
+</style>
+<script>
+function doTest() {
+ window.scrollTo(0, 100);
+ document.getElementById('scroller').scrollTop = 100;
+}
+window.addEventListener('load', doTest, false);
+</script>
+<div class="sticky">
+ <div class="relative">
+ </div>
+</div>
+<div id="scroller">
+ <div class="sticky">
+ <div class="relative">
chrishtr 2016/06/06 22:18:39 Was the bug that it wasn't including the bounds of
flackr 2016/06/09 17:13:32 Because the relative positioned element is clipped
+ </div>
+ </div>
+ <div class="spacer"></div>
+</div>
+

Powered by Google App Engine
This is Rietveld 408576698