Skip to content

fix: out of text range while composing error#979

Merged
LucasXu0 merged 1 commit into
AppFlowy-IO:mainfrom
kyungilcho:fix_text_range_error
Dec 30, 2024
Merged

fix: out of text range while composing error#979
LucasXu0 merged 1 commit into
AppFlowy-IO:mainfrom
kyungilcho:fix_text_range_error

Conversation

@kyungilcho

Copy link
Copy Markdown
Contributor

Problem:

Screen_Recording_20241202-230013.mp4

Re-positioning the cursor in the middle of sentence while composing occurs following error.

스크린샷 2024-12-02 오후 11 00 51

so I add logic for checking validity of cursor position.

@codecov

codecov Bot commented Dec 3, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.25%. Comparing base (817c965) to head (c8b245b).
Report is 22 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #979   +/-   ##
=======================================
  Coverage   72.25%   72.25%           
=======================================
  Files         318      318           
  Lines       15132    15134    +2     
=======================================
+ Hits        10933    10935    +2     
  Misses       4199     4199           

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

@LucasXu0

LucasXu0 commented Dec 4, 2024

Copy link
Copy Markdown
Collaborator

@kyungilcho Your PR may fix the composing error, but I think the attached position is still incorrect. The position is empty if the assertion happens.

@LucasXu0 LucasXu0 merged commit 6ce63ec into AppFlowy-IO:main Dec 30, 2024
Hai-Pi pushed a commit to Hai-Pi/appflowy-editor that referenced this pull request Mar 23, 2026
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.

2 participants