LadybirdBrowser / ladybird

Truly independent web browser
https://ladybird.org
BSD 2-Clause "Simplified" License
22.36k stars 994 forks source link

LibWeb: Remove non-spec condition in scrollable overflow calculation #2525

Open Psychpsyo opened 2 days ago

Psychpsyo commented 2 days ago

Nothing in the spec says that this part should unconditionally run for the viewport and this is breaking on https://battle.crossuniverse.net/intro, making the page scrollable when it shouldn't be.

I wasn't sure what to call the test for this, so if anyone has better ideas, I'd love to rename it. Also, yes, it needs to append the child with javascript, if the child is in the page initially, the bug does not occur.