]> cat aescling's git repositories - mastodon.git/commitdiff
Fix manual scrolling issue on Firefox/Windows (#12648)
authorThibG <thib@sitedethib.com>
Thu, 19 Dec 2019 11:47:19 +0000 (12:47 +0100)
committerEugen Rochko <eugen@zeonfederated.com>
Thu, 19 Dec 2019 11:47:19 +0000 (12:47 +0100)
Fixes #12607

`will-change: transform` apparently makes manual scrolling impossible on
Firefox/Windows. While this should probably be considered a Firefox bug,
`will-change: transform` seem like a very aggressive performance hint that
may possibly make the browser consume more resources than needed, especially
in multiple-column mode.

This was originally added to improve scrolling performances on mobile, but
I think this isn't necessary anymore, because of the two following reasons:
- `contain: paint` (which is implied by `contain: strict`, which we apply
  whenever the browser supports grids) should have similar effects
- in single-column mode, the scrolling container is the root element, which
  I believe is optimized in at least Chromium

Keep in mind that I have not been able to make in-depth benchmarks, and
especially not been able to try on mobile, so performances should probably
be investigated further…

app/javascript/styles/mastodon/components.scss

index 13f9dfae76795245d47c6e225caaf1a57473cb78..4c7ce9ba7c389966b97b2c9668232d31b3bfea8e 100644 (file)
@@ -2512,7 +2512,6 @@ a.account__display-name {
   overflow-x: hidden;
   flex: 1 1 auto;
   -webkit-overflow-scrolling: touch;
-  will-change: transform; // improves perf in mobile Chrome
 
   &.optionally-scrollable {
     overflow-y: auto;