Skip to content
This repository has been 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.

None yet

2 participants