Closed Bug 287738 Opened 19 years ago Closed 19 years ago

Not all content exposed in iframes

Categories

(Core :: Disability Access APIs, defect)

x86
Windows XP
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: aaronlev, Assigned: aaronlev)

References

Details

(Keywords: access)

Attachments

(3 files, 1 obsolete file)

Steps:
Load test case and accessible explorer
Drag object from point target over iframe

Notice that "One Two" and the checkbox are exposed, but not "Three"

Also, if the iframe is increased in size, the bug does not appear.
Attached file The actual testcase
Blocks: deera11y
The problem was really that when the viewing area is small, a new text node
sometimes has a 0x0 frame for the primary frame -- all of the frames with
something in them are pushed to the next line. We were checking the primary
frame's size to see if the text node is empty.
Attachment #183532 - Flags: superreview?(bzbarsky)
Attachment #183532 - Flags: review?(timeless)
Is there a reason this isn't just calling IsEmpty() on the frame?  That should
give the right answer, I would think...
Attachment #183532 - Attachment is obsolete: true
Attachment #183532 - Flags: superreview?(bzbarsky)
Attachment #183532 - Flags: review?(timeless)
Attachment #183538 - Flags: superreview?(bzbarsky)
Attachment #183538 - Flags: review?(timeless)
Attachment #183538 - Flags: review?
Attachment #183538 - Flags: review?(timeless) → review+
Attachment #183538 - Flags: superreview?(bzbarsky) → superreview+
Attachment #183538 - Flags: approval-l10n?
Attachment #183538 - Flags: approval1.8b2?
Attachment #183538 - Flags: approval-l10n?
Comment on attachment 183538 [details] [diff] [review]
Use IsEmpty() which allows us to remove our own hack which checks for empty text frames

a=chofmann
Attachment #183538 - Flags: approval1.8b2? → approval1.8b2+
Checking in accessible/src/base/nsAccessibilityService.cpp;
/cvsroot/mozilla/accessible/src/base/nsAccessibilityService.cpp,v  <-- 
nsAccessibilityService.cpp
new revision: 1.140; previous revision: 1.139
done
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: