Skip to content

fixing metric vis to correctly show scrollbars when overflown#9481

Merged
ppisljar merged 1 commit intoelastic:masterfrom
ppisljar:metric/overflow
Dec 19, 2016
Merged

fixing metric vis to correctly show scrollbars when overflown#9481
ppisljar merged 1 commit intoelastic:masterfrom
ppisljar:metric/overflow

Conversation

@ppisljar
Copy link
Contributor

fixes #8782

Large values in the metric vis can get cut off on the left hand side, with no way to scroll and view it.

By simply applying align-items: center and justify-content: center to the flex container, your flex item(s) will be vertically and horizontally centered.

However, there is a problem with this method when the flex item is bigger than the flex container. When the flex item overflows the container the left side becomes inaccessible.

To work around this margin: auto is applied to the child of flex container instead of align-items: center and justify-content: center

Copy link
Contributor

@Bargs Bargs left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you for fixing this @ppisljar!

@tbragin tbragin added the Feature:Visualizations Generic visualization features (in case no more specific feature label is available) label Dec 16, 2016
@weltenwort weltenwort assigned weltenwort and unassigned weltenwort Dec 19, 2016
@weltenwort weltenwort self-requested a review December 19, 2016 11:04
Copy link
Member

@weltenwort weltenwort left a comment

Choose a reason for hiding this comment

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

LGTM

@ppisljar ppisljar merged commit 90c821b into elastic:master Dec 19, 2016
elastic-jasper added a commit that referenced this pull request Dec 19, 2016
Backports PR #9481

**Commit 1:**
fixing metric vis to correctly show scrollbars when overflown

* Original sha: d046424
* Authored by ppisljar <peter.pisljar@gmail.com> on 2016-12-14T10:03:54Z
ppisljar pushed a commit that referenced this pull request Dec 19, 2016
Backports PR #9481

**Commit 1:**
fixing metric vis to correctly show scrollbars when overflown

* Original sha: d046424
* Authored by ppisljar <peter.pisljar@gmail.com> on 2016-12-14T10:03:54Z
@ppisljar ppisljar deleted the metric/overflow branch December 19, 2016 12:23
stacey-gammon pushed a commit to stacey-gammon/kibana that referenced this pull request Jan 19, 2017
clean up

show spy panel in embed mode

[webpack] fix loader query string usage (elastic#9497)

* [webpack] pin to fork with fixed loader aliases

* [optimizer] upgrade to postcss+autoprefixer

* [timelion] convert uiExports.modules -> webpackShims

* [uiExports] remove implementation-leaking and unused uiExport types

* [optimizer] remove unused imports

* [uiBundlerEnv] add a method for exporting global import aliases for special cases

[dev tools] Hide app link when there are no tools (elastic#9489)

* [dev tools] Hide app link when there are no tools

* [dev tools] Add tests for setting app as hidden

pie chart unhandled error fix (elastic#9422)

Add NoResults and Panel components. (elastic#9516)

* Add NoResults and Panel components.

* Lighten noResults text.

Update ToolBarFooter component to support content on the left side. (elastic#9514)

Fix bug with Button component appearance inside of a ToolBar. (elastic#9526)

Make basic Button hover state the same both in and out of ToolBar. (elastic#9528)

[grunt/eslint] fix precommit linting (elastic#9510)

* [grunt/eslint] fix precommit linting

 - remove use of `minimatch.makeRe()` because it does not support the entire glob syntax
 - log a warning whenever a js file is excluded by the `lintStagedFiles` task
 - eslint globs are relative to the project root, ensure that we check against relative version

* [grunt/eslint] only log warning wtr grunt paths

Add Tabs component. (elastic#9536)

- Fix bugs with Button and CheckBox focused states.
- Fix appearance of cell content in Table.

Disable linting for Tabs component example JS. (elastic#9538)

Set Button component to display: inline-block, to ensure it has the same height when applied to both button elements and anchor tags. (elastic#9541)

fixing metric vis to correctly show scrollbars when overflown (elastic#9481)

Adding Safari 7 support to autoprefixer (elastic#9534)

PhantomJS is using a rather outdated version of WebKit, which requires
various css-prefixes to render correctly. PhantomJS doesn't have a specific
user-agent, and Safari 7 is the closet version of WebKit.

use Stop Editing instead of Preview

Warn the user if they Stop Editing with unsaved changes

- Refresh the dashboard after stop editing so unsaved changes are lost
and no temporary edits will be shown in non-edit mode.

Don't watch the variable on scope, but the config attribute
@ppisljar ppisljar restored the metric/overflow branch September 26, 2018 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backported Feature:Visualizations Generic visualization features (in case no more specific feature label is available) review v5.2.0 v6.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Large values get cut off in metric vis

5 participants