Editorial: Updating header and footer - Contexts in which this element can be used. by carolgilabert · Pull Request #1640 · w3c/html · GitHub | Latest TMZ Celebrity News & Gossip | Watch TMZ Live
Skip to content
This repository was archived by the owner on Jul 30, 2019. It is now read-only.

Editorial: Updating header and footer - Contexts in which this element can be used. #1640

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Editorial: Updating header and footer - Contexts in which this element can be used. #1640

wants to merge 2 commits into from

Conversation

carolgilabert
Copy link
Contributor

Hopefully fixing #1596 🤞

Copy link
Collaborator

@chaals chaals left a comment

Choose a reason for hiding this comment

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

Trying to find a clearer phrasing (which is difficult because english doesn't handle multiple nesting very well :(

@scottaohara, @patricia-gallardo @LJWatson any thoughts on how to make this easier to understand?

Your mileage may vary.

@@ -1029,7 +1029,9 @@
<dd><a>Flow content</a>.</dd>
<dd><a>Palpable content</a>.</dd>
<dt><a>Contexts in which this element can be used</a>:</dt>
<dd>Where <a>flow content</a> is expected.</dd>
<dd>
Where <a>flow content</a> is expected, but with no address, dt, or th element ancestors, or header, footer elements that are not ancestors of sectioning content which is a ancestors of the header.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think it might be clearer to phrase this as follows:

where flow content is expected, but not within address, dt or th elements. Inside header/footer elements, only within a new sectioning element inside those elements.

<dd>Where <a>flow content</a> is expected.</dd>
<dd>
Where <a>flow content</a> is expected, but with no address, dt, or th element ancestors, or header, footer elements that are not ancestors of sectioning content which is a ancestors of the header.
</dd>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Ditto...

@carolgilabert
Copy link
Contributor Author

@chaals

I've updated the descriptions, but not sure whether you meant for it to be header/footer (so you know it's the same for both), or for me to use header or footer for each description 🙃

Let me know and I can edit more if necessary 😄

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants

TMZ Celebrity News – Breaking Stories, Videos & Gossip

Looking for the latest TMZ celebrity news? You've come to the right place. From shocking Hollywood scandals to exclusive videos, TMZ delivers it all in real time.

Whether it’s a red carpet slip-up, a viral paparazzi moment, or a legal drama involving your favorite stars, TMZ news is always first to break the story. Stay in the loop with daily updates, insider tips, and jaw-dropping photos.

🎥 Watch TMZ Live

TMZ Live brings you daily celebrity news and interviews straight from the TMZ newsroom. Don’t miss a beat—watch now and see what’s trending in Hollywood.