feat(eui): remove xl size from EuiTabs

[weronikaolejniczak]

Summary

Important

This is a breaking change.

Closes #8302

On this PR, I:

• feat(eui): remove xl size from EuiTabs
• chore(eui): update snapshot tests

Kibana impact

There are no usages in Kibana of EuiTabs with xl size (search) or EuiPageHeader / KibanaPageTemplate.Header with tabs but no pageTitle.

Cloud UI impact

There are no usages in Cloud UI of EuiTabs with xl size (search) or EuiPageHeader with tabs but no pageTitle.

QA

Specific checklist

• EUI builds with no errors
• Unit tests pass
• The Kibana and Cloud UI affected usages list is correct

General checklist

• Browser QA
  • Checked in both light and dark modes
  • Checked in both MacOS and Windows high contrast modes
    • (emulate forced colors if you do not have access to a Windows machine.)
  • Checked in mobile
  • Checked in Chrome, Safari, Edge, and Firefox
  • Checked for accessibility including keyboard-only and screenreader modes
• Docs site QA
  • Added documentation
  • Props have proper autodocs (using @default if default values are missing) and playground toggles
  • Checked Code Sandbox works for any docs examples
• Code quality checklist
  • Added or updated jest ~and cypress tests~
  • Updated visual regression tests
• 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)

[ryankeairns]

Would page_header_content.tsx also need to change?

eui/packages/eui/src/components/page/page_header/page_header_content.tsx

Line 275 in 803bdb9

const tabsSize: EuiTabsProps['size'] = pageTitle ? 'l' : 'xl';

[weronikaolejniczak]

@ryankeairns yes, we'd need to change page_header_content.tsx which would affect EuiPageHeader and KibanaPageTemplate.Header.

[github-actions]

This PR contains breaking changes. The opener of this pull request is asked to perform the following due diligence steps below, to assist EUI in our next Kibana upgrade:

• If this PR contains prop/API changes:
  • Search through Kibana for <EuiComponent usages (example search)
  • In the PR description or in a PR comment, include a count or list with the number of component usages in Kibana that will need to be updated (if that amount is "none", include that information as well)
• If this PR contains CSS changes:
  • Search through Kibana for the changed EUI selectors, e.g. .euiComponent (example search)
  • In the PR description or in a PR comment, include a count or list with the number of custom CSS overrides in Kibana that will need to be updated (if that amount is "none", include that information as well)
• 🔍 Tip: When searching through Kibana, consider excluding **/target, **/*.snap, **/*.storyshot files to reduce noise and only look at source code usages
• ⚠️ For extremely risky changes, the EUI team should potentially consider the following precautions:
  • Using a pre-release release candidate to test Kibana CI ahead of time
  • Using kibana-a-la-carte for manual QA, and to give other Kibana teams a staging server to quickly test against

Log | Bot Usage

[elasticmachine]

💚 Build Succeeded

• Buildkite Build
• Commit: a042b8c
• Documentation website

History

• 💔 Build #422 failed e6bd07c

cc @weronikaolejniczak

[elasticmachine]

💚 Build Succeeded

• Buildkite Build
• Commit: a042b8c

History

• 💔 Build #4114 failed e6bd07c

cc @weronikaolejniczak

[mgadewoll]

🟢 Changes and component usage stats for consumer upgrades LGTM 👍