-
Notifications
You must be signed in to change notification settings - Fork 284
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes #36968 - Add container manifest list to UI and aria-label for repo icon #10821
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -55,6 +56,11 @@ const AdditionalCapsuleContent = ({ counts }) => { | |||
{`${dockerManifestCount} Container manifests`}<br /> | |||
</> | |||
} | |||
{dockerManifestListCount > 0 && | |||
<> | |||
{`${dockerManifestListCount} Container manifest lists`}<br /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added https://projects.theforeman.org/issues/36988 for adding the CV version container manifest list counts and comparisons.. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Translations work now!
…epo icon (Katello#10821) * Fixes #36968 - Add container manifest list to UI and aria-label for repo icon * Fixes #36968 - Add translations to content counts (cherry picked from commit c40e0d8)
What are the changes introduced in this pull request?
Considerations taken when implementing this change?
What are the testing steps for this pull request?
Create a docker repo with manifest lists. ex: libpod/testimage from quay.io
Sync repo and have the repo sync to a capsule
Check the capsule content page to make sure you see manifest lists in content counts.
Also do an inspect element on repo type icon. It should have aria-lebel = Repo type