fix: Layout issue with flex and layout breakpoints #6528

[aigoncharov]

fix: Layout issue with flex and layout breakpoints

Change class definition order in flex-properties-for-name mixin.

Closes #6528

[googlebot]

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.

---

• If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
• If you signed the CLA as a corporation, please let us know the company's name.

[googlebot]

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.

---

• If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
• If you signed the CLA as a corporation, please let us know the company's name.

[aigoncharov]

I signed it!

[googlebot]

CLAs look good, thanks!

[googlebot]

CLAs look good, thanks!

[ThomasBurleson]

This will not work in you have markup like this:

<div layout="row" layout-sm="column">
  <div flex="33"  flex-sm="95">
  <div flex="33"  flex-sm="95">
  <div flex="33"  flex-sm="95">
</div>

[aigoncharov]

Sorry, my bad! Kind of never user 33 and 66. FIxed it up.

[aigoncharov]

Sorry, my bad! Kind of never user 33 and 66. FIxed it up.

[ThomasBurleson]

@keenondrums - why did you change the class definition order in flex-properties-for-name mixin.

[aigoncharov]

@ThomasBurleson, maybe my mind is not at the right place but that was kind of the whole issue behind the bug described in the related issue #6528. Please correct me if I'm wrong. I'd be happy to answer any questions via Skype if that's more convenient for you. My skype id is keenondrums.