third_party/WebKit/LayoutTests/fast/css/sticky/sticky-top-auto-get-computedstyle.html - Issue 2870983002: Update return value of getComputedStyle for sticky elements

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: third_party/WebKit/LayoutTests/fast/css/sticky/sticky-top-auto-get-computedstyle.html

Issue 2870983002: Update return value of getComputedStyle for sticky elements (Closed)

Patch Set: Created 3 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:

OLD	NEW
(Empty)
	1 <!DOCTYPE html>

	2 <style>

	3 .box {

	4 position: sticky;

	5 }

	6

	7 .without-top {

	8 top: auto;

	9 }

	10

	11 .with-top {

	12 top: 2px;
	flackr 2017/05/09 20:26:34 nit: use top: 0px to showcase the original issue t nit: use top: 0px to showcase the original issue the bug was about. yigu 2017/05/09 20:43:58 Done. Show quoted text On 2017/05/09 20:26:34, flackr wrote: > nit: use top: 0px to showcase the original issue the bug was about. Done.
	13 }

	14

	15 .with-bottom {

	16 bottom: 2px;

	17 }

	18

	19 </style>

	20

	21 <body>

	22 <div class="box without-top">Sticky, no top</div>

	23 <div class="box with-top">Sticky, with top</div>

	24 <div class="box with-bottom">Sticky, with bottom</div>

	25 </body>

	26

	27 <!-- top should be auto when not specified for sticky elements. Same for the

	28 other attributes.-->

	29 <script>

	30 if (window.testRunner)

	31 testRunner.dumpAsText();

	32

	33 function doTest() {

	34 var boxes = document.getElementsByClassName('box');

	35 for (var i = 0; i < boxes.length; ++i) {

	36 boxes[i].textContent += ': top = ' + getComputedStyle(boxes[i]).top;

	37 boxes[i].textContent += '; bottom = ' + getComputedStyle(boxes[i]).bottom;

	38 boxes[i].textContent += '; left = ' + getComputedStyle(boxes[i]).left;

	39 boxes[i].textContent += '; right = ' + getComputedStyle(boxes[i]).right;

	40 }

	41 }

	42

	43 window.addEventListener('load', doTest, false);

	44 </script>

OLD	NEW