Skip to content

Conversation

@pickypg
Copy link
Member

@pickypg pickypg commented Dec 1, 2016

The padding pushes the text up.

Before:

Padding

After:

No Padding

The padding pushes the text up.
@pickypg pickypg added bug Fixes for quality problems that affect the customer experience good first issue low hanging fruit review v5.1.1 v5.2.0 v6.0.0 labels Dec 1, 2016
@pickypg
Copy link
Member Author

pickypg commented Dec 1, 2016

cc @cjcenizal

@uboness
Copy link

uboness commented Dec 1, 2016

why is the logo green?

@pickypg
Copy link
Member Author

pickypg commented Dec 1, 2016

@uboness I just happened to take the screenshot at the same time I guess. It transitions through the colors and has for awhile, right?

Pink -> Yellow -> Green -> Pink -> ...

@cjcenizal
Copy link
Contributor

cjcenizal commented Dec 1, 2016

Hey Chris... we had a very spirited, long discussion about this on the original PR: #8970

Some people really don't like the way it is... some people really do... in such a case I think we can make an objective decision with either a) a vote or b) relying on convention / best practices. Since a vote will probably just be tampered with by Russian hackers, I think we need to rely on the design field practice of visual centering.

Could we (please please please) close this PR and just put the debate to rest? 😂 🙏 🍺 🍺 🍺

@pickypg
Copy link
Member Author

pickypg commented Dec 1, 2016

Haha. Sure, I didn't realize this was debated. I saw it was "off" (guess I'm one of those people) and went in to fix it.

@pickypg pickypg closed this Dec 1, 2016
@pickypg pickypg deleted the fix/vertically-center-loading-kibana branch December 1, 2016 23:25
acstll added a commit that referenced this pull request Jan 28, 2026
`112.0.0` ⏩ `112.1.0`

[Questions? Please see our Kibana upgrade
FAQ.](https://github.com/elastic/eui/blob/main/wiki/eui-team-processes/upgrading-kibana.md#faq-for-kibana-teams)

## Changes

- Update `span_links_badge.tsx` conditional `onClick` props on
`EuiBadge` to satisfy TypeScript
93d7fae
- Update snapshots 24c2f9a

## Package updates

### `@elastic/eui`
[`v112.1.0`](https://github.com/elastic/eui/releases/tag/v112.1.0)

- Updated `timeline` icon glyph
([#9331](elastic/eui#9331))
- Updated `EuiContextMenu` panels to allow passing `data-test-subj`,
`aria-label`, `className` and `css` props
([#9323](elastic/eui#9323))
- Added "zoom in" functionality to time window buttons in
`EuiSuperDatePicker` ([#9325](elastic/eui#9325))
- Added `displayName` to `EuiButton`, `EuiButtonEmpty`,
`EuiDescriptionList` and its sub-components, `EuiEmptyPrompt`,
`EuiFlexGrid`, `EuiFlexItem`, `EuiIcon`, `EuiImage`, `EuiLoadingLogo`,
`EuiPageSection`, `EuiPageSidebar`, `EuiPageTemplate` and its
sub-components and `EuiPanel`
([#9324](elastic/eui#9324))
- Added `fill` prop (defaults to `false`) to `EuiBadge` component that
controls whether the badge should use filled or non-filled (less
intense) colors. By default, badges will now render as the non-filled
variant. ([#9306](elastic/eui#9306))
- Updated EuiBadge design to have rounded corners and improved paddings
([#9302](elastic/eui#9302))

**Bug fixes**

- Fixed non-virtualized `EuiSelectable` throwing SyntaxError when
selecting an option ([#9326](elastic/eui#9326))
- Fixed an issue where `push` flyouts in a stacked layout calculated the
content offset based on the hidden main flyout's width instead of the
visible child flyout's width
([#9322](elastic/eui#9322))

### @elastic/eui-theme-borealis
[`v5.4.0`](https://github.com/elastic/eui/blob/16f9b31d753d963d7738049e7a176fcaf6e81e73/packages/eui-theme-borealis/changelogs/CHANGELOG_2026.md#v540)

- Updated `badgeBackground` color token value to equal
`backgroundFilledText`
([#9306](elastic/eui#9306))

### @elastic/eui-docusaurus-theme
[`v2.2.0`](https://github.com/elastic/eui/blob/16f9b31d753d963d7738049e7a176fcaf6e81e73/packages/docusaurus-theme/changelogs/CHANGELOG_2026.md#v220)

- Added `extraFiles` prop to the `Demo` component. It allows to pass
extra files that will be added to the Codesandbox instance.
([#9317](elastic/eui#9317))
- Updated the `IMPORT_REGEX` to include relative imports so that all
imports are removed from the snippet. All imported references have to be
passed to `Demo` in the `scope` prop.
([#9317](elastic/eui#9317))

---------

Co-authored-by: Tomasz Kajtoch <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Fixes for quality problems that affect the customer experience good first issue low hanging fruit won't fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants