Skip to content

Not able to configure a pure HTML signature without getting it defaced by the editor. #4557

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

Open
stevearbour opened this issue Feb 18, 2021 · 13 comments · May be fixed by #11125
Open

Not able to configure a pure HTML signature without getting it defaced by the editor. #4557

stevearbour opened this issue Feb 18, 2021 · 13 comments · May be fixed by #11125

Comments

@stevearbour
Copy link

Expected behavior

I should be able to have a radio box like this :
image

Actual behavior

The users are not able to set their email signature properly using optimized and properly formatted HTML. When they paste it inside there no option to not get their signature defaced by the CKEditor. Even by using javascript hack to fill the ckinstance end-up getting filtered. I can understand that you don't want to have PHP code and stuff in there, but sanitizing html markup has been something quite documented by now, we should be able to use a standard set of html markup tags, without being bothered by the editor. It should be concat with the message body. This is actually important because there no way any users want to be limited by the CKEditor for generating a signature and moving them from any others mail client such as thunderbird, it quite clear that this is a required (not optional but absolutely necessary) thing to fix this issues. Otherwise the employees prefer to continue to use desktop version of thunderbird and have the proper business image sent to customers when they send out email, not just deficient RTF signature.

Mail app

Mail app version: 1.8.2

Mailserver: Unrelated, Exim.

Server configuration

Operating system: CentOS

Web server: Apache

Database: MariaDB

PHP version: 7.4

Nextcloud Version: 20.0.7

Client configuration

Browser: Chrome, latest.
Operating system: Mac, Windows and Linux, it all the same problem.

@ChristophWurst

This comment has been minimized.

@stevearbour

This comment has been minimized.

@ChristophWurst

This comment has been minimized.

@stevearbour

This comment has been minimized.

@ChristophWurst

This comment has been minimized.

@lhoff94
Copy link

lhoff94 commented Jul 7, 2021

I want to second that request.
That is the single reason why we are still running a roundcube instance at my association. We have standardized signature and even if it would be possible to recreate them with the editor, it would be a pain to roll it out.

@ChristophWurst

This comment has been minimized.

@sascha08-15
Copy link

I have a related (potentially feature request): I want to add an HTML signature. I see, Nextcloud already supports some, but not all of it, e.g. including an embedded does not work. Should I open a new issue for that? Thanks!

@ChristophWurst ChristophWurst moved this from 🧭 Planning evaluation to 📄 To do in 💌 📅 👥 Groupware team Mar 6, 2025
@jancborchardt jancborchardt moved this to 🏗️ At engineering in 🖍 Design team Mar 6, 2025
@ChristophWurst
Copy link
Member

We (Nextcloud GmbH) will work on this as the bug report with the highest upvotes.

@codingPotato21

This comment has been minimized.

@sascha08-15

This comment has been minimized.

@PeopleInside

This comment has been minimized.

@ChristophWurst

This comment has been minimized.

@GretaD GretaD linked a pull request May 9, 2025 that will close this issue
@ChristophWurst ChristophWurst moved this to 🏗️ In progress in 💌 📅 👥 Groupware team May 9, 2025
@ChristophWurst ChristophWurst moved this from 🏗️ In progress to 📄 To do in 💌 📅 👥 Groupware team May 15, 2025
@ChristophWurst ChristophWurst moved this from 📄 To do to 🏗️ In progress in 💌 📅 👥 Groupware team May 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 🏗️ In progress
Status: 🏗️ At engineering
Development

Successfully merging a pull request may close this issue.

7 participants