-
-
Notifications
You must be signed in to change notification settings - Fork 469
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(sheet): editor change sheet #1241
Conversation
View Deployment
|
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## dev #1241 +/- ##
==========================================
- Coverage 29.63% 29.62% -0.01%
==========================================
Files 829 829
Lines 46860 46869 +9
Branches 9590 9592 +2
==========================================
Hits 13885 13885
- Misses 28761 28768 +7
- Partials 4214 4216 +2 ☔ View full report in Codecov by Sentry. |
@DR-Univer 未退出编辑态切换子表直接编辑,单元格位置还不对,文字无法输入 |
* fix: unexpected autofill popup menu disapear * fix: unexpected scroll after move end. close#1167
* chore: fix lint * chore: optimize actions
…m/univer into fix-editor-change-sheet
|
@zhaolixin7 这个问题和禁用菜单的问题相关,我一起解了,可以一起看看 |
@zhaolixin7 This problem is related to the problem of disabling the menu. I solved it together. You can take a look together. |
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.
带有 isAutoHeight 重构的代码?
@@ -1330,8 +1330,8 @@ export class SpreadsheetSkeleton extends Skeleton { | |||
if (!rowDataItem) { | |||
continue; | |||
} | |||
const { h = defaultRowHeight, ah, isAutoHeight } = rowDataItem; | |||
if ((isAutoHeight == null || !!isAutoHeight) && typeof ah === 'number') { | |||
const { h = defaultRowHeight, ah, ia } = rowDataItem; |
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.
这是我漏改了吗😅
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.
这个分支废弃吧 哲能帮我弄了
dev
branch.close #991
close #1054