Skip to content
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: TableActionHandler editorState.editable [Bug] (issue (#717) ) #733

Merged

Conversation

Taverz
Copy link
Contributor

@Taverz Taverz commented Mar 4, 2024

fIx: Table Icon visible on hover in Editable and causes exception when clicked [Bug] (issue (#717) )

  • fix: Check visible Widget TableActionHandler when widget.editorState.editable == true

  • Update: lib/src/editor/block_component/table_block_component/table_action_handler.dart

…O#717) )

fIx: Table Icon visible on hover in Editable and causes exception when clicked  [Bug]   (issue (AppFlowy-IO#717) )

* fix:  Check visible Widget TableActionHandler when widget.editorState.editable == true

* Update:  lib/src/editor/block_component/table_block_component/table_action_handler.dart
@CLAassistant
Copy link

CLAassistant commented Mar 4, 2024

CLA assistant check
All committers have signed the CLA.

@Xazin
Copy link
Collaborator

Xazin commented Mar 4, 2024

Please run dart format and commit the change. CI is failing due to it.

…O#717) ) dart format

fIx: Table Icon visible on hover in Editable and causes exception when clicked  [Bug]   (issue (AppFlowy-IO#717) )

* fix:  Check visible Widget TableActionHandler when widget.editorState.editable == true

* Update:  lib/src/editor/block_component/table_block_component/table_action_handler.dart
@Taverz
Copy link
Contributor Author

Taverz commented Mar 5, 2024

I run dart format

Copy link

codecov bot commented Mar 5, 2024

Codecov Report

Attention: Patch coverage is 50.00000% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 75.77%. Comparing base (d4d35c0) to head (9bddb84).

Files Patch % Lines
...nt/table_block_component/table_action_handler.dart 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #733      +/-   ##
==========================================
- Coverage   75.78%   75.77%   -0.01%     
==========================================
  Files         297      297              
  Lines       13750    13751       +1     
==========================================
  Hits        10420    10420              
- Misses       3330     3331       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Taverz
Copy link
Contributor Author

Taverz commented Mar 5, 2024

Сodecov bot said that one line was lost.
I didn't delete it, what's going on?
🫣🥸🤔

@penkzhou
Copy link
Contributor

penkzhou commented Mar 6, 2024

Сodecov bot said that one line was lost. I didn't delete it, what's going on? 🫣🥸🤔

It is not about the file missing, it is about the unit test of this line missing.

@LucasXu0
Copy link
Collaborator

LucasXu0 commented Mar 7, 2024

Never mind. I can merge it.

@LucasXu0 LucasXu0 merged commit 8996fba into AppFlowy-IO:main Mar 7, 2024
8 of 11 checks passed
q200892907 added a commit to q200892907/appflowy-editor that referenced this pull request Mar 12, 2024
* main:
  feat: disable converting heading block to numbered list block (AppFlowy-IO#740)
  fix: xtra space from numbered list item (AppFlowy-IO#724)
  fIx: disable tableActionHandler when the editorState is not editable (AppFlowy-IO#733)
  feat: add fixed toolbar example (AppFlowy-IO#739)
  chore: bump version 2.3.3 (AppFlowy-IO#735)
  fix: transfer the whole line into link when text contains url (AppFlowy-IO#693) (AppFlowy-IO#737)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants