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

Fixed regressions from Global Time Picker #1044

Merged
merged 3 commits into from
Jul 23, 2018

Conversation

cchaos
Copy link
Contributor

@cchaos cchaos commented Jul 23, 2018

Removed IE flex column fix, in favor of forcing the consumer to add a grow prop.

Added a IE message to docs.

Depending on the nested structure of your flex groups, it is possible that flex-items inside a column directed flex group will not show. To counter this, add the grow prop and set to either false or a number. Setting grow to true will not suffice. You may also need to adjust the flex-basis value.


AND removed max-width on universal selector of popover children which messed up the popover title.

In favor of forcing the consumer to add a `grow` prop. Added a IE message to docs.
@cchaos cchaos added the bug label Jul 23, 2018
@cchaos cchaos requested review from chandlerprall and snide July 23, 2018 20:37
@cchaos cchaos changed the title Removed IE flex column fix Fixed regression from Global Time Picker Jul 23, 2018
@cchaos cchaos changed the title Fixed regression from Global Time Picker Fixed regressions from Global Time Picker Jul 23, 2018
@cchaos
Copy link
Contributor Author

cchaos commented Jul 23, 2018

Ugh, sorry... I'm just gonna stop messing with things.

Copy link
Contributor

@chandlerprall chandlerprall left a comment

Choose a reason for hiding this comment

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

change LGTM

Copy link
Contributor

@snide snide left a comment

Choose a reason for hiding this comment

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

Small comment on changelog.


## [`3.2.0`](https://github.com/elastic/eui/tree/v3.2.0)

**Note: this release creates a minor regression to the display of `EuiFlexItem`s inside a `column` `EuiFlexGroup`. This is fixed in `master`.**
Copy link
Contributor

Choose a reason for hiding this comment

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

Was this an IE regression only, or at large? If only IE, can you label it. I wouldn't mention master (maybe, ("use a later release instead"), because master will change. Let's just pop a new release 3.2.1 once this goes in.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, it wasn't IE only.

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 put in "master" for now, but it should be changed when a release is made to point to the correct release version number as has been the pattern.

@cchaos cchaos merged commit aa1e8b7 into elastic:master Jul 23, 2018
@cchaos cchaos deleted the flex-column-fix branch July 23, 2018 22:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants