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

Conversation

@rachelandrew
Copy link
Collaborator

Examples for:

  • flex-direction
  • flex-flow
  • flex-basis
  • flex-grow
  • flex-shrink

@@ -0,0 +1,36 @@
<section id="example-choice-list" class="example-choice-list large" data-property="flex-basis">

Choose a reason for hiding this comment

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

Super nit: remove empty line for consistency with other files.

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.

Some nits and a few suggestions. This is going to be so useful to so many people! Thanks @rachelandrew

</div>



Choose a reason for hiding this comment

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

Super nit: remove empty lines for consistency with other files.

@@ -0,0 +1,41 @@
<section id="example-choice-list" class="example-choice-list large" data-property="flex-direction">

Choose a reason for hiding this comment

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

Super nit: remove empty line for consistency with other files.

<span class="visually-hidden">Copy to Clipboard</span>
</button>
</div>

Choose a reason for hiding this comment

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

Super nit: remove empty line for consistency with other files.

@@ -0,0 +1,44 @@
<section id="example-choice-list" class="example-choice-list large" data-property="flex-flow">

Choose a reason for hiding this comment

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

Super nit: remove empty line for consistency with other files.

<span class="visually-hidden">Copy to Clipboard</span>
</button>
</div>

Choose a reason for hiding this comment

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

Super nit: remove empty line for consistency with other files.

@@ -0,0 +1,36 @@
<section id="example-choice-list" class="example-choice-list large" data-property="flex-shrink">

Choose a reason for hiding this comment

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

Super nit: remove empty line for consistency with other files.

</div>



Choose a reason for hiding this comment

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

Super nit: remove empty lines for consistency with other files.

<div id="output" class="output large hidden">
<section id="default-example" class="default-example">
<div class="example-container">
<div id="example-element" class="transition-all">Item One</div>

Choose a reason for hiding this comment

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

I am wondering, could we title this one as "I grow"? Makes it clearer that the flex-grow rule is applied to this specific element.

<div id="output" class="output large hidden">
<section id="default-example" class="default-example">
<div class="example-container">
<div id="example-element" class="transition-all">Item One</div>

Choose a reason for hiding this comment

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

I am wondering, could we title this one as "I shrink"? Makes it clearer that the flex-shrink rule is applied to this specific element.

</div>

<div class="example-choice">
<pre><code class="language-css">flex-shrink: 3;</code></pre>

Choose a reason for hiding this comment

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

Maybe these could be? Thoughts @rachelandrew @wbamberg

  1. flex-shrink: 0;
  2. flex-shrink: 1;
  3. flex-shrink: 2;

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 @rachelandrew 🎉

@schalkneethling schalkneethling merged commit 27e144d into mdn:master Feb 8, 2018
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)
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