AppFlowy-IO/AppFlowy
1
0
Fork 0
mirror of https://github.com/AppFlowy-IO/AppFlowy.git synced 2026-03-24 04:46:56 +00:00
Code Issues 952 Projects Packages Wiki Activity

[PR #6985] [MERGED] fix: simple table issues #7672

New issue
Closed
opened 2026-03-23 23:20:41 +00:00 by mirror · 0 comments
mirror commented 2026-03-23 23:20:41 +00:00
Owner
Copy link

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/6985
Author: @LucasXu0
Created: 12/13/2024
Status: ✅ Merged
Merged: 12/16/2024
Merged by: @LucasXu0

Base: main ← Head: fix_simple_table_issues

📝 Commits (10+)

  • 77548a9 fix: list padding in table cell is too wide
  • 5f5d9ad feat: improve tab in table cell
  • 1164ab2 feat: improve shift+tab in table cell
  • 63fa9cc fix: unable to edit cell after deleting an image
  • a40f7e6 fix: inline attribute issue
  • 18ed53d fix: disable dragging a block into table
  • 1907c71 feat: add distribute column evenly in column action menu
  • 1bbad8d fix: numbered list icon align in table cell
  • f428d74 feat: add setToPageWidth and distributeColumnEvenly in table menu
  • df88817 feat: support highlight color

📊 Changes

25 files changed (+407 additions, -40 deletions)

View changed files

📝 frontend/appflowy_flutter/integration_test/desktop/document/document_with_image_block_test.dart (+0 -7)
📝 frontend/appflowy_flutter/integration_test/desktop/document/document_with_inline_math_equation_test.dart (+36 -0)
📝 frontend/appflowy_flutter/integration_test/desktop/document/document_with_inline_page_test.dart (+14 -0)
📝 frontend/appflowy_flutter/integration_test/desktop/document/document_with_simple_table_test.dart (+154 -0)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_configuration.dart (+10 -1)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_page.dart (+5 -0)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/actions/block_action_option_cubit.dart (+22 -0)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/actions/drag_to_reorder/draggable_option_button.dart (+1 -0)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/actions/drag_to_reorder/util.dart (+26 -5)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/actions/drag_to_reorder/visual_drag_area.dart (+7 -2)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/actions/option/option_actions.dart (+19 -1)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/numbered_list/numbered_list_icon.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/shortcuts/command_shortcuts.dart (+1 -0)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/simple_table/simple_table_block_component.dart (+7 -0)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/simple_table/simple_table_cell_block_component.dart (+38 -12)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/simple_table/simple_table_more_action.dart (+1 -0)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/simple_table/simple_table_operations/simple_table_map_operation.dart (+10 -0)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/simple_table/simple_table_operations/simple_table_node_extension.dart (+5 -0)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/simple_table/simple_table_operations/simple_table_style_operation.dart (+4 -0)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/simple_table/simple_table_row_block_component.dart (+5 -1)

...and 5 more files

📄 Description

Feature Preview

  • list padding in table cell is too wide (no test required)
  • improve tab shortcut in table cell (test)
  • improve shift+tab shortcut in table cell (test)
  • improve enter shortcut in table cell (test)
  • import select all shortcut in table (test)
  • unable to edit cell after deleting an image (test)
  • inline attribute issue (test)
  • add distribute column evenly in column action menu (test)
  • numbered list icon align in table cell (no test required)
  • add setToPageWidth and distributeColumnEvenly in table menu (test)
  • support highlight color in table (no test required)
  • keep the same column width after setting "distribute columns evenly" (test)

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/6985 **Author:** [@LucasXu0](https://github.com/LucasXu0) **Created:** 12/13/2024 **Status:** ✅ Merged **Merged:** 12/16/2024 **Merged by:** [@LucasXu0](https://github.com/LucasXu0) **Base:** `main` ← **Head:** `fix_simple_table_issues` --- ### 📝 Commits (10+) - [`77548a9`](https://github.com/AppFlowy-IO/AppFlowy/commit/77548a9ed77e7072f60a7aa660f605b4ce513147) fix: list padding in table cell is too wide - [`5f5d9ad`](https://github.com/AppFlowy-IO/AppFlowy/commit/5f5d9ad396ddac936fbe319877001eaa8ee0e6b5) feat: improve tab in table cell - [`1164ab2`](https://github.com/AppFlowy-IO/AppFlowy/commit/1164ab2a8197fa77f1d1ed684a8f7a6f69e0b8f9) feat: improve shift+tab in table cell - [`63fa9cc`](https://github.com/AppFlowy-IO/AppFlowy/commit/63fa9cc6d8089ceab18669a92f8af2043723d3d8) fix: unable to edit cell after deleting an image - [`a40f7e6`](https://github.com/AppFlowy-IO/AppFlowy/commit/a40f7e6d658f128dbab3271b2ca659a2eb58a327) fix: inline attribute issue - [`18ed53d`](https://github.com/AppFlowy-IO/AppFlowy/commit/18ed53d2417ea27be44a008335bb19b971b27438) fix: disable dragging a block into table - [`1907c71`](https://github.com/AppFlowy-IO/AppFlowy/commit/1907c7126b8165ad8227aa2012dc34d422dccdec) feat: add distribute column evenly in column action menu - [`1bbad8d`](https://github.com/AppFlowy-IO/AppFlowy/commit/1bbad8d2122c9d66bea410e88416526c1e56632a) fix: numbered list icon align in table cell - [`f428d74`](https://github.com/AppFlowy-IO/AppFlowy/commit/f428d74d1e626d953ca6e21a324f83b84ee1e3c8) feat: add setToPageWidth and distributeColumnEvenly in table menu - [`df88817`](https://github.com/AppFlowy-IO/AppFlowy/commit/df88817eef6f10e64ac85a8716245e3c7ef600eb) feat: support highlight color ### 📊 Changes **25 files changed** (+407 additions, -40 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/integration_test/desktop/document/document_with_image_block_test.dart` (+0 -7) 📝 `frontend/appflowy_flutter/integration_test/desktop/document/document_with_inline_math_equation_test.dart` (+36 -0) 📝 `frontend/appflowy_flutter/integration_test/desktop/document/document_with_inline_page_test.dart` (+14 -0) 📝 `frontend/appflowy_flutter/integration_test/desktop/document/document_with_simple_table_test.dart` (+154 -0) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_configuration.dart` (+10 -1) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_page.dart` (+5 -0) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/actions/block_action_option_cubit.dart` (+22 -0) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/actions/drag_to_reorder/draggable_option_button.dart` (+1 -0) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/actions/drag_to_reorder/util.dart` (+26 -5) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/actions/drag_to_reorder/visual_drag_area.dart` (+7 -2) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/actions/option/option_actions.dart` (+19 -1) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/numbered_list/numbered_list_icon.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/shortcuts/command_shortcuts.dart` (+1 -0) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/simple_table/simple_table_block_component.dart` (+7 -0) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/simple_table/simple_table_cell_block_component.dart` (+38 -12) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/simple_table/simple_table_more_action.dart` (+1 -0) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/simple_table/simple_table_operations/simple_table_map_operation.dart` (+10 -0) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/simple_table/simple_table_operations/simple_table_node_extension.dart` (+5 -0) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/simple_table/simple_table_operations/simple_table_style_operation.dart` (+4 -0) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/simple_table/simple_table_row_block_component.dart` (+5 -1) _...and 5 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 - [x] list padding in table cell is too wide (no test required) - [x] improve tab shortcut in table cell (test) - [x] improve shift+tab shortcut in table cell (test) - [x] improve enter shortcut in table cell (test) - [x] import select all shortcut in table (test) - [x] unable to edit cell after deleting an image (test) - [x] inline attribute issue (test) - [x] add distribute column evenly in column action menu (test) - [x] numbered list icon align in table cell (no test required) - [x] add setToPageWidth and distributeColumnEvenly in table menu (test) - [x] support highlight color in table (no test required) - [x] keep the same column width after setting "distribute columns evenly" (test) <!--- 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 - [x] My code adheres to [AppFlowy's Conventions](https://docs.appflowy.io/docs/documentation/software-contributions/conventions) - [x] I've listed at least one issue that this PR fixes in the description above. - [x] I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes. - [x] 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 23:20:41 +00:00
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
copilot/fix-date-format-issue
feat/mention_a_person
merge_upstream_v6
mobile/release/0.9.3
release/0.9.3
release/0.9.2
select_source
release/0.9.1
refactor_import_data
release/0.8.9.1
release/0.8.9
anon_user_mode
release/0.8.8
fix_windows_selection
release/0.8.7
annieappflowy-patch-1
windows_local_ai
release/0.8.6
release/0.8.5
release/0.8.4
chore/optional-response-format
release/0.8.3
release/0.8.2
release/0.8.1
release/0.8.0
release/0.7.9
feat/support-rename-workspace
chat_rag_only
calculate_ui_test
release/0.7.8
release/0.7.7
stringfi_cells
hotfix/0.7.5
release/0.7.4
client-api-update
feat/collab-editing
fix_calculation_bug
feat/upload-file-in-document
update-client-api
feat/support-global-comment-on-web
update-client-api-2
feat/support-duplicate-publish-on-web
feat/ai-plan-billing
chore/test-web-deploy
chore/enable-billing-for-testing
build/test
ai_tag_feature
chore/enable-billing-plan
feat/billing-client-windows
chore/support-openai-chat
fix/try-no-media-kit
sign-in-error-format
workspace-invite-ci-test
last-workspace
workspace-invite-wip
feat/date-text-input
appflowy_v1
0.11.1
0.11.0
0.10.9
0.11.2
0.10.7
0.10.6
0.10.5
0.11.3
0.11.4
0.10.8
0.10.4
0.10.3
0.10.2
0.10.0
0.10.1
0.9.9
0.9.8
0.9.7
0.9.5
0.9.6
0.9.4
0.9.3
0.9.2
0.9.1
0.9.0
0.8.9
0.8.8
0.8.7
0.8.6
0.8.5
0.8.4
0.8.3
0.8.2
0.8.1
0.8.0
0.7.9
0.7.8
0.7.7
0.7.6
0.7.5
0.7.4
0.7.3
0.7.2
0.7.1
0.7.0
0.6.9
0.6.8
0.6.7.2
0.6.7.1
0.6.7
0.6.6
0.6.5
0.6.4
0.6.3
0.6.2
0.6.1
v0.6.1
0.6.0
0.5.9
0.5.8
0.5.7
0.5.6
0.5.5
0.5.4
0.5.3
0.5.2
0.5.1
0.5.0
0.4.9
0.4.8
0.4.7
0.4.6
0.4.5
0.4.4
0.4.3
0.4.2
0.4.1
0.4.0
0.3.9.1
0.3.9
0.3.8
0.3.7
0.3.6
0.3.5
0.3.4
0.3.3
0.3.2
0.3.1
0.3.0
0.2.9
0.2.8
0.2.7
0.2.6
0.2.5
0.2.4
0.2.3
0.2.2
0.2.1
0.2.0
0.1.6
0.1.5
0.1.4
0.1.3
0.1.2
0.1.1
0.1.0
0.0.9.1
0.0.9
0.0.8.1
0.0.8
0.0.7.1
0.0.7
0.0.6.2
0.0.6.1
0.0.6
0.0.6_beta.1
0.0.5.3
0.0.5.2
0.0.5.1
0.0.5
0.0.5_beta.2
0.0.5_beta.1
0.0.4
0.0.4_beta.3
0.0.4_beta.2
0.0.4_beta.1
0.0.3
0.0.3_beta.2
0.0.3_beta.1
0.0.2
0.0.1
Labels
Clear labels   
2024

   
2025

   
2026

   
acct mgmt

   
AI

   
automation

   
bug

   
calendar

   
ci

   
CJK

   
cloud

   
code-block

   
collaboration

   
copy-paste

   
database

   
data migration

   
data sync

   
deploy

   
desktop

   
develop

   
develop

   
documentation

   
duplicate

   
editor

   
editor-plugin

   
emoji

   
export

   
files

   
flutter-only

   
follow-up

   
formula

   
good first issue for devs

   
good first issue for experienced devs

   
grid

   
hacktoberfest

   
HACKTOBERFEST-ACCEPTED

   
help wanted

   
i18n

   
icons

   
images

   
importer

   
improvements

   
infra

   
install

   
integrations

   
IR

   
kanban board

   
login

   
look and joy

   
mentorship

   
mobile

   
mobile

   
needs design

   
new feature

   
new feature

   
non-coding

   
notes

   
notifications

   
onboarding

   
organization

   
P0+

   
permission

   
platform-linux

   
platform-mac

   
platform-windows

   
plugins

   
program

   
pull-request

Mirrored from GitHub Pull Request

  
Q1 25

   
Q1 26

   
Q2 24

   
Q2 25

   
Q3 24

   
Q3 25

   
Q4 24

   
Q4 25

   
react

   
regression

   
rust

   
rust

   
Rust-only

   
Rust-only

   
Rust-starter

   
Rust-starter

   
self-hosted

   
shortcuts

   
side panel

   
slash-menu

   
sync v2

   
table

   
tablet

   
task

   
tauri

   
templates

   
tests

   
themes

   
translation

   
v0.5.6

   
v0.5.8

   
v0.5.9

   
v0.6.0

   
v0.6.1

   
v0.6.4

   
v0.6.7

   
v0.6.8

   
v0.7.1

   
v0.7.4

   
v0.7.4

   
v0.7.5

   
v0.7.6

   
v0.7.7

   
v0.7.8

   
v0.8.0

   
v0.8.4

   
v0.8.5

   
v0.8.9

   
web
No labels
2024
2025
2026
acct mgmt
AI
automation
bug
calendar
ci
CJK
cloud
code-block
collaboration
copy-paste
database
data migration
data sync
deploy
desktop
develop
develop
documentation
duplicate
editor
editor-plugin
emoji
export
files
flutter-only
follow-up
formula
good first issue for devs
good first issue for experienced devs
grid
hacktoberfest
HACKTOBERFEST-ACCEPTED
help wanted
i18n
icons
images
importer
improvements
infra
install
integrations
IR
kanban board
login
look and joy
mentorship
mobile
mobile
needs design
new feature
new feature
non-coding
notes
notifications
onboarding
organization
P0+
permission
platform-linux
platform-mac
platform-windows
plugins
program
pull-request
Q1 25
Q1 26
Q2 24
Q2 25
Q3 24
Q3 25
Q4 24
Q4 25
react
regression
rust
rust
Rust-only
Rust-only
Rust-starter
Rust-starter
self-hosted
shortcuts
side panel
slash-menu
sync v2
table
tablet
task
tauri
templates
tests
themes
translation
v0.5.6
v0.5.8
v0.5.9
v0.6.0
v0.6.1
v0.6.4
v0.6.7
v0.6.8
v0.7.1
v0.7.4
v0.7.4
v0.7.5
v0.7.6
v0.7.7
v0.7.8
v0.8.0
v0.8.4
v0.8.5
v0.8.9
web
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
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#7672
No description provided.