fix(IText): some RTL selection issues

[ShaMan123]

• Renamed relevant left/right methods to start/end (I am quite sure I missed some, logic is quite messy). This will make things clearer.
• Adapted the move cursor logic. It is still far from perfect but is much better.
• Exposed method setSelectionRange to align with html standard and selectionDirection
• deprecated _selectionDirection, didn't remove it yet. It has no affect but I want you to look at it before I remove it to see I am not missing out on anything. selectionDirection is now updated by the textarea on _updateTextarea but that might not be the right way to do it.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[github-actions]

Code Coverage Summary

> fabric@5.1.0 coverage:report
> nyc report --reporter=lcov --reporter=text

-----------|---------|----------|---------|---------|-----------------------------------------------
File       | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s                             
-----------|---------|----------|---------|---------|-----------------------------------------------
All files  |   82.85 |    75.66 |   85.68 |   82.57 |                                               
 fabric.js |   82.85 |    75.66 |   85.68 |   82.57 | ...,30947,31021,31032-31097,31220,31319,31555 
-----------|---------|----------|---------|---------|-----------------------------------------------

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[igalya]

@ShaMan123 hi there, I'm working with RTL direction in textBox, In version 4.6 it has problems with selection when text is aligned to center or right.
I checked 5.2.4 and it seems to be all messed up.
Am I missing something?
Here's an example.
http://jsfiddle.net/Igal_lib/bxm0j2Lv/7/

[ShaMan123]

This PR isn't merged so it is obvious your fiddle is broken.
http://jsfiddle.net/7f916mpj/1/

[igalya]

oh, thanks.
is there a merged version that implements RTL text selection ?

[ShaMan123]

No
The demo uses a branch with a few fixes to text and others.
I am committed to this fix so I hope I will get to migrating it to v6 soon, then merging.
Contributions are welcome

[ShaMan123]

Home End still broken

[igalya]

I wish I could help but I can't at the moment.
Thanks for your prompt replay and best of luck!

[faizux]

@ShaMan123 Many thanks for the RTL update. I successfully integrated fabric-rtl@5.1.0 into my project, and it's working well. However, I encountered some issues after downgrading from fabric@5.3.0, which I was previously using. I wonder if you have a similar implementation for fabric@5.3.0 that can be used? TY

[ShaMan123]

I plan to migrate it to v6.
Hopefully published in 6.1 but can't promise
I recommend all devs to migrate to v6. Too many fixes to miss out

[oustr]

I plan to migrate it to v6. Hopefully published in 6.1 but can't promise I recommend all devs to migrate to v6. Too many fixes to miss out

I still encountered text selection problems when direction is "rtl" and textAlign is "right", that I couldn't select any part of the text, and when I click in the middle of the text the cursor is always at the right. My version of fabricjs is "^6.0.0-beta19", and I wonder if this bug is solved or not as all the rtl issues and discussions I found here on github still confuse me a lot😥 @ShaMan123 Seems the fix is still not merged?