Skip to content

[css-fonts-4] typo: miscounted number of generic font families #2148

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
mlc opened this issue Dec 29, 2017 · 4 comments
Closed

[css-fonts-4] typo: miscounted number of generic font families #2148

mlc opened this issue Dec 29, 2017 · 4 comments
Assignees
@litherum
Labels
Closed Accepted as Obvious Bugfix Commenter Satisfied Commenter has indicated satisfaction with the resolution / edits. css-fonts-4 Current Work

Comments

@mlc
Copy link

mlc commented Dec 29, 2017

css-fonts-4 section 2.1.1 begins by describing behavior for "all five generic font families", but then goes on to list nine such families. I would suggest removing the word "five" (twice in the same sentence), replacing it with "nine" (ditto), or explaining how five of the families behave differently from the other four.

Thanks!
@litherum
Copy link
Contributor

We should just remove the redundant numbers altogether.

@litherum litherum self-assigned this Dec 30, 2017
litherum pushed a commit that referenced this issue Mar 6, 2018
[css-fonts-4] typo: miscounted number of generic font families
64a6f62 
#2148
@litherum
Copy link
Contributor

litherum commented Mar 6, 2018

64a6f62

@litherum litherum closed this as completed Mar 6, 2018
litherum pushed a commit that referenced this issue Mar 6, 2018
[css-fonts-4] typo: miscounted number of generic font families
d219169 
#2148
@litherum
Copy link
Contributor

litherum commented Mar 6, 2018

d219169

@mlc
Copy link
Author

mlc commented Mar 6, 2018

thanks for having a look!

@svgeesus svgeesus added Closed Accepted as Obvious Bugfix Commenter Satisfied Commenter has indicated satisfaction with the resolution / edits. and removed Needs Edits labels Mar 6, 2018
@svgeesus svgeesus mentioned this issue Apr 23, 2020
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@litherum litherum

Labels
Closed Accepted as Obvious Bugfix Commenter Satisfied Commenter has indicated satisfaction with the resolution / edits. css-fonts-4 Current Work
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mlc @litherum @svgeesus