Skip to content
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

New/Edit Contact: remove the Cancel button #23994

Closed
wants to merge 1 commit into from

Conversation

mlutfy
Copy link
Member

@mlutfy mlutfy commented Jul 13, 2022

Overview

When creating a new Contact, or editing a contact, do not display a Cancel button, as it is unlikely to be used for its intended purpose.

Before

image

After

image

Comments

  • Using the Cancel button is rather unlikely. Users are more likely to hit back or use the navigation menu.
  • If they do hit the Cancel button, it's probably by accident, which can be pretty frustrating.

@civibot
Copy link

civibot bot commented Jul 13, 2022

(Standard links)

@civibot civibot bot added the master label Jul 13, 2022
@mattwire
Copy link
Contributor

Fine by me. Does this work ok without cancel button on popup forms?

@yashodha
Copy link
Contributor

@mlutfy
Does this have a related ticket for this? Are there more screens that need to be consistent with this behavior? There is cancel button on almost all the forms there are.

@mlutfy
Copy link
Member Author

mlutfy commented Jul 14, 2022

I incorrectly mentioned that this impacts only new Contacts, but it also impacts "Edit Contact". I still think it's fine, because clicking cancel after doing an edit would still be cringe. Updated description.

@mattwire re:popups, I guess New Contact is very rarely used in a popup, but New Case is (and is most often used in popups). So I reverted the change on New Case.

image

Technically, however, it does not make the popup bug if the cancel button is absent (I tested).

@yashodha I don't have a Gitlab issue. So far I removed Cancel from here: #23696, but that was an obvious candidate. I did grep for other occurrences of cancel, but I prefer to only PR when I see a clear reason to remove it. In admin screens, for example, it can be more debatable.

I get a lot of user feedback about the 1) excessive number of buttons 2) how easy it is to click the wrong one, 3) incorrect labels (often made worse by confused translators). However, I understand that UI changes can be very disruptive (see also #23831)

@mlutfy mlutfy changed the title New Contact/Case: remove the Cancel button New/Edit Contact: remove the Cancel button Jul 14, 2022
@mlutfy
Copy link
Member Author

mlutfy commented Sep 20, 2022

I'm fine with closing this. I was thinking of maybe creating an extension "unclutter" or something that could be opt-in (https://chat.civicrm.org/civicrm/pl/oi9aehbi77n7jm46gjo999ajgc)

@mlutfy mlutfy closed this Sep 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants