Skip to content

Conversation

@ryankeairns
Copy link
Contributor

@ryankeairns ryankeairns commented Mar 14, 2025

Related to #8425 (comment)

Important

This will be shared as a POC prior to review/merge

Summary

The now-previous theme, Amsterdam, brought a larger base font size with noticeably heavy titles. Some of these aspects were baked directly into the EuiPageHeader.

  1. Tone down some hallmark elements of Amsterdam
  2. Fix spacing between right side items

Changes

For item 1

  • Decrease title size (and associated icon)
  • Decrease description size
  • Decrease tabs size

For item 2

  • Gutter size reduced on right-side items to match the original design as long seen in the Figma component.

EUI docs

Note

The current EUI docs site uses the page header component.

This PR contains a style override that preserves visual hierarchy of headings in the EUI docs site. The style override results in no visual changes to longtime users of the current docs site (i.e. title font size remains 30px).

Alternatively, I considered reworking all the 'guidelines' titles - downsizing each - but decided against this given the imminent switchover to EUI+.


XL tabs

Note

It is likely we will deprecate/downsize xl tabs, separately.

QA

Remove or strikethrough items that do not apply to your PR.

General checklist

  • Browser QA
    • Checked in both light and dark modes
    • Checked in mobile
    • Checked in Chrome, Safari, Edge, and Firefox
    • Checked for accessibility including keyboard-only and screenreader modes
  • Docs site QA
  • Code quality checklist
  • Release checklist
    • A changelog entry exists and is marked appropriately.
    • If applicable, added the breaking change issue label (and filled out the breaking change checklist)
  • Designer checklist
    • If applicable, file an issue to update EUI's Figma library with any corresponding UI changes. (This is an internal repo, if you are external to Elastic, ask a maintainer to submit this request)

@kibanamachine
Copy link

Preview staging links for this PR:

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

@ryankeairns
Copy link
Contributor Author

Fixed the gap here; will address page title heading separately.

@ryankeairns ryankeairns closed this May 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants