[PR #4917] [MERGED] feat: support markdown for heading 4-6 and inline math #6529

Closed
opened 2026-03-23 22:23:37 +00:00 by mirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/4917
Author: @qinluhe
Created: 3/15/2024
Status: Merged
Merged: 3/18/2024
Merged by: @qinluhe

Base: mainHead: fix/tauri-bugs-0315


📝 Commits (2)

  • 3ff2186 feat: support-OAuth-login
  • d026b35 fix: optimize editor experience and fix bugs (0315)

📊 Changes

48 files changed (+746 additions, -509 deletions)

View changed files

📝 frontend/appflowy_tauri/src/appflowy_app/components/_shared/view_title/ViewBanner.tsx (+1 -1)
📝 frontend/appflowy_tauri/src/appflowy_app/components/_shared/view_title/ViewIconGroup.tsx (+3 -3)
📝 frontend/appflowy_tauri/src/appflowy_app/components/editor/command/index.ts (+68 -18)
📝 frontend/appflowy_tauri/src/appflowy_app/components/editor/components/blocks/_shared/PlaceholderContent.tsx (+5 -5)
📝 frontend/appflowy_tauri/src/appflowy_app/components/editor/components/blocks/code/SelectLanguage.tsx (+6 -0)
📝 frontend/appflowy_tauri/src/appflowy_app/components/editor/components/blocks/database/DatabaseList.tsx (+1 -1)
📝 frontend/appflowy_tauri/src/appflowy_app/components/editor/components/blocks/divider/DividerNode.tsx (+1 -1)
📝 frontend/appflowy_tauri/src/appflowy_app/components/editor/components/blocks/image/ImageBlock.tsx (+2 -2)
📝 frontend/appflowy_tauri/src/appflowy_app/components/editor/components/blocks/image/ImageRender.tsx (+1 -1)
📝 frontend/appflowy_tauri/src/appflowy_app/components/editor/components/blocks/text/StartIcon.hooks.tsx (+1 -1)
📝 frontend/appflowy_tauri/src/appflowy_app/components/editor/components/blocks/text/Text.tsx (+3 -3)
📝 frontend/appflowy_tauri/src/appflowy_app/components/editor/components/blocks/todo_list/CheckboxIcon.tsx (+21 -4)
📝 frontend/appflowy_tauri/src/appflowy_app/components/editor/components/blocks/todo_list/TodoList.tsx (+1 -1)
📝 frontend/appflowy_tauri/src/appflowy_app/components/editor/components/blocks/toggle_list/ToggleList.tsx (+2 -2)
📝 frontend/appflowy_tauri/src/appflowy_app/components/editor/components/editor/Editor.hooks.ts (+2 -2)
📝 frontend/appflowy_tauri/src/appflowy_app/components/editor/components/inline_nodes/link/Link.tsx (+1 -1)
📝 frontend/appflowy_tauri/src/appflowy_app/components/editor/components/inline_nodes/link/LinkEditContent.tsx (+3 -3)
📝 frontend/appflowy_tauri/src/appflowy_app/components/editor/components/inline_nodes/link/LinkEditInput.tsx (+2 -2)
📝 frontend/appflowy_tauri/src/appflowy_app/components/editor/components/inline_nodes/link/LinkEditPopover.tsx (+1 -1)
📝 frontend/appflowy_tauri/src/appflowy_app/components/editor/components/inline_nodes/mention/MentionLeaf.tsx (+1 -1)

...and 28 more files

📄 Description

Feature Preview


Bugs Checklist

  • Cannot create an inline math equation. Markdown syntax: $
  • Heading 4 - 6 are not supported; should be supported by markdown
  • Quickly double tapping word and then using CMD+K to bring up link menu, has the selection menu open in a wrong offset
  • Should deleting something from trash (permanently) require an extra step?
  • Selecting all of document for copy+paste, it also copies the “language” of a code block in raw text, should it be skipped from selection?
  • Selection gets stuck if I drag to select, stop at a link and as I lift my mouse I immediately click on the link (see recording)
  • Empty nodes sometimes bug out not showing the block menu / drag handle (see recording for repro)
  • Selecting multiple checkboxes and using CMD+ENTER doesn’t toggle the selected ones
  • SHIFT+PRESS on a checkbox should toggle it and it’s children (See how it works in the Flutter version)
  • Hovering the Page title at the top bar my cursor looks like I can select it, but I can’t
  • Cannot type the text “- “. After converting to a bullet item, pressing backspace should make it return to “- “,Same as above for Numbered lists. I cannot type the text “1. “
  • Heading should be able to co-exist. https://github.com/AppFlowy-IO/AppFlowy/issues/2801
  • quote (trigger by shortcut) → type hello → select all the text → select checkbox → document crash. After crash, can’t open other documents
  • the divider is not aligned with the beginning of a line
  • the document of restore all trash is wrong
  • Cannot select more than one checklist items and delete at the same time. There seems to be multiple problems here. Sometimes it works, sometimes it doesn’t work, sometimes it crash.
  • Clicking Shift+Enter on an empty checklist item breaks that item.
  • in the Quote block, selecting the second line triggers an incomplete toolbar while the first line pops up a complete toolbar
  • Pressing “Enter” at the start of Header(## my header), will make the block lose header.
  • long text in a checklist item doesn’t wrap into the second line (same issue with numbered and bulleted lists
  • Should strip space when the title is just a space
  • click the bulleted list button in floating button again should cancel the format.

PR Checklist

  • My code adheres to AppFlowy's Conventions
  • I've listed at least one issue that this PR fixes in the description above.
  • I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes.
  • All existing tests are passing.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/AppFlowy-IO/AppFlowy/pull/4917 **Author:** [@qinluhe](https://github.com/qinluhe) **Created:** 3/15/2024 **Status:** ✅ Merged **Merged:** 3/18/2024 **Merged by:** [@qinluhe](https://github.com/qinluhe) **Base:** `main` ← **Head:** `fix/tauri-bugs-0315` --- ### 📝 Commits (2) - [`3ff2186`](https://github.com/AppFlowy-IO/AppFlowy/commit/3ff21863e47cde51899691331d9b1ee35df1d912) feat: support-OAuth-login - [`d026b35`](https://github.com/AppFlowy-IO/AppFlowy/commit/d026b35d5f44d48e7876b2b9ff6475e1357cd6e4) fix: optimize editor experience and fix bugs (0315) ### 📊 Changes **48 files changed** (+746 additions, -509 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_tauri/src/appflowy_app/components/_shared/view_title/ViewBanner.tsx` (+1 -1) 📝 `frontend/appflowy_tauri/src/appflowy_app/components/_shared/view_title/ViewIconGroup.tsx` (+3 -3) 📝 `frontend/appflowy_tauri/src/appflowy_app/components/editor/command/index.ts` (+68 -18) 📝 `frontend/appflowy_tauri/src/appflowy_app/components/editor/components/blocks/_shared/PlaceholderContent.tsx` (+5 -5) 📝 `frontend/appflowy_tauri/src/appflowy_app/components/editor/components/blocks/code/SelectLanguage.tsx` (+6 -0) 📝 `frontend/appflowy_tauri/src/appflowy_app/components/editor/components/blocks/database/DatabaseList.tsx` (+1 -1) 📝 `frontend/appflowy_tauri/src/appflowy_app/components/editor/components/blocks/divider/DividerNode.tsx` (+1 -1) 📝 `frontend/appflowy_tauri/src/appflowy_app/components/editor/components/blocks/image/ImageBlock.tsx` (+2 -2) 📝 `frontend/appflowy_tauri/src/appflowy_app/components/editor/components/blocks/image/ImageRender.tsx` (+1 -1) 📝 `frontend/appflowy_tauri/src/appflowy_app/components/editor/components/blocks/text/StartIcon.hooks.tsx` (+1 -1) 📝 `frontend/appflowy_tauri/src/appflowy_app/components/editor/components/blocks/text/Text.tsx` (+3 -3) 📝 `frontend/appflowy_tauri/src/appflowy_app/components/editor/components/blocks/todo_list/CheckboxIcon.tsx` (+21 -4) 📝 `frontend/appflowy_tauri/src/appflowy_app/components/editor/components/blocks/todo_list/TodoList.tsx` (+1 -1) 📝 `frontend/appflowy_tauri/src/appflowy_app/components/editor/components/blocks/toggle_list/ToggleList.tsx` (+2 -2) 📝 `frontend/appflowy_tauri/src/appflowy_app/components/editor/components/editor/Editor.hooks.ts` (+2 -2) 📝 `frontend/appflowy_tauri/src/appflowy_app/components/editor/components/inline_nodes/link/Link.tsx` (+1 -1) 📝 `frontend/appflowy_tauri/src/appflowy_app/components/editor/components/inline_nodes/link/LinkEditContent.tsx` (+3 -3) 📝 `frontend/appflowy_tauri/src/appflowy_app/components/editor/components/inline_nodes/link/LinkEditInput.tsx` (+2 -2) 📝 `frontend/appflowy_tauri/src/appflowy_app/components/editor/components/inline_nodes/link/LinkEditPopover.tsx` (+1 -1) 📝 `frontend/appflowy_tauri/src/appflowy_app/components/editor/components/inline_nodes/mention/MentionLeaf.tsx` (+1 -1) _...and 28 more files_ </details> ### 📄 Description <!--- Thank you for submitting a pull request to AppFlowy. The team will dedicate their best efforts to reviewing and approving your pull request. If you have any questions about the project or feedback for us, please join our [Discord](https://discord.gg/wdjWUXXhtw). --> <!--- If your pull request adds a new feature, please drag and drop a video into this section to showcase what you've done! If not, you may delete this section. --> ### Feature Preview --- #### Bugs Checklist - [x] Cannot create an inline math equation. Markdown syntax: $ - [x] Heading 4 - 6 are not supported; should be supported by markdown - [x] Quickly double tapping word and then using CMD+K to bring up link menu, has the selection menu open in a wrong offset - [x] Should deleting something from trash (permanently) require an extra step? - [x] Selecting all of document for copy+paste, it also copies the “language” of a code block in raw text, should it be skipped from selection? - [x] Selection gets stuck if I drag to select, stop at a link and as I lift my mouse I immediately click on the link (see recording) - [x] Empty nodes sometimes bug out not showing the block menu / drag handle (see recording for repro) - [x] Selecting multiple checkboxes and using CMD+ENTER doesn’t toggle the selected ones - [x] SHIFT+PRESS on a checkbox should toggle it and it’s children (See how it works in the Flutter version) - [x] Hovering the Page title at the top bar my cursor looks like I can select it, but I can’t - [x] Cannot type the text “- “. After converting to a bullet item, pressing backspace should make it return to “- “,Same as above for Numbered lists. I cannot type the text “1. “ - [x] Heading should be able to co-exist. https://github.com/AppFlowy-IO/AppFlowy/issues/2801 - [x] quote (trigger by shortcut) → type hello → select all the text → select checkbox → document crash. After crash, can’t open other documents - [x] the divider is not aligned with the beginning of a line - [x] the document of restore all trash is wrong - [x] Cannot select more than one checklist items and delete at the same time. There seems to be multiple problems here. Sometimes it works, sometimes it doesn’t work, sometimes it crash. - [x] Clicking Shift+Enter on an empty checklist item breaks that item. - [x] in the Quote block, selecting the second line triggers an incomplete toolbar while the first line pops up a complete toolbar - [x] Pressing “Enter” at the start of Header(## my header), will make the block lose header. - [x] long text in a checklist item doesn’t wrap into the second line (same issue with numbered and bulleted lists - [x] Should strip space when the title is just a space - [x] click the bulleted list button in floating button again should cancel the format. <!--- List at least one issue here that this PR addresses. If it fixes the issue, please use the [fixes](https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/using-keywords-in-issues-and-pull-requests) keyword to close the issue. For example: fixes https://github.com/AppFlowy-IO/AppFlowy/pull/2106 --> --- <!--- Before you mark this PR ready for review, run through this checklist! --> #### PR Checklist - [ ] My code adheres to [AppFlowy's Conventions](https://docs.appflowy.io/docs/documentation/software-contributions/conventions) - [ ] I've listed at least one issue that this PR fixes in the description above. - [ ] I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes. - [ ] All existing tests are passing. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
mirror 2026-03-23 22:23:37 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
AppFlowy-IO/AppFlowy#6529
No description provided.