Skip to content

Conversation

@frivoal
Copy link
Collaborator

@frivoal frivoal commented Oct 12, 2017

Closes #1823

<a>block container boxes</a> that contains only <a>block-level</a> boxes.
->
<a>block container boxes</a> that contain only <a>block-level</a> boxes.
@fantasai
Copy link
Collaborator

fantasai commented Nov 6, 2017

This is not confusing enough that it needs to be stated twice. Just fixing the Applies to line seems enough. If you're really insistent to duplicate the point, then make the paragraph after the propdef box a note at the end of the section so it's not before the actual description of what the property is for (which is way more important imho than pointing out where it doesn't apply).

Also, stop putting things that don't need WG discussion on the Agenda just because you want my attention. You have my phone number, if you're that desperate you can just call me!!! >:[

Copy link
Collaborator

@fantasai fantasai left a comment

Choose a reason for hiding this comment

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

See comment above.

@frivoal
Copy link
Collaborator Author

frivoal commented Nov 6, 2017

This is not confusing enough that it needs to be stated twice. Just fixing the Applies to line seems enough.

Done.

Also, stop putting things that don't need WG discussion on the Agenda just because you want my attention. You have my phone number, if you're that desperate you can just call me!!!

Sorry, that wasn't the intent. I don't want your attention specifically, I just want feedback in general, and you're not the only editor of that spec. I do appreciate when I get your feedback, but I am not trying to single you out. Since you edit all the specs, I could call you anytime I want feedback on something, but that doesn't scale very well. That said, regardless of intent, if this is irking you, I'm doing it wrong. So point taken, I'll try to be more conservative when adding things to the agenda.

@frivoal frivoal merged commit 8acefce into w3c:master Nov 6, 2017
@frivoal
Copy link
Collaborator Author

frivoal commented Nov 6, 2017

Oops. I intended to apply the fix the merge, but clicked merge before pushing the fix. I've applied it on the master branch, the the final result is OK, but sorry for messing that up.

frivoal added a commit to frivoal/wpt that referenced this pull request Feb 8, 2018
@frivoal frivoal deleted the widows-orphans-clarification branch February 16, 2018 01:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

css-break-3 Current Work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants