Bug 2221902 - Edit icon needs to be standardized
Summary: Edit icon needs to be standardized
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Container Native Virtualization (CNV)
Classification: Red Hat
Component: User Experience
Version: 4.14.0
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: ---
: ---
Assignee: Tal Nisan
QA Contact: Guohua Ouyang
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2023-07-11 08:28 UTC by Avital Pinnick
Modified: 2023-07-20 09:56 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2023-07-17 16:12:55 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Inconsistent edit icons (43.70 KB, image/png)
2023-07-11 08:28 UTC, Avital Pinnick
no flags Details
Edit icons with labels (42.42 KB, image/png)
2023-07-11 08:29 UTC, Avital Pinnick
no flags Details
VM Template Scripts tab and Edit label with pencil icon (94.10 KB, image/png)
2023-07-12 10:27 UTC, Hilda Stastna
no flags Details
Review and create VirtualMachine Scripts tab with Edit label and pencil icon (121.12 KB, image/png)
2023-07-12 10:28 UTC, Hilda Stastna
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker CNV-30855 0 None None None 2023-07-11 08:29:04 UTC

Description Avital Pinnick 2023-07-11 08:28:47 UTC
Created attachment 1975105 [details]
Inconsistent edit icons

Description of problem:

The edit icon (pencil) appears frequently in the UI. Sometimes it appears with a label ("Edit") and sometimes without. I recommend that this be standardized so that the label is removed.

Comment 2 Hilda Stastna 2023-07-11 11:42:12 UTC
Hi Yifat,

WDYT about this issue? Thanks!

Comment 3 Yifat Menchik 2023-07-11 13:19:06 UTC
The only place it appears with the label is in the "Labels" section. In all other places only the pencil icon is shown. Anyway, this is a console behavior that was already discussed in the past. Maybe @gouyang can recall what was the explanation for it when it was raised last time.

Comment 4 Guohua Ouyang 2023-07-12 07:37:39 UTC
(In reply to Yifat Menchik from comment #3)
> The only place it appears with the label is in the "Labels" section. In all
> other places only the pencil icon is shown. Anyway, this is a console
> behavior that was already discussed in the past. Maybe @gouyang
> can recall what was the explanation for it when it was raised last time.

The last time we fixed the issue is https://bugzilla.redhat.com/show_bug.cgi?id=2090042, which is about move the button close to the text.
We also talked about the button for "Labels" in VM details tab, but it just aligns with OCP, so it's not using the same style like other buttons in VM details tab.
This issue is about removing "Edit" label from the edit button in VM scripts tab, I agree removing it to make them looking consistently.

Comment 5 Yifat Menchik 2023-07-12 08:02:31 UTC
Thank you @gouyang for your explanation. The attachment 1975105 [details]  screen shot is not related to the Scripts tab, so it got me confused.
+1 to remove the "Edit" label from the edit button in VM scripts tab.

Comment 6 Hilda Stastna 2023-07-12 10:26:08 UTC
Hi folks,

I'd like to ask a few questions:
- and what about template's and "Review and create VirtualMachine" Scripts tabs? (see the attachments I've added)
- are we fine with the fact that if we remove "Edit" label and leave the pencil icon there, it will be less visible for the user if the item is editable/not editable?
- are we fine with the fact that in other places with only pencil icon, there's the specific editable value displayed using blue font color, while in Scripts tab achieving the same is problematic?

Thanks in advance!

Comment 9 Yifat Menchik 2023-07-13 09:46:53 UTC
I think @hstastna is right. I agree that if we remove "Edit" label and only have the pencil icon there, it will be less clear for the user to know the item is editable or not editable.

If we keep this logic that in the places with only the pencil icon, we show a specific editable value in blue font color I think it's ok.

WDYT?

Comment 10 Hilda Stastna 2023-07-13 10:05:39 UTC
I'd leave this as it is and close this bug, as I think adding blue font color to the appropriate values and removing "Edit" in Scripts tab would not make the screens look better.
We would have to change the behavior of how we edit each specific value, too, to make it work and be consistent with rest of the Ui. We would have to decide how to display those values,
to have a specific modal for each value etc, which does not seem to me practical at all. Actually, we can change both the username and pwd in one modal. So IMO in these specific screens
it is better to accept inconsistency. But that's just my opinion.

Comment 11 Guohua Ouyang 2023-07-17 03:41:26 UTC
I'm fine if we close the bug as won't fix.


Note You need to log in before you can comment on or make changes to this bug.