misskey-dev / mfm.js

An MFM parser implementation with TypeScript.

MIT License

112 stars 22 forks source link

Closed anatawa12 closed 3 months ago

anatawa12 commented 3 months ago

What

Fixes #145

これがリリースされてmisskeyで取り込まれたら https://github.com/misskey-dev/misskey/issues/11194 も多分 close 可能

syuilo commented 3 months ago

parseSimpleは非MFMなテキスト(非Misskeyから来るテキストなど)のパース用だからplainに対応するのは違う気がするわね

anatawa12 commented 3 months ago

なるほど

であれば misskey-dev/misskey#11194 の対応はどっちかというとsimpleじゃない方にして、パースしたあとのVNode化のタイミング(MkMfm)で様々な機能(boldやfn?)を無効化する感じのほうがいいですかね

(通知のところはMFMなのでそもそもsimple使ってるのが間違いってことになりますかね)

syuilo commented 3 months ago

parseSimpleは非MFMなテキスト(非Misskeyから来るテキストなど)のパース用だからplainに対応するのは違う気がするわね

あーそんなことないかも？

syuilo commented 3 months ago

使い分けを明確にする必要があるわね

syuilo commented 3 months ago

🙏🏻

anatawa12 commented 3 months ago

レビューで半分拒否されてたのでtypeエラー蟲師してましたがmergeされるのであれば治す必要がありました。直さず放置してて分かりづらくてすみません