Skip to content
This repository was archived by the owner on Oct 17, 2025. It is now read-only.

Conversation

@mfluehr
Copy link
Contributor

@mfluehr mfluehr commented Feb 6, 2018

This includes column-count, column-fill, column-gap, column-rule-color,column-rule-style,column-rule-width, column-rule, column-width, and columns.

Copy link

@schalkneethling schalkneethling left a comment

Choose a reason for hiding this comment

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

Couple of items to address. Thanks a lot for all your hard work @mfluehr

</div>

<div class="example-choice">
<pre><code id="example_six" class="language-css">column-rule-style: inset;</code></pre>

Choose a reason for hiding this comment

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

It is very hard to see the difference between solid, ridge and inset. If you lighten the column-rule-color to say #999 for ridge, it is clearer. My suggestion would then be, drop the inset example, and add the lighter column-rule-color for the ridge example.

</div>

<div class="example-choice">
<pre><code id="example_four" class="language-css">column-width: 19ch;</code></pre>

Choose a reason for hiding this comment

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

Not sure if it is a browser support issue but, clicking through these I do not see any changes to the columns on the right. Looking at browser support, it does not look like that is the issue. Perhaps this is not well suited for interactive examples, as it may rely on resizing the content output area? @mfluehr @wbamberg

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I took another stab at this, so take a look. It's showing variety for me now in both Firefox and Chrome. (However, results do vary some depending on browser size.)

</div>

<div class="example-choice">
<pre><code id="example_two" class="language-css">columns: 10rem 2;</code></pre>

Choose a reason for hiding this comment

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

I wonder, could something like this be interesting here? columns: 6rem auto; - When you play around with the rem value you can see how the browser splits the content into more or less rows. Thoughts @mfluehr

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, good idea.

Copy link

@schalkneethling schalkneethling left a comment

Choose a reason for hiding this comment

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

r+ Thanks @mfluehr

@schalkneethling schalkneethling merged commit 662660d into mdn:master Feb 8, 2018
@mfluehr mfluehr deleted the multi-column-layout branch February 8, 2018 13:35
wbamberg pushed a commit to wbamberg/interactive-examples that referenced this pull request Feb 8, 2018
* upstream/master:
  Adding list-style css example. (mdn#547)
  Flex examples (mdn#558)
  Add column examples (mdn#549)
  Remove CSS example id attributes (mdn#556)
  Add various text examples (mdn#545)
  Add contribution item (mdn#552)
  Adds @helmutgranda as contributor (mdn#550)
  Adding border radius example with recommended changes. (mdn#546)
  Add `quotes` example (mdn#543)
@wbamberg wbamberg mentioned this pull request Feb 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants