[iOS] Key Commands Refactor

[Velin92]

The issue with the current KeyCommandsHandler, is that even if the key command is set to be false, the command is not really ignored and was stll being handled by the default function, which at the same time was not working properly with plain text mode since it forced an update in the model.

Also the usage of the enter() function was redundant also for the RTE mode, since we already handle the enter directly in the replaceText properly, and we are constraining the usage of the key commands only to the ones we have defined in the library not giving to the users of the library a proper control on which and when they should be and should enabled.

The alternative approach was to disabled the default handling in plain text mode for such commands, but this created two issues:

• The command was still handled by the TextView, which was then preventing the key to use the replace text function in the delegate properly, which means that shift+enter and enter were not doing anything.
• The ignored command was not really ignored at all, but just silently handled in the background, which kinda created a side effect that was not clear to the user of the library

The current solution allows full control and customisation for the key commands while also providing the possibility of creating some default rapid key commands (like .enter) for the general use cases, while also fixing the shift+enter bug in plain text mode.

[sonarqubecloud]

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

[codecov-commenter]

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (91fae40) 87.39% compared to head (200c096) 89.49%.

Files	Patch %	Lines
...mponents/WysiwygComposerView/WysiwygTextView.swift	60.00%	2 Missing ⚠️
...ents/WysiwygComposerView/WysiwygComposerView.swift	83.33%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##               main     #908      +/-   ##
============================================
+ Coverage     87.39%   89.49%   +2.09%     
============================================
  Files           166      118      -48     
  Lines         19027    16906    -2121     
  Branches       1052      616     -436     
============================================
- Hits          16629    15130    -1499     
+ Misses         2096     1752     -344     
+ Partials        302       24     -278     

Flag	Coverage Δ
uitests	78.89% <76.92%> (+7.98%)	⬆️
uitests-android	?
uitests-ios	78.89% <76.92%> (+0.87%)	⬆️
unittests	88.15% <0.00%> (+2.75%)	⬆️
unittests-android	?
unittests-ios	75.54% <0.00%> (+1.61%)	⬆️
unittests-rust	89.42% <ø> (ø)

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.

[pixlwave]

Whilst I'm not familiar with the insides of RTE, this looks like a totally reasonable change to me and makes the API much nicer to use to my eyes.