Skip to content
This repository has been archived by the owner on Nov 11, 2023. It is now read-only.

Allow HTML in server notices #84

Closed
rugk opened this issue May 13, 2016 · 2 comments
Closed

Allow HTML in server notices #84

rugk opened this issue May 13, 2016 · 2 comments
Labels

Comments

@rugk
Copy link

rugk commented May 13, 2016

What about allowing HTML in server noticed? This would e.g. allow a server admin to add a link "More information" to a page where there is a more detailed explanation instead of the short notice.

@elrido
Copy link
Owner

elrido commented May 14, 2016

I looked shortly into it. Since you enter this message in the configuration, you would need to use i.e. single instead of double quotes (which would then be reverted back in the template rendering) or escape the double quotes. So we would need to change code in the templates (don't escape that string) and the variable assignment (replace single quotes or handle escaping of double quotes).

For now I would suggest to simply create your own template with a custom message. That is simpler and a lot less error prone as you work directly with HTML.

Let me know if this is an OK solution for your use case.

@rugk
Copy link
Author

rugk commented May 14, 2016

Yes, this is okay. My suggestion was also more hypothetical.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants