-
-
Notifications
You must be signed in to change notification settings - Fork 538
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(sheets): refactor mergecell generating logic #1636
Conversation
View Deployment
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## dev #1636 +/- ##
==========================================
+ Coverage 29.65% 29.79% +0.14%
==========================================
Files 1138 1141 +3
Lines 61858 62029 +171
Branches 12939 13019 +80
==========================================
+ Hits 18343 18482 +139
- Misses 43515 43547 +32 ☔ View full report in Codecov by Sentry. |
2c8ec7c
to
2ec1e6a
Compare
26f9d3a
to
06df474
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good work! Some problems need to be addressed to improve code quality.
packages/sheets-ui/src/commands/commands/add-worksheet-merge.command.ts
Outdated
Show resolved
Hide resolved
packages/sheets/src/controllers/insert-or-remove-row-col.controller.ts
Outdated
Show resolved
Hide resolved
packages/sheets/src/controllers/insert-or-remove-row-col.controller.ts
Outdated
Show resolved
Hide resolved
c8945c5
to
92eacca
Compare
Origin Title: fix(sheets): refactor mergecell generating logic Title: fix(sheets): refactor mergecell generating logic 需要回归合并单元格关于增删行列 移动行列的交叉场景 |
6bf8e96
to
92eacca
Compare
3ab64fa
to
5dc1bb9
Compare
需要回归合并单元格关于增删行列 移动行列的交叉场景