search

justinross / foundry-bitd-alternate-sheets

MIT License
5 stars 11 forks source link

Weird interaction between TinyMCE and Markdown #4

Closed drewg13 closed 2 years ago

drewg13 commented 2 years ago

image

TinyMCE adds <p> tags, perhaps you could use your md helper to strip them out?

justinross commented 2 years ago

Is that a custom ability? I do need to sanitize those strings down to just text. I think I actually had in my old system version.

drewg13 commented 2 years ago

Yes, I just did Add New Ability, instead of Add Existing Ability.

drewg13 commented 2 years ago

One interesting thing to note is that V10 is adding Markdown support as well as changing out TinyMCE for ProseMirror for Journals, but they'll likely be available for use on sheets as well.

justinross commented 2 years ago

Interesting. That'll be a nice addition. Prosemirror will probably still cause the same issue. Basically I just need to strip out HTML tags before rendering the text. Ezpz.

drewg13 commented 2 years ago

Playing with the new ProseMirror and Markdown type sheets right now. They both do add the same <p> tags, but the ProseMirror simultaneous collaborative editing is really slick.