search

melMass / comfy_mtb

Animation oriented nodes pack for ComfyUI
MIT License
426 stars 48 forks source link

[bug] Note+ strips html attributes? #198

Open fuzhao119 opened 1 month ago

fuzhao119 commented 1 month ago

Describe the bug

The target attribute in a <a> link is stripped

Reproduction

This:

<a href="https://xxx" target="_blank"></a>.

Is turned into this on save of Note+:

<a href="https://xxx"></a>.

Expected behavior

Not stripping attributes

Operating System

Windows (Default)

Comfy Mode

Comfy Portable (embed) (Default)

Console output

No response

Additional context

No response

fuzhao119 commented 1 month ago

After clicking the SAVE button, changes to

melMass commented 1 month ago

I think you forgot something. I have no idea what you are referring to

melMass commented 1 month ago

I guessed what you meant and rewrote the issue

melMass commented 1 month ago

This is true for the HTML mode, I will check what makes sense to allow. For markdown, _blank is the default, so this would do what you want:

## Note+
Go here [site](https://www.xxx.com)

Markdown is the preferred mode as it should be a superset of HTML in Note+

fuzhao119 commented 4 weeks ago

My purpose is to open a window in a new tab, rather than refreshing the current comfyui page

fuzhao119 commented 4 weeks ago

test

fuzhao119 commented 4 weeks ago

@melMass