557795 - wrong list bullet text of accessible for the first numbered HTML:li in CKEditor

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[alexander :surkov (:asurkov)]

create numbered list in CKEditor and inspect the tree, bullet accessible of the first HTML li has "0." accessible name however visually number is 1.

It looks like nsBulletFrame::SetListItemOrdinal() is called after we are notified the HTML li is appended to DOM.

Comment 2

[alexander :surkov (:asurkov)]

Attachment: patch

Don't cache bullet text, request it when it's necessary. Since accessible events are fired after layout then AT asks for bullet text when it's ready.

Comment 3

[alexander :surkov (:asurkov)]

Bug 570275 (that has blockign 2.0 status) is going to make things even more worse, all inserted list bullet will have "0. " text. Based on that asking blocking status for this bug as well.

Comment 4

[alexander :surkov (:asurkov)]

(In reply to comment #3)
> Bug 570275 (that has blockign 2.0 status) is going to make things even more
> worse,

just fyi, it's just side affect of reworked update tree code and doesn't mean logic in that patch of bug 570275 is wrong.

Comment 5

[David Bolter [:davidb] (NeedInfo me for attention)]

Blocking approved. I made this block 570275, and status is inherited.

Comment 6

[alexander :surkov (:asurkov)]

landed on 2.0 - http://hg.mozilla.org/mozilla-central/rev/9b24526d8132

Comment 7

[alexander :surkov (:asurkov)]

non-libxul bustage

Comment 8

[alexander :surkov (:asurkov)]

Attachment: libxul fix

Comment 9

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 483151 [details] [diff] [review]
libxul fix

r=me

Comment 10

[alexander :surkov (:asurkov)]

libxul bustage fix is landed - http://hg.mozilla.org/mozilla-central/rev/5a35da9caa98