1551577 - [Inactive CSS] Display a warning when scroll-padding-* properties are used on a non-scrollable element

Status: RESOLVED FIXED

(DevTools :: Inspector: Rules, enhancement, P3)

Description

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

Only valid on non-scrollable elements:

[
  "scroll-snap-coordinate",
  "scroll-snap-destination",
  "scroll-snap-points-x",
  "scroll-snap-points-y",
  "scroll-snap-type",
  "scroll-snap-type-x",
  "scroll-snap-type-y",
]

[This property] doesn't have an effect because the element doesn't scroll. Try adding a property like overflow:scroll.

Comment 1

[Hiroyuki Ikezoe (:hiro)]

(In reply to Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe] from comment #0)

Only valid on non-scrollable elements:

[
  "scroll-snap-coordinate",
  "scroll-snap-destination",
  "scroll-snap-points-x",
  "scroll-snap-points-y",
  "scroll-snap-type",
  "scroll-snap-type-x",
  "scroll-snap-type-y",
]

[This property] doesn't have an effect because the element doesn't scroll. Try adding a property like overflow:scroll.

Those properties other than scroll-snap-type will be dropped in bug 1218265 and bug 1218264.

That's said, I think scroll-snap-type is also valid on scrollable element. It can be applied on all elements.

Comment 2

[Mike Ratcliffe [:miker] [:mratcliffe] [:mikeratcliffe]]

You are right:

Applies to: all elements

So I guess we need to drop this one.

@hiro Thanks for letting us know!

Comment 3

[Sebastian Zartner [:sebo]]

According to the latest version of the specification, only scroll-padding-* properties are restricted to scroll containers. Therefore I've changed the summary accordingly.

Sebastian

Comment 4

[Sebastian Zartner [:sebo]]

Attachment: Bug 1551577 - New inactive CSS rule to show warning when scroll-padding-* is used on non-scrollable elements. r=nchevobbe

Comment 5

[Sebastian Zartner [:sebo]]

Nicolas, could you please do the review?

Sebastian

Comment 6

[Nicolas Chevobbe [:nchevobbe]]

sure, sorry for the delay

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/576f61f7bb0f
New inactive CSS rule to show warning when scroll-padding-* is used on non-scrollable elements. r=nchevobbe,fluent-reviewers,flod

Comment 8

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/576f61f7bb0f

Comment 9

[Chris Mills [:cmills]]

Documentation done; see https://github.com/mdn/content/issues/2505