Closed
Bug 469983
Opened 15 years ago
Closed 15 years ago
Chatzilla layout busted
Categories
(Other Applications :: ChatZilla, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jmjjeffery, Assigned: dbaron)
References
Details
(Keywords: regression, Whiteboard: [cz-0.9.85])
Attachments
(1 file)
1.53 KB,
patch
|
Gijs
:
review+
|
Details | Diff | Splinter Review |
Starting with changeset ID: http://hg.mozilla.org/mozilla-central/rev/4a73efb88edc Chatzilla layout is busted and unusable. Works OK in changeset: http://hg.mozilla.org/mozilla-central/rev/214c605bfb64 Patches in that range: https://bugzilla.mozilla.org/show_bug.cgi?id=421203 https://bugzilla.mozilla.org/show_bug.cgi?id=454349
Assignee | ||
Comment 1•15 years ago
|
||
Look at the error console and see if there's an error saying "XUL box for %1$S element contained an inline %2$S child, forcing all its children to be wrapped in a block". If so, what file, and what elements?
Reporter | ||
Comment 2•15 years ago
|
||
This in the Console2: Warning: XUL box for window element contained an inline link child, forcing all its children to be wrapped in a block. Source file: chrome://chatzilla/content/chatzilla.xul Clicking the link to the source file give no further info... Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.2a1pre) Gecko/20081217 Minefield/3.2a1pre Firefox/3.0.4 ID:20081217070445
Assignee | ||
Comment 3•15 years ago
|
||
This ought to fix it.
Attachment #353452 -
Flags: review?(gijskruitbosch+bugs)
Comment 4•15 years ago
|
||
(In reply to comment #3) > Created an attachment (id=353452) [details] > patch > > This ought to fix it. I don't like sounding like an idiot, but at the risk of it, why do we need this fix? Isn't html:link already supposed to be display:none? If not, what's its graphical representation? (I regret to say I know little about layout, so again, my questions may be wrongly phrased or misplaced, but I would like to be enlightened a little bit here)
Assignee | ||
Comment 5•15 years ago
|
||
html:head is display:none, and html:link is supposed to be inside html:head.
Comment 6•15 years ago
|
||
Comment on attachment 353452 [details] [diff] [review] patch Alright, thanks for clarifying. r=me, then
Attachment #353452 -
Flags: review?(gijskruitbosch+bugs) → review+
Assignee | ||
Comment 7•15 years ago
|
||
Fix checked in to CVS trunk.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 8•15 years ago
|
||
Not M-C ?? Its also broken - at least that's what I filed the bug under..
Assignee | ||
Comment 9•15 years ago
|
||
The fix is to Chatzilla, which is in CVS.
Comment 10•15 years ago
|
||
Would it make sense to add link { display:none } to html.css as a backstop against future instances of this problem?
Assignee | ||
Comment 11•15 years ago
|
||
I don't think so. It would probably also break some of Hixie's tests...
Comment 12•15 years ago
|
||
(In reply to comment #9) > The fix is to Chatzilla, which is in CVS. Correct. This fix should be in .85, which may take a while to release. Meanwhile you can use trunk builds from CVS. SeaMonkey imports it from there, I believe. If that does not work for you, you can also install a nightly version once it appears on http://twpol.dyndns.org/mozilla/chatzilla/nightly/ (should be within the next hour or so) (In reply to comment #10) > Would it make sense to add link { display:none } to html.css as a backstop > against future instances of this problem? That would technically be a different bug, I think? Related to that, David, can this bug be moved to ChatZilla (Other Apps) so we can find it more easily? It was fixed in our code, anyway. Thanks for your help in getting this fixed quickly, much appreciated! :-)
Whiteboard: [cz-0.9.85]
Comment 13•15 years ago
|
||
(In reply to comment #12) > (In reply to comment #10) > > Would it make sense to add link { display:none } to html.css as a backstop > > against future instances of this problem? > > That would technically be a different bug, I think? Sorry about that, I'd reloaded the page - hence not getting mid-aired - without rereading for new comments; my bad!
Assignee | ||
Updated•15 years ago
|
Assignee: nobody → dbaron
Component: Layout → ChatZilla
Product: Core → Other Applications
QA Contact: layout → chatzilla
Reporter | ||
Comment 14•15 years ago
|
||
Latest nightly build from the link in comment #12 fixed Chatzilla on trunk-builds. Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.2a1pre) Gecko/20081217 Minefield/3.2a1pre Firefox/3.0.4 ID:20081217070445
You need to log in
before you can comment on or make changes to this bug.
Description
•