Skip to content

Commit 256fa4d

Browse files
committed
[css-scroll-snap] Fix definition so it fits with the idea that scroll-snap-stop applies to element's snap position, not to all snap positions in a scroll container.
1 parent 948f4e4 commit 256fa4d

File tree

3 files changed

+19
-5
lines changed

3 files changed

+19
-5
lines changed

css-scroll-snap/Overview.bs

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ Introduction {#intro}
5353
Module interactions {#placement}
5454
--------------------------------
5555

56-
This module extends the scrolling user interface features defined in [[!CSS21]] section 11.1.
56+
This module extends the scrolling user interface features defined in [[!CSS2]] section 11.1.
5757

5858
None of the properties in this module apply to the ''::first-line'' and ''::first-letter'' pseudo-elements.
5959

@@ -745,8 +745,8 @@ Scroll Snap Limits: the 'scroll-snap-stop' property {#scroll-snap-stop}
745745
</pre>
746746

747747
This property specifies whether the <a>scroll container</a>
748-
must stop at each <a>snap position</a> it passes,
749-
or may pass multiple <a>snap positions</a> before coming to rest.
748+
must stop at a <a>snap position</a> definited by this element,
749+
or may pass through it before coming to rest.
750750
Values are defined as follows:
751751

752752
<dl dfn-type=value dfn-for=scroll-snap-stop>
@@ -1090,6 +1090,8 @@ Changes {#changes}
10901090
<ins><p class="note">This specification only applies to scrolling methods supported by the user agent;
10911091
it does not require the user agent to support any particular input or scrolling method.</p></ins>
10921092
</blockquote>
1093+
<li>Clarified that 'scroll-snap-stop' is applied to the <a>snap positions</a> defined by the element,
1094+
not applied to all <a>snap positions</a> in the <a>scroll snap container</a>.
10931095
<li>Fixed some syntax errors in examples and added a new one to the 'scroll-snap-type' section.
10941096
(<a href="https://github.com/w3c/csswg-drafts/issues/827">Issue 827</a>)
10951097
</ul>

css-scroll-snap/issues-cr-2016.bsi

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ Response: https://github.com/w3c/csswg-drafts/issues/1050#issuecomment-309578167
3131
Changes: https://github.com/w3c/csswg-drafts/commit/38caa2ee20ca9833a1ad401b03c96debcac0cc21
3232
Comment: https://github.com/w3c/csswg-drafts/issues/1050#issuecomment-309591432
3333
Open: Accepted
34-
Verified: =plinss= Verify.
34+
Verified: https://github.com/w3c/csswg-drafts/issues/1050#issuecomment-311517366
3535
----
3636
Issue 4.
3737
Summary: Should negative scroll-padding be allowed
@@ -52,3 +52,9 @@ Changes: https://hg.csswg.org/drafts/diff/ae445d9c2266/css-scroll-snap/Overview
5252
Closed: Accepted
5353
Resolved: Editorial
5454
----
55+
Issue 6.
56+
Summary: scroll-snap-stop definition not quite right
57+
From: Majid Valipour
58+
Comment: https://github.com/w3c/csswg-drafts/issues/1552
59+
Open
60+
----

css-scroll-snap/issues-cr-2016.html

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ <h1>CSS Scroll Snapping Level 1 Disposition of Comments for 2016-10-20 CR</h1>
6666
<span>Changes: <a href='https://github.com/w3c/csswg-drafts/commit/38caa2ee20ca9833a1ad401b03c96debcac0cc21'>https://github.com/w3c/csswg-drafts/commit/38caa2ee20ca9833a1ad401b03c96debcac0cc21</a></span>
6767
<span>Comment: <a href='https://github.com/w3c/csswg-drafts/issues/1050#issuecomment-309591432'>https://github.com/w3c/csswg-drafts/issues/1050#issuecomment-309591432</a></span>
6868
<span class="a">Open: Accepted</span>
69-
<span class="a">Verified: =plinss= Verify.</span></pre>
69+
<span class="a">Verified: <a href='https://github.com/w3c/csswg-drafts/issues/1050#issuecomment-311517366'>https://github.com/w3c/csswg-drafts/issues/1050#issuecomment-311517366</a></span></pre>
7070
<pre class=' a' id='issue-4'>
7171
<span>Issue 4. <a href='#issue-4'>#</a></span>
7272
<span>Summary: Should negative scroll-padding be allowed</span>
@@ -86,6 +86,12 @@ <h1>CSS Scroll Snapping Level 1 Disposition of Comments for 2016-10-20 CR</h1>
8686
<span>Changes: <a href='https://hg.csswg.org/drafts/diff/ae445d9c2266/css-scroll-snap/Overview.bs'>https://hg.csswg.org/drafts/diff/ae445d9c2266/css-scroll-snap/Overview.bs</a></span>
8787
<span class="a">Closed: Accepted</span>
8888
<span>Resolved: Editorial</span></pre>
89+
<pre class='open ' id='issue-6'>
90+
<span>Issue 6. <a href='#issue-6'>#</a></span>
91+
<span>Summary: scroll-snap-stop definition not quite right</span>
92+
<span>From: Majid Valipour</span>
93+
<span>Comment: <a href='https://github.com/w3c/csswg-drafts/issues/1552'>https://github.com/w3c/csswg-drafts/issues/1552</a></span>
94+
<span>Open</span></pre>
8995
<script>
9096
(function () {
9197
var sheet = document.styleSheets[0];

0 commit comments

Comments
 (0)