[Enhancement] span TapGestureRecognizer to work with arabic font!

[msdYqb]

Summary

span TapGestureRecognizer is not working with Arabic font
when span with Arabic text is tapped in it its either goes to wrong span or nothing happened

API Changes

example

<Label Grid.Row="0" x:Name="allVerses" FlowDirection="RightToLeft" HorizontalTextAlignment="Center">
            </Label>

FormattedString formattedString = new FormattedString();
            var arabic = new List<myClass>
            {
                new myClass{Text = "بعض الكلمات بعض الكلمات بعض الكلمات بعض الكلمات", ID = 1 },
                new myClass{Text = "بعض الكلمات بعض الكلمات بعض الكلمات بعض الكلمات", ID = 2 },
                new myClass{Text = "بعض الكلمات بعض الكلمات بعض الكلمات بعض الكلمات بعض الكلمات بعض الكلمات", ID = 3 },
                new myClass{Text = "بعض الكلمات بعض الكلمات بعض الكلمات", ID = 4 },
            };
            foreach (var text in arabic)
            {
                Span verse = new Span();
                var gesture = new TapGestureRecognizer();
                gesture.Command = Command;
                gesture.CommandParameter = text.ID;
                verse.GestureRecognizers.Add(gesture);
                
                verse.Text = text.Text + " ";
                verse.FontSize = 44;
                verse.ClassId = text.ID.ToString();
                formattedString.Spans.Add(verse);
            }
            allVerses.FormattedText = formattedString;

public Command Command
        {
            get
            {
                return new Command((ID) =>
                {
                    allVerses.FormattedText.Spans.Where(s => s.ClassId == ID.ToString()).First().TextColor = Color.Red;
                });
            }
        }

Intended Use Case

I want to be able to highlight selected span

[jfversluis]

Thanks for the issue @msdYqb.

I'm wondering if this is an actual enhancement or this is a bug that should just work. I'm leaning towards the last one. But in either case, I see you've posted some code already, would you be able to put it together in a sample project so we can see what is going on?

[hartez]

I agree with @jfversluis, the recognizers should work with RTL.

[hartez]

@msdYqb Are you seeing this problem on Android, iOS, or both?

[msdYqb]

on android and UWP

[msdYqb]

here is example project

thanks guys!
ArabicFormattedStringExample.zip

[msdYqb]

sorry about project name it was to long!

project name fixed
ArabicFS.zip

[StephaneDelcroix]

thanks

[PureWeen]

Looks like this is also an issue on iOS

it's very obvious that the position calculation just needs to be shifted to account for RTL.
When working on this let's look at a general way to express position when set to RTL so other code that needs to calculate location can reuse it

[Sauder91]

I'm seeing this as well on iOS when text wraps. The hashtags in the image below should all be individually tappable, but only the first one works. The same layout with different hash tags sometimes works; its hit or miss. If there is only one it is always selectable though.