Skip to content

Conversation

estelle
Copy link
Member

@estelle estelle commented Sep 9, 2025

Part of #40528

Part of github.com/openwebdocs/project/issues/147

@estelle estelle requested a review from a team as a code owner September 9, 2025 01:02
@estelle estelle requested review from chrisdavidmills and removed request for a team September 9, 2025 01:02
@github-actions github-actions bot added Content:CSS Cascading Style Sheets docs size/m [PR only] 51-500 LoC changed labels Sep 9, 2025
Copy link
Contributor

github-actions bot commented Sep 9, 2025

seems not a single file was built! 🙀

(comment last updated: 2025-09-09 12:07:10)

Copy link
Contributor

@chrisdavidmills chrisdavidmills left a comment

Choose a reason for hiding this comment

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

@estelle A few bits of feedback for you, but nothing major.

### Glossary terms and definitions

- {{glossary("Viewport")}}
- [Actual viewport](/en-US/docs/Web/CSS/CSSOM_view/Viewport_concepts#what_is_a_viewport)
Copy link
Contributor

Choose a reason for hiding this comment

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

Seems a bit weird to link both of these to the same place, and to put them in the Glossary section when they are not glossary entries. Maybe:

  1. Put them in related concepts.
  2. Edit the linked page to include a DL for the terms, so there will be a separate fragment to link to for each one.

Copy link
Member Author

Choose a reason for hiding this comment

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

it's both "glossary terms" and "definitions" because i think we're trying to get away from single use / spec related glossary entries. I'll create a DL


The _actual viewport_ is the viewport you get after processing the [viewport `<meta>` tag](/en-US/docs/Web/HTML/Reference/Elements/meta/name/viewport). Content designed for large viewports may exhibit a variety of bugs when viewed in smaller viewports, including unintended wrapping, clipped content, wrong-sized {{glossary("scroll containers")}}. HTML provides a [viewport meta tag](/en-US/docs/Web/HTML/Reference/Elements/meta/name/viewport), `<meta name="viewport">`, to provide hints about the initial size of the viewport. The actual viewport is the size defined by its [`content`](/en-US/docs/Web/HTML/Reference/Elements/meta#content) attribute. If this tag is omitted, some mobile browsers use a fixed initial containing block width (typically `980px`), setting the width of the actual viewport to this values, then scales the content down making making the CSS pixel size smaller than an actual pixel.

A viewport represents the area in computer graphics being currently viewed. In web browser terms, it is generally the same as the browser window, excluding the UI, menu bar, etc. That is the part of the document you are viewing. For paged media, the initial containing block is based on the page area. The page area can be set through {{cssxref("@page")}} rules.
Copy link
Contributor

Choose a reason for hiding this comment

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

It feels like this general description of a viewport should be at the top of the section, before the specifics about initial and actual viewport.

Copy link
Member Author

Choose a reason for hiding this comment

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

updated.

@estelle estelle marked this pull request as ready for review September 9, 2025 12:05
Copy link
Contributor

@chrisdavidmills chrisdavidmills left a comment

Choose a reason for hiding this comment

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

W00t!

@chrisdavidmills chrisdavidmills merged commit 9f1ac43 into main Sep 9, 2025
12 checks passed
@chrisdavidmills chrisdavidmills deleted the cssviewport branch September 9, 2025 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:CSS Cascading Style Sheets docs size/m [PR only] 51-500 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants