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

dev/user-interface#38 Contact Edit: Only display signatures if contact has a CMS account #21103

Merged
merged 1 commit into from
Aug 13, 2021

Conversation

mlutfy
Copy link
Member

@mlutfy mlutfy commented Aug 12, 2021

Overview

When editing a contact, we have an option to add an email signature which is then used when sending emails from CiviCRM (via Contact > Actions > Email).

It looks something like this:

image

Expanding the signature block displays a WYSIWYG and text field to enter an email signature.

This small link is easy to ignore by experienced CiviCRM users, but pretty overwhelming to new CiviCRM users. Not only does expanding explode the UI in a pretty scary way, it's also in an area of CiviCRM that is very overwhelming (location type, On Hold, Mass Mail?, Primary?, Delete).

Before

Email signature is always displayed:

image

After

Email signature is only displayed if the Contact displayed has a CMS account.

ui38-nosig-2021-08-12_09-32

If the contact has a CMS account, the signature works the same as before:

image

Comments

There was Gitlab discussion that the email signature does not really belong here at all (and I agree), but that's a bigger change and requires more reflexion about the concept of user preferences.

@civibot
Copy link

civibot bot commented Aug 12, 2021

(Standard links)

@civibot civibot bot added the master label Aug 12, 2021
@agileware-justin
Copy link
Contributor

agileware-justin commented Aug 13, 2021

👍 works for us - thanks @mlutfy

Rabbit!

@demeritcowboy demeritcowboy merged commit 7a2e612 into civicrm:master Aug 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants