Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docs: LibWebFromLoadingToPainting.md fix grammar and plural #2957

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

SergeCroise
Copy link

@SergeCroise SergeCroise commented Dec 18, 2024

  • a mechanism called formatting contexts
  • for all boxes that were ...

@@ -104,7 +104,7 @@ A number of tree fix-ups are also performed:
Layout starts at the ICB (initial containing block), which is the layout node that corresponds to the DOM document node.
CSS says that the ICB should be the size of the viewport, so the very first thing layout does is assign these dimensions.

Layout works through a mechanism call formatting contexts. CSS defines a number of different formatting contexts:
Layout works through a mechanism that calls formatting contexts. CSS defines a number of different formatting contexts:
Copy link
Contributor

@trflynn89 trflynn89 Dec 18, 2024

Choose a reason for hiding this comment

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

I think "called" is what was meant here.

@SergeCroise
Copy link
Author

Hello @trflynn89 ,
please review!

@InvalidUsernameException
Copy link
Contributor

The second commit fixes something that was incorrect in the first one, so they should probably be squashed into a single commit.

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.

3 participants