Closed Bug 724339 Opened 12 years ago Closed 12 years ago

text selection on mobile devices must support right-to-left (rtl) languages

Categories

(Firefox for Android Graveyard :: Text Selection, enhancement, P1)

ARM
Android
enhancement

Tracking

(firefox15 verified, firefox16 verified, firefox17 fixed, fennec+)

VERIFIED FIXED
Firefox 16
Tracking Status
firefox15 --- verified
firefox16 --- verified
firefox17 --- fixed
fennec + ---

People

(Reporter: amir.aharoni, Assigned: Margaret)

References

Details

Attachments

(1 file)

Text selection doesn't exist on Fennec at this time at all. When it will be done, it must support right-to-left text selection. It works well in Safari on iPhone; it works more or less well on Opera, although the rendering of the selection background color is weird; it doesn't work in Webkit (on 2.3.4; it maybe be better in ICS).
Depends on: text-selection
(In reply to Amir Aharoni from comment #0)
> it doesn't work in Webkit (on 2.3.4; it maybe be better in ICS).
It does work on WebKit (Browser/Android 2.3.7/Cyanogenmod), but it appears to not handle multiline selections correctly. 

As for Firefox, Firefox Beta on the same device does handle Right to left selection good enough.
(In reply to Tomer Cohen :tomer from comment #1)
> (In reply to Amir Aharoni from comment #0)
> > it doesn't work in Webkit (on 2.3.4; it maybe be better in ICS).
> It does work on WebKit (Browser/Android 2.3.7/Cyanogenmod), but it appears
> to not handle multiline selections correctly. 

That's the point of this bug - if it doesn't handle multiline selection of right-to-left text correctly, then right-to-left text selection doesn't really work.
Status: UNCONFIRMED → NEW
Ever confirmed: true
tracking-fennec: --- → +
Priority: -- → P1
Version: Firefox 11 → Trunk
Attached patch patchSplinter Review
(Based on top of my patch for bug 765072)

On an RTL page, the start handle is technically at the end of the selection, and vice versa. Our handle code doesn't need to know about that, but the code that sends mouse events to actually maintain the selection does.

For testing I used: http://people.mozilla.com/~mleibovic/test/lorem_ipsum_rtl.html
Assignee: nobody → margaret.leibovic
Attachment #635937 - Flags: review?(mark.finkle)
Attachment #635937 - Flags: review?(mark.finkle) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/594e394c497e
Target Milestone: --- → Firefox 16
https://hg.mozilla.org/mozilla-central/rev/594e394c497e
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Uplifted to aurora as part of a roll-up patch:
https://hg.mozilla.org/releases/mozilla-aurora/rev/2fb0a358eaf6
Component: General → Text Selection
Status: RESOLVED → VERIFIED
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.