Rename drawSystemFocusRing to drawFocusIfNeeded

RESOLVED FIXED in mozilla29

Status

()

defect
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: cabanier, Assigned: cabanier)

Tracking

(Blocks 1 bug, {access, dev-doc-complete, html5})

Trunk
mozilla29
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
The HTML A11y group decided to rename drawSystemFocusRing to drawFocusIfNeeded since it is less confusing.
(Assignee)

Updated

5 years ago
Blocks: 935992
Keywords: access, html5
(Assignee)

Updated

5 years ago
Assignee: nobody → cabanier
Comment on attachment 8360152 [details] [diff] [review]
Renamed idl + impl + test files

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

Thanks!
Attachment #8360152 - Flags: review?(roc) → review+
By the way, should we remove drawCustomFocusRing? It seems the name and/or behavior are likely to change and it won't be standardized for some time.
(Assignee)

Comment 4

5 years ago
(In reply to Robert O'Callahan (:roc) (Mozilla Corporation) from comment #3)
> By the way, should we remove drawCustomFocusRing? It seems the name and/or
> behavior are likely to change and it won't be standardized for some time.

I was planning on leaving it behind the runtime flag for now. 
Maybe it's better to remove it since it is not going to be adopted? If you think so, I will create a bug to remove it.
(Assignee)

Updated

5 years ago
Attachment #8360152 - Flags: checkin?(ryanvm)
Maybe it should have its own runtime flag. We probably should enable drawFocusIfNeeded before drawCustomFocusRing.
(Assignee)

Comment 6

5 years ago
(In reply to Robert O'Callahan (:roc) (Mozilla Corporation) from comment #5)
> Maybe it should have its own runtime flag. We probably should enable
> drawFocusIfNeeded before drawCustomFocusRing.

OK. I will introduce a new flag for customfocusring.
Attachment #8360152 - Flags: checkin?(ryanvm)
https://hg.mozilla.org/mozilla-central/rev/81e542a2ab70
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
You need to log in before you can comment on or make changes to this bug.