Closed
Bug 287738
Opened 19 years ago
Closed 19 years ago
Not all content exposed in iframes
Categories
(Core :: Disability Access APIs, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: aaronlev, Assigned: aaronlev)
References
Details
(Keywords: access)
Attachments
(3 files, 1 obsolete file)
404 bytes,
text/html
|
Details | |
208 bytes,
text/html
|
Details | |
1.94 KB,
patch
|
timeless
:
review+
bzbarsky
:
superreview+
chofmann
:
approval1.8b2+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•19 years ago
|
||
Assignee | ||
Comment 2•19 years ago
|
||
Assignee | ||
Comment 3•19 years ago
|
||
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.
Assignee | ||
Updated•19 years ago
|
Attachment #183532 -
Flags: superreview?(bzbarsky)
Attachment #183532 -
Flags: review?(timeless)
Comment 4•19 years ago
|
||
Is there a reason this isn't just calling IsEmpty() on the frame? That should give the right answer, I would think...
Assignee | ||
Updated•19 years ago
|
Attachment #183532 -
Attachment is obsolete: true
Attachment #183532 -
Flags: superreview?(bzbarsky)
Attachment #183532 -
Flags: review?(timeless)
Assignee | ||
Comment 5•19 years ago
|
||
Attachment #183538 -
Flags: review?
Assignee | ||
Updated•19 years ago
|
Attachment #183538 -
Flags: superreview?(bzbarsky)
Attachment #183538 -
Flags: review?(timeless)
Attachment #183538 -
Flags: review?
Attachment #183538 -
Flags: review?(timeless) → review+
Updated•19 years ago
|
Attachment #183538 -
Flags: superreview?(bzbarsky) → superreview+
Assignee | ||
Updated•19 years ago
|
Attachment #183538 -
Flags: approval-l10n?
Assignee | ||
Updated•19 years ago
|
Attachment #183538 -
Flags: approval1.8b2?
Assignee | ||
Updated•19 years ago
|
Attachment #183538 -
Flags: approval-l10n?
Comment 6•19 years ago
|
||
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+
Assignee | ||
Comment 7•19 years ago
|
||
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.
Description
•