Skip to content

[Fix]Revert user profile row height - #288451

Open
christineweng wants to merge 1 commit into
elastic:mainfrom
christineweng:update-user-profile-row-height
Open

[Fix]Revert user profile row height#288451
christineweng wants to merge 1 commit into
elastic:mainfrom
christineweng:update-user-profile-row-height

Conversation

@christineweng

Copy link
Copy Markdown
Contributor

Summary

Fix a user profile spacing issue that was changed in #260949, reverted back to 48px.

Before -> After
image

Checklist

  • Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support
  • Documentation was added for features that require explanation or tutorials
  • Unit or functional tests were updated or added to match the most common scenarios
  • If a plugin configuration key changed, check if it needs to be allowlisted in the cloud and added to the docker list
  • This was checked for breaking HTTP API changes, and any breaking changes have been approved by the breaking-change committee. The release_note:breaking label should be applied in these situations.
  • Flaky Test Runner was used on any tests changed
  • The PR description includes the appropriate Release Notes section, and the correct release_note:* label is applied per the guidelines
  • Review the backport guidelines and apply applicable backport:* labels.

@christineweng
christineweng requested review from mgadewoll and a lite review from Copilot September 1, 2026 15:57
@christineweng christineweng self-assigned this Sep 1, 2026
@christineweng
christineweng requested a review from a team as a code owner September 1, 2026 15:57
@christineweng christineweng added backport:version Backport to applied version labels v9.6.0 v9.5.3 labels Sep 1, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Note

Copilot was unable to run its full agentic suite in this review.

Pull request overview

Reverts the user profile selectable list row height to address a spacing regression introduced in #260949 by restoring the previous 48px row height.

Changes:

  • Changed EuiSelectable listProps.rowHeight from 32 back to 48 for user profile rows.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@kibanamachine

Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

cc @christineweng

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants