feat: delete "sentence" shortcut

[Xazin]

Closes: #4

Need to check if the commands are properly setup for Linux and Windows!

[codecov]

Codecov Report

Merging #32 (f6b5597) into main (6e72e58) will increase coverage by 0.97%.
The diff coverage is 91.66%.

@@            Coverage Diff             @@
##             main      #32      +/-   ##
==========================================
+ Coverage   81.68%   82.65%   +0.97%     
==========================================
  Files         124      124              
  Lines        7178     7184       +6     
==========================================
+ Hits         5863     5938      +75     
+ Misses       1315     1246      -69     

Impacted Files	Coverage Δ
lib/src/core/document/text_delta.dart	95.66% <ø> (ø)
...nternal_key_event_handlers/arrow_keys_handler.dart	98.32% <91.30%> (+17.66%)	⬆️
...rvice/shortcut_event/built_in_shortcut_events.dart	98.16% <100.00%> (+0.03%)	⬆️

... and 11 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[LucasXu0]

Hey, @Xazin. Can you add more tests for the missing cover line?

[Xazin]

I will look at it tonight 👍

[LucasXu0]

There are some lines of code that don't have test coverage. I think it would be helpful to have test coverage for all of the lines. Would you be willing to work on that?

[Xazin]

There are some lines of code that don't have test coverage. I think it would be helpful to have test coverage for all of the lines. Would you be willing to work on that?

Yes for sure.

I ended up busy this weekend with another project. But I'm back looking at this, will make things a bit prettier and improve test coverage for all of arrow_keys_handler.dart.

[Xazin]

@LucasXu0 Ready for review

Made some changes to the arrow keys handler file, but the logic is and should be the same.

The file should have 98% coverage now, few lines not tested, not worth the effort imo atm.

[LucasXu0]

LGTM.