Skip to content
This repository has been archived by the owner on Sep 27, 2024. It is now read-only.

Fix enter right after mention crashing the editor #904

Merged
merged 1 commit into from
Dec 13, 2023

Conversation

jmartinesp
Copy link
Contributor

Should fix #903 .

@jmartinesp jmartinesp requested a review from a team December 13, 2023 08:55
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a87e5e6) 87.37% compared to head (17c9eff) 89.42%.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #904      +/-   ##
============================================
+ Coverage     87.37%   89.42%   +2.05%     
============================================
  Files           166       86      -80     
  Lines         18999    15352    -3647     
  Branches       1052        0    -1052     
============================================
- Hits          16600    13729    -2871     
+ Misses         2097     1623     -474     
+ Partials        302        0     -302     
Flag Coverage Δ
uitests ?
uitests-android ?
uitests-ios ?
unittests 89.42% <100.00%> (+4.04%) ⬆️
unittests-android ?
unittests-ios ?
unittests-rust 89.42% <100.00%> (+0.03%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

Copy link

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

Copy link
Contributor

@bmarty bmarty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. I do not understand everything, but I guess you fixed the issue.
I am wondering if we would need some Rust dev review for such PR (and not a review from https://github.com/orgs/matrix-org/teams/element-android).

@jmartinesp jmartinesp requested a review from Velin92 December 13, 2023 10:23
@jmartinesp jmartinesp merged commit 91fae40 into main Dec 13, 2023
7 checks passed
@jmartinesp jmartinesp deleted the fix-enter-after-mention branch December 13, 2023 12:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Crash when adding a new line right after a mention
4 participants