Set Breeze scrollbars only on HTML tag
ClosedPublic

Authored by broulik on Apr 16 2019, 8:12 AM.

Details

Summary

This ensures we don't mess with the website's internal scrollbars.

Test Plan

YouTube can now scroll when in fullscreen causing us to show annoying scrollbars.
By setting the scrollbars only on html tag, a website can override the scrollbars easily and we also don't mess with the site's internal scrollbars, e.g. I have often seen code snippet viewers looking awful.
I still get scrollbars for the main page but not anything inside and YT fullscreen also works fine now

Diff Detail

Repository
R856 Plasma Browser Integration
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik created this revision.Apr 16 2019, 8:12 AM
Restricted Application added a project: Plasma. · View Herald TranscriptApr 16 2019, 8:12 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.Apr 16 2019, 8:12 AM
fvogt accepted this revision.Apr 16 2019, 8:49 AM

No idea about CSS - as this commit is about breaking the style for most cases though it's probably fine.

This revision is now accepted and ready to land.Apr 16 2019, 8:49 AM
This revision was automatically updated to reflect the committed changes.