-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Lens] Quick Function Incompatability Indicator Getting Cut Off #160046
Conversation
0cc2080
to
f4f1cbb
Compare
f4f1cbb
to
2b11dc0
Compare
Pinging @elastic/kibana-visualizations @elastic/kibana-visualizations-external (Team:Visualizations) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested locally with Safari 👍
<EuiFlexItem grow={false} style={{ marginRight: euiTheme.size.xs }}> | ||
{label} | ||
<EuiFlexItem grow={false} style={{ marginRight: euiTheme.size.xs, minWidth: 0 }}> | ||
<span className="lnsIndexPatternDimensionEditor__operationLabel">{label}</span> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: is still scss preferred over emotion
css prop?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably not, I just missed the change 😅 Will change!
a8e2cee
to
715d091
Compare
💚 Build Succeeded
Metrics [docs]Async chunks
Unknown metric groupsESLint disabled line counts
Total ESLint disabled count
History
To update your PR or re-run it, just comment with: |
Summary
Fixes #152520
Before

After

Also fixed broken theme and missing I18nProvider from the extra action that has a separate mounting point.
