Closed Bug 891967 Opened 6 years ago Closed 6 years ago

[AccessFu] Show caret in braille when editing

Categories

(Core :: Disability Access APIs, defect)

All
Android
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla25

People

(Reporter: maxli, Assigned: maxli)

Details

Attachments

(1 file, 2 obsolete files)

This is useful.
Attached patch Patch (obsolete) — Splinter Review
Attachment #773988 - Flags: review?(eitan)
Comment on attachment 773988 [details] [diff] [review]
Patch

Review of attachment 773988 [details] [diff] [review]:
-----------------------------------------------------------------

IRL review happened.
Attachment #773988 - Flags: review?(eitan)
Attached patch Patch v2 (obsolete) — Splinter Review
Attachment #773988 - Attachment is obsolete: true
Attachment #774115 - Flags: review?(eitan)
Attached patch Patch v3Splinter Review
Attachment #774115 - Attachment is obsolete: true
Attachment #774115 - Flags: review?(eitan)
Attachment #774188 - Flags: review?(eitan)
Comment on attachment 774188 [details] [diff] [review]
Patch v3

Review of attachment 774188 [details] [diff] [review]:
-----------------------------------------------------------------

This looks good.
Attachment #774188 - Flags: review?(eitan) → review+
Attachment #774188 - Flags: review?(bugmail.mozilla)
Comment on attachment 774188 [details] [diff] [review]
Patch v3

Review of attachment 774188 [details] [diff] [review]:
-----------------------------------------------------------------

Java bits look fine to me.
Attachment #774188 - Flags: review?(bugmail.mozilla) → review+
https://hg.mozilla.org/mozilla-central/rev/e6de1128c616
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
You need to log in before you can comment on or make changes to this bug.