-
Notifications
You must be signed in to change notification settings - Fork 218
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
Ability to add new CSS attributes to the CssSchema DEFINITION #70
Comments
What definitions do you need to add? |
There are definitions related to microsoft specific properties and addons. |
We strip vendor prefixes so any |
we see the following some other schema attributes coming mso-margin-top-alt,mso-add-space, tab-stops, mso-fareast-font-family, mso-bidi-font-family,mso-bidi-theme-font, looking at the code pointed out by you it seems that they are not stripped. |
I want to add "display: flex" in style attribute. |
Any update on this? |
Float can be dangerous via clickjackinf but let’s wait to hear what Mike has to say.
…--
Jim Manico
@manicode
On Jan 31, 2020, at 2:31 PM, Johannesklint ***@***.***> wrote:
Any update on this?
I want to add display and float properties but that is stripped away.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
Even if new HtmlPolicyBuilder()
.allowCommonInlineFormattingElements()
.allowCommonBlockElements()
.allowStyling(
CssSchema.withProperties(
Map.of("display",
new CssSchema.Property(0,
Set.of("none", "flex"),
Collections.emptyMap())))) |
Currently we can only define a whitelist using CssSchema.withProperties method, the properties mentioned in this should be present int the master-set DEFINITIONS mentioned in the class.
In case while configuring we need to add properties to this master-set we can can't as there is not public API exposed to do the same.
The text was updated successfully, but these errors were encountered: