From a18ad132bebf3cc92d0432ef18075ec0be9fd56b Mon Sep 17 00:00:00 2001 From: naskya Date: Thu, 16 May 2024 04:49:49 +0900 Subject: [PATCH] fix: remove $[center] MFM function --- docs/changelog.md | 2 ++ packages/client/src/components/mfm.ts | 11 ----------- packages/client/src/pages/mfm-cheat-sheet.vue | 4 +--- packages/client/src/scripts/mfm-tags.ts | 1 - 4 files changed, 3 insertions(+), 15 deletions(-) diff --git a/docs/changelog.md b/docs/changelog.md index 7db20ed2f2..69d58053f0 100644 --- a/docs/changelog.md +++ b/docs/changelog.md @@ -8,6 +8,8 @@ Critical security updates are indicated by the :warning: icon. ## Unreleased - Improve timeline UX +- Remove `$[center]` MFM function + - This function was suddenly added last year (https://firefish.dev/firefish/firefish/-/commit/1a971efa689323d54eebb4d3646e102fb4d1d95a), but according to the [MFM spec](https://github.com/misskey-dev/mfm.js/blob/develop/docs/syntax.md#fn), `$[something]` must be an inline element (while `center` is a block element), so such a syntax is not expected by MFM renderers. Please use `
` instead. - Fix bugs ## [v20240504](https://firefish.dev/firefish/firefish/-/merge_requests/10790/commits) diff --git a/packages/client/src/components/mfm.ts b/packages/client/src/components/mfm.ts index bdba3c0cbe..87f3f6d8a3 100644 --- a/packages/client/src/components/mfm.ts +++ b/packages/client/src/components/mfm.ts @@ -349,17 +349,6 @@ export default defineComponent({ ), ]; } - case "center": { - return [ - h( - "div", - { - style: "text-align: center;", - }, - genEl(token.children), - ), - ]; - } } if (style == null) { return [ diff --git a/packages/client/src/pages/mfm-cheat-sheet.vue b/packages/client/src/pages/mfm-cheat-sheet.vue index e5e858ea65..a0335e0bab 100644 --- a/packages/client/src/pages/mfm-cheat-sheet.vue +++ b/packages/client/src/pages/mfm-cheat-sheet.vue @@ -464,9 +464,7 @@ const preview_bold = ref(`**${i18n.ts._mfm.dummy}**`); const preview_small = ref( `${i18n.ts._mfm.dummy} $[small ${i18n.ts._mfm.dummy}]`, ); -const preview_center = ref( - `
${i18n.ts._mfm.dummy}
\n$[center ${i18n.ts._mfm.dummy}]`, -); +const preview_center = ref(`
${i18n.ts._mfm.dummy}
`); const preview_inlineCode = ref('`<: "Hello, world!"`'); const preview_blockCode = ref( '```\n~ (#i, 100) {\n\t<: ? ((i % 15) = 0) "FizzBuzz"\n\t\t.? ((i % 3) = 0) "Fizz"\n\t\t.? ((i % 5) = 0) "Buzz"\n\t\t. i\n}\n```', diff --git a/packages/client/src/scripts/mfm-tags.ts b/packages/client/src/scripts/mfm-tags.ts index 418bed00b3..ebb12f4e84 100644 --- a/packages/client/src/scripts/mfm-tags.ts +++ b/packages/client/src/scripts/mfm-tags.ts @@ -22,5 +22,4 @@ export const MFM_TAGS = [ "rotate", "fade", "small", - "center", ];