-
Notifications
You must be signed in to change notification settings - Fork 533
update to prose #1444
update to prose #1444
Conversation
fixes #1443 Adds note about elements described as transparent. Other minor wording updates to the previous prose of this section.
In the new note I think it would be more easily understandable if it provided an example of how this could be used. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add an example on the new note
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
question, request for less jargon-y prose
sections/introduction.include
Outdated
</p> | ||
|
||
<p class="note"> | ||
An exception to disallowing <a>block box</a> within <a>inline box</a> rendered elements |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you call these something clearer? "elements that are typically rendered as a ..."?
(sorry, no better ideas right now)
sections/introduction.include
Outdated
</p> | ||
|
||
<p class="note"> | ||
An exception to disallowing <a>block box</a> within <a>inline box</a> rendered elements |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this the only exception, or are there others (which is what seems to be implied here)?
@patricia-gallardo & @chaals, |
@patricia-gallardo, @chaals any more that needed to be done on this? |
Looks good to me. @patricia-gallardo, if you are happy please merge this. |
fixes #1443
Adds note about elements described as transparent.
Other minor wording updates to the previous prose of this section.