Closed
Bug 810586
Opened 12 years ago
Closed 12 years ago
Collapsible table (wikipedia) not rendered correctly
Categories
(Core :: Layout: Tables, defect)
Tracking
()
RESOLVED
FIXED
mozilla20
People
(Reporter: bugzilla_acct_1959, Assigned: dbaron)
References
Details
(Keywords: regression, testcase)
Attachments
(6 files)
475.98 KB,
image/jpeg
|
Details | |
480 bytes,
text/html
|
Details | |
220 bytes,
text/html
|
Details | |
150 bytes,
text/html
|
Details | |
9.03 KB,
patch
|
bajaj
:
approval-mozilla-aurora+
bajaj
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
5.87 KB,
patch
|
dholbert
:
review+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:19.0) Gecko/19.0 Firefox/19.0 Build ID: 20121109030635 Steps to reproduce: Go to http://en.wikipedia.org/wiki/Hundred_of_Hoo_Railway (Nightly 19.0a1 2012-11-09) Actual results: Collapsible table (showing diagram of railway line) has too much whitespace to right of table. When browser window is narrow, table intrudes over text. Error not present on current Firefox (16.0.2). Expected results: Incorrect whitespace shouldn't appear - should render as in 16.0.2
(In reply to Jonathan from comment #0) > Incorrect whitespace shouldn't appear - should render as in 16.0.2 I observe the same behavior with FF16 and FF19, the table "enters" into the text when the browser window is reduced horizontaly.
Comment 2•12 years ago
|
||
Thank you good fellows! I confirm this wrong behavior on FF 19.0a1 (2012-11-12) on Win7 x64 when reducing the browser dimensions. Setting the resolution to NEW.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment 3•12 years ago
|
||
I've reduced the problem to the attached testcase - it seems to be the combination of colspan and width=100% that causes each spanned column to be as wide as the content. Firefox 17, IE 9 and Chrome 23 fit the table to the text; in Firefox 18 the table is twice as wide.
Thanks Stuart, confirmed in Firefox 18 also - I've changed the branch to reflect this.
Severity: normal → major
Attachment #693057 -
Attachment mime type: text/plain → text/html
Also upped severity to major - Firefox 18 due for release w/c 6 Jan and this bug must affect a lot of webpages.
Regression range: m-c good=2012-09-07 bad=2012-09-08 http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=36427d4b2cf6&tochange=1d4fc0c60063 Need more investigation.
status-firefox17:
--- → unaffected
tracking-firefox18:
--- → ?
tracking-firefox19:
--- → ?
tracking-firefox20:
--- → ?
Comment 7•12 years ago
|
||
I suspect bug 734569 but I'm currently bisecting
Component: Untriaged → Layout: Tables
Product: Firefox → Core
![]() |
||
Comment 8•12 years ago
|
||
Regression window(m-i) Good: http://hg.mozilla.org/integration/mozilla-inbound/rev/0afb00f99006 Mozilla/5.0 (Windows NT 6.1; WOW64; rv:18.0) Gecko/18.0 Firefox/18.0 ID:20120907155516 Bad: http://hg.mozilla.org/integration/mozilla-inbound/rev/3916c4ac4a81 Mozilla/5.0 (Windows NT 6.1; WOW64; rv:18.0) Gecko/18.0 Firefox/18.0 ID:20120907160717 Pushlog: http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=0afb00f99006&tochange=3916c4ac4a81 Suspected :Bug 734569
Updated•12 years ago
|
Blocks: 734569
Keywords: regressionwindow-wanted
Comment 9•12 years ago
|
||
The first bad revision is: changeset: 104624:1ead7ec99cea user: L. David Baron <dbaron@dbaron.org> date: Fri Sep 07 15:57:06 2012 -0700 summary: Distribute the width of column-spanning cells to columns with nothi ng in them, like we do for final table widths. (Bug 734569) r=dholbert
Updated•12 years ago
|
Assignee: nobody → dbaron
Comment 10•12 years ago
|
||
Considering where we are in the FF18 and this is not a very severe issue "wontfix" for FF18 & tracking it fir FF19 and above.:Dbaron, can you please help investigate ?
Updated•12 years ago
|
Assignee | ||
Comment 11•12 years ago
|
||
Please don't assign bugs directly to me without sending me email; I'm unlikely to notice. (Really, I think that's true in general; you shouldn't change the assignee field without making sure that the assignee is actually aware of that.)
Assignee | ||
Comment 12•12 years ago
|
||
Assignee | ||
Comment 13•12 years ago
|
||
Assignee | ||
Comment 14•12 years ago
|
||
For beta I'd like to back out bug 734569. It's pretty isolated from other changes so it should below risk to go back to the way things used to be; it's also not unlikely other occurrences of this same pattern will show up and break other pages. (For trunk, patches coming shortly.) [Approval Request Comment] Bug caused by (feature/regressing bug #): bug 734569 User impact if declined: bad layout of Web pages Testing completed (on m-c, etc.): it's our previous behavior Risk to taking this patch (and alternatives if risky): low String or UUID changes made by this patch: none
Attachment #694439 -
Flags: approval-mozilla-beta?
Assignee | ||
Updated•12 years ago
|
status-firefox18:
wontfix → ---
Comment 15•12 years ago
|
||
Comment on attachment 694439 [details] [diff] [review] back out 734569 for beta Approving the backout for FF18.Please make sure to perform the backout before Wednesday(12/26) .
Attachment #694439 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Updated•12 years ago
|
status-firefox18:
--- → affected
Assignee | ||
Comment 16•12 years ago
|
||
https://hg.mozilla.org/releases/mozilla-beta/rev/931590387ec9 Still need to merge to b2g18.
Assignee | ||
Comment 17•12 years ago
|
||
https://hg.mozilla.org/releases/mozilla-b2g18/rev/931590387ec9
status-b2g18:
--- → fixed
Assignee | ||
Comment 18•12 years ago
|
||
https://tbpl.mozilla.org/?tree=Try&rev=d9f469a80bdb
Attachment #697224 -
Flags: review?(dholbert)
Assignee | ||
Comment 19•12 years ago
|
||
Comment on attachment 694439 [details] [diff] [review] back out 734569 for beta At this point I'd also like to take the backout for Aurora. Same [Approval Request Comment] as before.
Attachment #694439 -
Flags: approval-mozilla-aurora?
Updated•12 years ago
|
OS: Windows 7 → All
Hardware: x86_64 → All
Updated•12 years ago
|
Attachment #697224 -
Flags: review?(dholbert) → review+
Updated•12 years ago
|
Attachment #694439 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 20•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/637e063b2055
Assignee | ||
Comment 21•12 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/1313ee568d60
Comment 22•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/637e063b2055
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla20
Comment 23•12 years ago
|
||
(In reply to Loic from comment #1) > the table "enters" into the > text when the browser window is reduced horizontaly. This is still reproducible on Nightly 21.0a1 (2013-01-09), FF 19b1 on http://en.wikipedia.org/wiki/Hundred_of_Hoo_Railway, so the problem is not completely solved. See Chrome and Opera behaviors.
Comment 24•12 years ago
|
||
Can someone please advise based on comment 23? Should this bug be reopened? Should a follow-up bug be filed?
Comment 25•12 years ago
|
||
Yup, this still isn't quite right. Testing w/ my nightly (2013-01-09), it looks like this part is fixed: (In reply to Jonathan from comment #0) > Collapsible table (showing diagram of railway line) has too much whitespace > to right of table. But this part is broken: > When browser window is narrow, table intrudes over text. We should probably track that in a followup, rather than trying to fix both issues in this bug.
Comment 26•12 years ago
|
||
(In reply to Daniel Holbert [:dholbert] from comment #25) > Testing w/ my nightly (2013-01-09), it looks like this part is fixed: > (In reply to Jonathan from comment #0) > > Collapsible table (showing diagram of railway line) has too much whitespace > > to right of table. Verified fixed FF 18.0.1, 19b2, 20.0a2 (2013-01-22) > But this part is broken: > > When browser window is narrow, table intrudes over text. > > We should probably track that in a followup, rather than trying to fix both > issues in this bug. bug 833788 filed
Keywords: verifyme
You need to log in
before you can comment on or make changes to this bug.
Description
•