Skip to content

LibWeb: Implement window.length#15103

Merged
linusg merged 2 commits intoSerenityOS:masterfrom
networkException:window-length
Sep 9, 2022
Merged

LibWeb: Implement window.length#15103
linusg merged 2 commits intoSerenityOS:masterfrom
networkException:window-length

Conversation

@networkException
Copy link
Copy Markdown
Member

After seeing people have fun with the web on twitter I thought we could also partake in the fun and gain a bit of spec compliance in the process :^)

@networkException networkException force-pushed the window-length branch 2 times, most recently from 3a59464 to 123fcb9 Compare September 6, 2022 17:58
@networkException
Copy link
Copy Markdown
Member Author

I've changed the call to for_each_child_of_type<BrowsingContext> to for_each_child as there were issues with the lagom build. See https://discord.com/channels/830522505605283862/830739873119207426/1016836457027092490

This patch makes use of helpers implemented for window.length to resolve
two FIXMEs in WindowProxy previously simply assuming no child browsing
contexts :^)
Copy link
Copy Markdown
Member

@linusg linusg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, I hate it ;^)

@linusg linusg merged commit 75ac47d into SerenityOS:master Sep 9, 2022
@networkException networkException changed the title LibWeb: Implement window.length LibWeb: Implement window.length Sep 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants