1481125 - Tracking -moz-scrollbars-none when creating webcompat issues.

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, enhancement, P3)

Description

[Karl Dubost💡 :karlcow]

On some websites, we noticed the usages of 

```
html {
	overflow: -moz-scrollbars-none;
}
```

This usually makes the viewport unscrollable.
Some cases (some of these might have changed since because the site changed or fixed it).


* WashingtonPost
  https://bugzilla.mozilla.org/show_bug.cgi?id=1480037
  https://webcompat.com/issues/17965


* www.hawkeyeinnovations.com 
  https://webcompat.com/issues/17525

  body {
    -ms-overflow-style: none;
    overflow: -moz-scrollbars-none;
  }


* www.abc.es
  https://webcompat.com/issues/13645

  #main_navigation {
	overflow-y: auto;
	-webkit-overflow-scrolling: touch;
	overflow-scrolling: touch;
	overflow: -moz-scrollbars-none;
	-ms-overflow-style: none;
  }



* www.shopclues.com
  https://webcompat.com/issues/7804

  .msc_wrapper {
	overflow: -moz-scrollbars-none;
  }


* www.justdial.com
  https://webcompat.com/issues/7650

.custom-indicator {
    overflow: auto;
    overflow: -moz-scrollbars-none;
    -webkit-overflow-scrolling: touch;
}
  

Related to 

# Impossible to scroll

https://webcompat.com/issues/7812
   overflow: -moz-scrollbars-vertical;

https://webcompat.com/issues/6970
https://webcompat.com/issues/3967
https://webcompat.com/issues/3855
   overflow: -moz-scrollbars-vertical;
   overflow-y: scroll;


https://github.com/morteza/bootstrap-rtl/issues/97#issuecomment-168671023
html {
overflow-y: scroll;
overflow-x: hidden;
overflow: -moz-scrollbars-vertical;
}

Comment 1

[Xidorn Quan [:xidorn] UTC+10]

Would it be better if we simply drop this value? "overflow: -moz-scrollbars-none;" currently simply means "overflow: hidden;"

Comment 2

[Xidorn Quan [:xidorn] UTC+10]

Related code: https://searchfox.org/mozilla-central/rev/51268dcbdff0f6f4a5cff7986df0f616efc5bcfd/servo/components/style/properties/shorthands/box.mako.rs#22-48

We can probably put them behind a pref and see what will happen.

Comment 3

[Xidorn Quan [:xidorn] UTC+10]

Attachment: Bug 1481125 - Put overflow: -moz-scrollbar-* behind pref. r?emilio

Comment 4

[Phabricator Automation]

Comment on attachment 8998153 [details]
Bug 1481125 - Put overflow: -moz-scrollbar-* behind pref. r?emilio

Emilio Cobos Álvarez (:emilio) has approved the revision.

https://phabricator.services.mozilla.com/D2845

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1c0a19134ba6
Put overflow: -moz-scrollbar-* behind pref. r=emilio

Comment 6

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/1c0a19134ba6

Comment 7

[Kohei Yoshino [:kohei]]

Posted the site compatibility note: https://www.fxsitecompat.com/en-CA/docs/2018/several-non-standard-overflow-property-values-have-been-dropped/