Skip to content

Conversation

@syimyuzya
Copy link
Member

Allowed tags: i, b, em, strong. Attributes disallowed.

Allowed tags: i, b, em, strong. Attributes disallowed.
@syimyuzya
Copy link
Member Author

目前只有彈出的說明文字(description)可以用 HTML,選項自身的文字(label)還沒加這個功能。

@graphemecluster
Copy link
Member

我認為 PR 本身是沒有問題的,不過現在推導結果允許的「custom element」有 f(無作用)、biussupsub 以及取一個值的 fgbgsize,而且採用了像 ({ b }) => b`文字` 的語法。日後最好能夠統一一下,或許最好還是採用 JSX,反正 transpile 我猜也是幾十毫秒的事。

dangerouslySetInnerHTML={{
__html: DOMPurify.sanitize(line, { ALLOWED_TAGS: ["i", "b", "em", "strong"], ALLOWED_ATTR: [] }),
}}
key={i}></p>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(是我的話我會用 self-closing tag,不過沒所謂吧 😄)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

哎 JSX 的話確實 (?)

(當時(升級 v0.13 還是 v0.15 時)大概是想着 HTML5 裡 <p> 只能成對(類似 <script> 即使有 src 也得用 </script>)然後寫的😂

@syimyuzya
Copy link
Member Author

f(無作用)

才知道原來 f 用作無作用真的是如此設計的 😂

@syimyuzya
Copy link
Member Author

我認為 PR 本身是沒有問題的,不過現在推導結果允許的「custom element」有 f(無作用)、biussupsub 以及取一個值的 fgbgsize,而且採用了像 ({ b }) => b`文字` 的語法。日後最好能夠統一一下,或許最好還是採用 JSX,反正 transpile 我猜也是幾十毫秒的事。

我也感覺之後要是統一用法會不錯(雖然統一成什麼樣子暫時想象不太出來,不過至少現今的選項說明樣式寫法足夠簡明,之後感覺讓它維持這個簡明易懂程度會比較好)

另外,兩種場合支援的格式也可以不盡相同(比如顏色,感覺就更適合推導結果,不那麼適合選項說明文字)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants