Closed
Bug 287738
Opened 20 years ago
Closed 20 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•20 years ago
|
||
Assignee | ||
Comment 2•20 years ago
|
||
Assignee | ||
Comment 3•20 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•20 years ago
|
Attachment #183532 -
Flags: superreview?(bzbarsky)
Attachment #183532 -
Flags: review?(timeless)
![]() |
||
Comment 4•20 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•20 years ago
|
Attachment #183532 -
Attachment is obsolete: true
Attachment #183532 -
Flags: superreview?(bzbarsky)
Attachment #183532 -
Flags: review?(timeless)
Assignee | ||
Comment 5•20 years ago
|
||
Attachment #183538 -
Flags: review?
Assignee | ||
Updated•20 years ago
|
Attachment #183538 -
Flags: superreview?(bzbarsky)
Attachment #183538 -
Flags: review?(timeless)
Attachment #183538 -
Flags: review?
Attachment #183538 -
Flags: review?(timeless) → review+
![]() |
||
Updated•20 years ago
|
Attachment #183538 -
Flags: superreview?(bzbarsky) → superreview+
Assignee | ||
Updated•20 years ago
|
Attachment #183538 -
Flags: approval-l10n?
Assignee | ||
Updated•20 years ago
|
Attachment #183538 -
Flags: approval1.8b2?
Assignee | ||
Updated•20 years ago
|
Attachment #183538 -
Flags: approval-l10n?
Comment 6•20 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•20 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: 20 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•