-
-
Notifications
You must be signed in to change notification settings - Fork 825
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
Remove discussion about format from html box on email form #27734
Conversation
🤖 Thank you for contributing to CiviCRM! ❤️ We will need to test and review this PR. 👷 Introduction for new contributors...
Quick links for reviewers...
|
8261fb3
to
e5953d3
Compare
Instead of "potentially obsolete":
What happens if the WYSIWYG is disabled? |
e5953d3
to
fd22229
Compare
@mlutfy to the last - the html box is still the place to configure the message - you just have to write it in html I guess? I don't know who disables WYSIWIG or why - but I assume they still have to ensure that any fields that expect html have entered html? Re wording - I updated to this One more note - I thought about having the accordian closed rather than open - but then I figured that should only be the case if empty - which seems fiddly & makes me think we might be better to focus on the Message Admin UI instead. Hmm maybe that's a case for closing by default - ie it's used from more than one screen & the main one where you might want to encourage them to empty it out is kinda deprecated. |
ah yeah, good point about the HTML/WYSIWYG. I think I only have one client who disables the editor, because they didn't really need it, and had some security concerns. Even one client who has a massively old-school kind of membership (with text email) still generate HTML by default (because the staff aren't that old school). Changes look good to me. thanks! |
@mlutfy they still generate html by default? I'm confused by what you mean? |
You can ignore my comment. I was trying to think of various edge cases, and in both cases, working with HTML-only is fine (with auto-generated text version). |
@mlutfy ok to merge? We can iterate if required |
Overview
Remove discussion about format from html box on email form
Before
After
Technical Details
Comments
I really can't think of anything else useful to say here - but this gets rid of the actively bad text. We could do more on the plain text bit (eg. remove it!) but out of scope for this PR