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

[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 Current Work

Comments

@myakura
Copy link
Contributor

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.

@myakura myakura added the css-multicol-1 Current Work label Sep 10, 2019
@frivoal
Copy link
Collaborator

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

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 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 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
Labels
css-multicol-1 Current Work
Projects
Development

No branches or pull requests

3 participants