Skip to content

#609: Registries Configuration summary section overflows on smaller screens#615

Merged
xingzhang-suse merged 2 commits into
rancher:mainfrom
rushk014:main
Jun 17, 2026
Merged

#609: Registries Configuration summary section overflows on smaller screens#615
xingzhang-suse merged 2 commits into
rancher:mainfrom
rushk014:main

Conversation

@rushk014

Copy link
Copy Markdown
Contributor

Description

Fix #609
Reworks status update row layout to match Figma design.
Screenshot 2026-06-17 at 4 34 51 PM

Test

  1. Open Registries Configuration with registries that have long names/URIs
  2. At default viewport: verify both summary panels are 50% width, status rows show registry name, URI, badges, and update time without overflow (albeit possible truncation)
  3. Shrink the viewport — verify content clips gracefully (rows truncate, panels don't bleed into each other)
  4. Verify "In progress" and other badge text never wraps or truncates
  5. Expand viewport — verify registry name and URI columns both grow proportionally

…on smaller screens

Rework status update row layout to match Figma design.
Cap left panel at 50% width, truncate long registry names and URIs
with ellipsis, and ensure badge text is never truncated or wrapped.
@rushk014 rushk014 requested a review from xingzhang-suse June 17, 2026 20:35
// Sort and limit the registryStatusList to 5 most recent updates
registryStatusList.sort((a, b) => new Date(b.lastTransitionTime) - new Date(a.lastTransitionTime)).slice(0, 5);
// Sort the registryStatusList by most recent update
registryStatusList.sort((a, b) => new Date(b.lastTransitionTime) - new Date(a.lastTransitionTime));

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Note the .slice(0, 5) is redundant since the return value is never assigned. The top5StatusUpdates from which registryStatusList is computed is already truncated above on L186 so the length will always be <= 5

@xingzhang-suse xingzhang-suse left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@xingzhang-suse xingzhang-suse merged commit d0e1eb5 into rancher:main Jun 17, 2026
4 checks passed
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.

Registries Configuration summary section overflows on smaller screens

2 participants