Skip to content
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.

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

[css-multicol-1] which paragraph says count+width not making sense? #4291

Closed
myakura opened this issue Sep 10, 2019 · 4 comments
Closed

[css-multicol-1] which paragraph says count+width not making sense? #4291

myakura opened this issue Sep 10, 2019 · 4 comments
Labels
css-multicol-1
Projects
css-multicol-1

Comments

@myakura
Copy link
Contributor

@myakura myakura commented Sep 10, 2019

However, as described below, setting both the width and number of columns rarely makes sense.

https://drafts.csswg.org/css-multicol/#example-38b72848

Although I'm pretty bad at reading, I couldn't find below where it's described. Example 4 doesn't seem to say that.
@frivoal
Copy link
Collaborator

@frivoal frivoal commented Sep 11, 2019

I agree, I cannot find the "below". Also, this statements is pretty strange. It's judgemental, but I don't agree with it. I think we should just drop it.

@rachelandrew , what do you think?

@rachelandrew
Copy link
Contributor

@rachelandrew rachelandrew commented Sep 11, 2019

Yep, I had wondered about it too. I think we can just remove that? If so I'll do it.

@myakura
Copy link
Contributor Author

@myakura myakura commented Sep 11, 2019

I just found MDN explains it pretty well (guess who wrote it ;-))
https://developer.mozilla.org/docs/Web/CSS/CSS_Columns/Basic_Concepts_of_Multicol#Using_column-count_and_column-width_together

perhaps the spec say similar. adding a note to tell setting these two might end up fewer columns is helpful for authors.

@frivoal
Copy link
Collaborator

@frivoal frivoal commented Sep 11, 2019

Yeah, we can just delete that, no need to go through the working group for that, it's just a weird little bit of non normative prose.

@rachelandrew rachelandrew added this to Needs action in css-multicol-1 Sep 16, 2019
@rachelandrew rachelandrew moved this from Needs action to Done in css-multicol-1 Sep 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
css-multicol-1
Projects
css-multicol-1
  
Done
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
@frivoal @myakura @rachelandrew