icewind1991 / files_markdown

Nextcloud markdown editor
GNU Affero General Public License v3.0
293 stars 28 forks source link

Using html tags #72

Open thesteff opened 7 years ago

thesteff commented 7 years ago

Hi !

I've tried to test the example code from your home page project but the html tags aren't previewed correctly. When I code this part : <h2 id="overview">Overview</h2> The preview panel shows <h2 id="overview">Overview</h2>

Is there a thing to config to make the preview panel interpret html or something else ? Thanks !

CasN commented 6 years ago

Having a similar issue. I wanted to added a HTML link to open a website in a new window. Standard not supported by Markdown hence regular HTML. This does not work, i am routed to a non-existing page. Anything missing in my installation or something to configure or still a bug?

cgrima commented 6 years ago

Same as @thesteff. html tags are interpreted in plain text in the preview panel.

I am on nextcloud 12.0.3 (from wonderfall/nextcloud docker image) with Markdown Editor 2.0.1

clorenz150 commented 6 years ago

All html tags are displayed instead of interpreted. Versions are Nextcloud 13.0.0 and files_markdown 2.0.1

gbcreation commented 6 years ago

Here is a quick and dirty workaround to enable HTML tags in Notes v2.3.2.

WARNING: USE AT YOUR OWN RISK! Do it ONLY if you understand what you do. This is your decision to apply this workaround, not mine. I cannot be held responsible for any damage to your NextCloud installation.

I successfully tested it on my NextCloud 13 installation running on ArchLinux.

Note that enabling HTML tags in markdown could be a security risk. Note also that this change will probably be lost as soon as you will update your Notes app to a new version.

EDIT: typo

Schmuuu commented 6 years ago

I'm glad @gbcreation posted a workaround, but I would really love to have this feature in the app without my intervention after every update.

Is there an ETA for this feature? I'm really missing that.

Just for now: @gbcreation is npm install absolutely necessary or is there another way without it?

simhnna commented 6 years ago

I'm not sure if I really want to have that feature. You can pass code that gets executed in so many ways and it's hard to catch them all. If this is provided, I think it should be an option that defaults to false.

Cancan68 commented 5 years ago

Would be nice to have this option as a parameter

en3r0 commented 4 years ago

I think this feature is still broken? It is still listed as a feature this provides though.