Closed
Bug 827799
Opened 13 years ago
Closed 13 years ago
Visually Hidden HTML 5 technique does not hide the caption in a table
Categories
(Core :: Web Painting, defect)
Tracking
()
RESOLVED
FIXED
mozilla21
People
(Reporter: dylan, Assigned: bzbarsky)
Details
(Keywords: dev-doc-needed, testcase)
Attachments
(2 files)
257 bytes,
text/html
|
Details | |
1.79 KB,
patch
|
roc
:
review+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_8_2) AppleWebKit/537.11 (KHTML, like Gecko) Chrome/23.0.1271.101 Safari/537.11
Steps to reproduce:
Apply the visuallyhidden style from http://davidwalsh.name/html5-boilerplate to a table with a caption.
Actual results:
Everything in the table is hidden except the caption.
Expected results:
Everything including the caption should have been hidden
1) Check, if problem exists in current nightly version from http://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/latest-mozilla-central/firefox-21.0a1.en-US.mac.dmg
2) If yes, attach to this bug simple testcase showing the issue
Component: Untriaged → Style System (CSS)
Flags: needinfo?(dylan)
Product: Firefox → Core
![]() |
Assignee | |
Comment 2•13 years ago
|
||
Phoenix, you can put the testcase together pretty easily from the link, no? I'll attach it in a sec.
It's not clear to me what the right behavior here is per spec, for a table... In particular, the overflow property does NOT apply to tables per spec as far as I can see, so there's no obvious reason the caption would be hidden. But the clip should work, I would think...
Component: Style System (CSS) → Layout: View Rendering
Flags: needinfo?(roc)
![]() |
Assignee | |
Comment 3•13 years ago
|
||
That sounds right. ::-moz-table-outer should probably inherit "clip".
Flags: needinfo?(roc)
![]() |
||
Updated•13 years ago
|
![]() |
Assignee | |
Comment 5•13 years ago
|
||
Attachment #700330 -
Flags: review?(roc)
![]() |
Assignee | |
Updated•13 years ago
|
Assignee: nobody → bzbarsky
Status: UNCONFIRMED → NEW
Ever confirmed: true
Whiteboard: [need review]
Comment 6•13 years ago
|
||
Adding dev-doc-needed, it will be a good opportunity to document ::-moz-table-outer on the MDN.
Keywords: dev-doc-needed
![]() |
Assignee | |
Comment 7•13 years ago
|
||
The current situation with dev-doc-needed is that it's a no-op. If you think there should be docs, you need to find someone (possibly yourself) to do them....
Comment 8•13 years ago
|
||
Yep, that's the idea. Dev-doc-needed appear in our tracker tool (even a long time after the release with the fix). Though we don't have the resource to take care of all of them, they still appear. I try to take care of all CSS-related dev-doc-needed once every month or so (depending of my time).
Also use the dev-doc-needed/complete list for a release in the process leading to the Release Notes (via the Aurora meeting).
And this bug, once fixed, is good candidate for our "Site compatibility for Firefox XY" and "Firefox XY for developers" page too. (Though the inclusion in these is not enough to clear the flag, ddn/ddc bugs are used to fill these pages too)
So dev-doc-needed, even if it doesn't lead to the proper doc to be automatically written, has some useful purposes ;-)
Attachment #700330 -
Flags: review?(roc) → review+
![]() |
Assignee | |
Comment 9•13 years ago
|
||
Flags: in-testsuite+
Whiteboard: [need review]
Target Milestone: --- → mozilla21
Comment 10•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•7 years ago
|
Component: Layout: View Rendering → Layout: Web Painting
You need to log in
before you can comment on or make changes to this bug.
Description
•