-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Revamp Copy Pull Command #21155
base: main
Are you sure you want to change the base?
Revamp Copy Pull Command #21155
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #21155 +/- ##
==========================================
+ Coverage 45.36% 46.21% +0.84%
==========================================
Files 244 248 +4
Lines 13333 13945 +612
Branches 2719 2880 +161
==========================================
+ Hits 6049 6445 +396
- Misses 6983 7157 +174
- Partials 301 343 +42
Flags with carried forward coverage won't be shown. Click here to find out more. |
61337ab
to
5f4a463
Compare
ready for review |
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.
.
92e89c8
to
b0bab10
Compare
We can certainly discuss this from different angles, but at the end of the day, UX should always prioritize simplicity and clarity. 1 click > 3 clicks.Our focus should always be on what's best for the user. The UI should be intuitive, easy to understand, and foolproof, not a guessing game. For example, having a copy button above the datagrid, similar to Docker Hub, makes it much easier for users to copy exactly what they need, with clear feedback. This saves them unnecessary clicks and results in better UX overall. Thanks for considering this perspective! |
Yes I do. |
There is a poll going regarding some UX aspects: Please vote if you didn't do so: |
This is something we really want. I did just want to make a suggestion and I hope it isn't too much work. In the preferences section, can you add "apptainer" and "singularity"? Or let those be defined by something in the config so the admins can set? As a harbor admin, I'd like to set a default container runtime based on the container tools we use for all users as well. |
This is a good idea. We will add a option in the dropdown where the user can define their prefered container runtime |
* This commit moves "Copy Pull Command" button inside the table * and add a separate column for better usability Signed-off-by: bupd <[email protected]>
* This Commit adds Preferences in navbar * Updates the navbar Signed-off-by: bupd <[email protected]>
Signed-off-by: bupd <[email protected]>
Signed-off-by: bupd <[email protected]>
Signed-off-by: bupd <[email protected]>
Signed-off-by: bupd <[email protected]>
Signed-off-by: bupd <[email protected]>
Signed-off-by: bupd <[email protected]>
Signed-off-by: bupd <[email protected]>
Signed-off-by: bupd <[email protected]>
Signed-off-by: bupd <[email protected]>
Signed-off-by: bupd <[email protected]>
* Updates title of preference settings * Updates container runtime to pull cmd prefix Signed-off-by: bupd <[email protected]>
Overview
1 click > 3 clicks
UX should prioritize simplicity and clarity. By adding a copy button above and in the datagrid, similar to DockerHub and other registries, users can quickly copy exactly what they need with clear feedback, eliminating unnecessary clicks and improving overall UX.
After reviewing the discussion (#21284), the consensus is clear:
This design reduces friction and enhances usability.
To-do:
Note
This design is based on thorough user feedback. As one user mentioned,
Before
After
Updated copy pull command Flow
Added Preference Settings
Summary of Changes Made
docker pull project/repo
).These changes revamp the user experience, reducing clicks and improving the flow for copying pull commands.
Issue being fixed
Fixes #21146
Please indicate you've done the following: