Skip to content

fix: v1 overview pages#743

Merged
dennisvankekem merged 8 commits into
mainfrom
APL-1630-clean
Mar 25, 2026
Merged

fix: v1 overview pages#743
dennisvankekem merged 8 commits into
mainfrom
APL-1630-clean

Conversation

@dennisvankekem
Copy link
Copy Markdown
Collaborator

Considerations

  • I have tested the changes in both light and dark mode.
  • I have considered the need for new unit tests.
  • I have tested the changes on a cluster.
  • I have included relevant documentation updates.
  • I have an approved Figma design or have reflected my changes in Figma
  • I have verified that the UI/UX is consistent in major browsers (e.g., Chrome, Firefox, Safari, Edge).
  • I have tested the changes for responsiveness in different screen resolutions.
  • I have tested expected error states and verified that the user is presented with informative error messages.
  • I have tested the feature with unusual or extreme inputs (e.g., very long strings, empty states, clicking a button multiple times quickly).

@ElderMatt
Copy link
Copy Markdown
Contributor

Why does the repository now show the full URL instead of only the copy icon?
image

@ElderMatt
Copy link
Copy Markdown
Contributor

It is also not aligned properly.
image

Copy link
Copy Markdown
Contributor

@ElderMatt ElderMatt left a comment

Choose a reason for hiding this comment

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

container overview page needs changes.

@dennisvankekem
Copy link
Copy Markdown
Collaborator Author

@ElderMatt apparently it's intended behavior to show the full URL it just didn't work before, it also scales well so let's keep it as is.

image

Copy icon has been aligned

Comment thread src/pages/builds/overview/BuildsOverviewPage.tsx Outdated
Comment thread src/pages/builds/overview/BuildsOverviewPage.tsx
@dennisvankekem dennisvankekem merged commit 8fd2057 into main Mar 25, 2026
6 checks passed
@dennisvankekem dennisvankekem deleted the APL-1630-clean branch March 25, 2026 14:25
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