-
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
Fix Snapshots Policies Alignment Issue in IE11 #54866
Fix Snapshots Policies Alignment Issue in IE11 #54866
Conversation
…ied and tested on all 3 browsers.
@elasticmachine merge upstream |
user doesn't have permission to update head repository |
@elasticmachine merge upstream |
user doesn't have permission to update head repository |
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.
@cuff-links thanks for fixing this! I had one suggestion related to your change.
.../plugins/snapshot_restore/public/app/sections/home/policy_list/policy_table/policy_table.tsx
Show resolved
Hide resolved
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
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.
LGTM. Thanks for fixing this @cuff-links!
…t-of-legacy * 'master' of github.com:elastic/kibana: (142 commits) [Vis] Move Timelion Vis to vis_type_timelion (elastic#52069) Deprecate `chrome.navlinks.update` and add documentation (elastic#54893) [ML] Single Metric Viewer: Fix time bounds with custom strings. (elastic#55045) [Vis: Default editor] EUIficate and Reactify the sidebar (elastic#49864) [Mappings editor] Fix cannot set boolean value for "null_value" param (elastic#55015) [SIEM] Adds support for apm-* to the network map (elastic#54876) [Reporting] Define shims of legacy dependencies (elastic#54082) Resolver is overflow: hidden to prevent obscured elements from showing up (elastic#55076) Upgraded EUI to 18.2.1 (elastic#55090) [Maps] Support styles on agg fields with _of_ in name (elastic#54965) Remove xpack_main requirement, it's no longer in use (elastic#55060) Fix Snapshots Policies Alignment Issue in IE11 (elastic#54866) first rule cuts (elastic#54990) [DOCS] Adds geocentroid note to coordinate map (elastic#54389) [Canvas] Fixes the Copy Post Url link (elastic#54831) Fixes bugs with full screen filters (elastic#54792) [ML] Fix decoding in the URL state (elastic#54915) Remove redundant `x-pack/typings`. (elastic#55042) [SIEM][Detection Engine] Adds critical missing status route to prepackaged rules Generate legacy vars when rendering all applications (elastic#54768) ... # Conflicts: # x-pack/plugins/translations/translations/ja-JP.json # x-pack/plugins/translations/translations/zh-CN.json
* Removed flex group because it's causing alignment issues on IE. Verified and tested on all 3 browsers. * Restored <EuiFlexItem> but added grow=false to properly align icons.
* Removed flex group because it's causing alignment issues on IE. Verified and tested on all 3 browsers. * Restored <EuiFlexItem> but added grow=false to properly align icons.
* Removed flex group because it's causing alignment issues on IE. Verified and tested on all 3 browsers. * Restored <EuiFlexItem> but added grow=false to properly align icons.
* Removed flex group because it's causing alignment issues on IE. Verified and tested on all 3 browsers. * Restored <EuiFlexItem> but added grow=false to properly align icons.
* Removed flex group because it's causing alignment issues on IE. Verified and tested on all 3 browsers. * Restored <EuiFlexItem> but added grow=false to properly align icons. Co-authored-by: Elastic Machine <[email protected]>
* Removed flex group because it's causing alignment issues on IE. Verified and tested on all 3 browsers. * Restored <EuiFlexItem> but added grow=false to properly align icons. Co-authored-by: Elastic Machine <[email protected]>
Fixes #49621 Removed flex group because it's causing alignment issues on IE. Verified and tested on all 3 browsers. Screenshot is of IE.