Last modified: 2014-04-15 20:49:44 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T64026, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 62026 - Clear:both for new table of contents doesn't work well with TOCRight template and divs with borders
Clear:both for new table of contents doesn't work well with TOCRight template...
Status: RESOLVED INVALID
Product: MediaWiki extensions
Classification: Unclassified
VectorBeta (Other open bugs)
unspecified
All All
: Unprioritized normal (vote)
: ---
Assigned To: Nobody - You can work on this!
https://www.mediawiki.org/wiki/Manual...
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-02-27 23:19 UTC by Ryan Kaldari
Modified: 2014-04-15 20:49 UTC (History)
3 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
screenshot (194.99 KB, image/png)
2014-02-27 23:19 UTC, Ryan Kaldari
Details
image of the page without issue. (205.40 KB, image/png)
2014-02-27 23:24 UTC, Jared Zimmerman (WMF)
Details
Failure in Chrome (247.95 KB, image/png)
2014-02-28 00:48 UTC, James Forrester
Details
Failure in Opera (204.36 KB, image/png)
2014-02-28 00:49 UTC, James Forrester
Details

Description Ryan Kaldari 2014-02-27 23:19:58 UTC
Created attachment 14711 [details]
screenshot

See attached screenshot, or visit...
https://www.mediawiki.org/wiki/Manual:Coding_conventions/PHP
...with Typography Refresh turned on.
Comment 1 Jared Zimmerman (WMF) 2014-02-27 23:24:41 UTC
Created attachment 14712 [details]
image of the page without issue.
Comment 2 Jared Zimmerman (WMF) 2014-02-27 23:25:00 UTC
I'm not seeing this issue.
Comment 3 James Forrester 2014-02-28 00:34:15 UTC
Confirm that I've got the same issue.
Comment 4 Ryan Kaldari 2014-02-28 00:40:22 UTC
Looks like the content clears in Chrome, but not in Safari or Firefox.
Comment 5 James Forrester 2014-02-28 00:48:58 UTC
Created attachment 14714 [details]
Failure in Chrome
Comment 6 James Forrester 2014-02-28 00:49:15 UTC
Created attachment 14715 [details]
Failure in Opera
Comment 7 James Forrester 2014-02-28 00:49:34 UTC
Nope, having the issue in all four of Chrome/Safari/Firefox/Opera.
Comment 8 Ryan Kaldari 2014-02-28 00:50:48 UTC
That's weird. What browser are you using Jared?
Comment 9 Jared Zimmerman (WMF) 2014-02-28 00:51:23 UTC
Chrome Version 33.0.1750.117 beta OS X 10.8
Comment 10 Jared Zimmerman (WMF) 2014-02-28 00:52:56 UTC
er, sorry 10.9.2 OSX
Comment 11 Ryan Kaldari 2014-02-28 00:54:36 UTC
I'm Chrome 33.0.1750.117 on OS X 10.8. I have no idea why Chrome would be clearing the TOC (it shouldn't), and even less idea why it isn't for James.
Comment 12 Ryan Kaldari 2014-03-06 00:50:49 UTC
Reduced to simplest test case:

<div style="width: 20em; float: right;">
Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.
</div>

<h2 style="overflow: hidden; margin-left: -0.01em;">Header</h2>

Try loading the following page in different browsers and tell me what you get:
http://toolserver.org/~kaldari/bug62026.html

I see the header below the text in Chrome 33.0.1750.146, but next to the text in Firefox and Safari.
Comment 13 Gerrit Notification Bot 2014-03-06 01:11:49 UTC
Change 117134 had a related patch set uploaded by Kaldari:
Removing negative margin for headers since it breaks Chrome :(

https://gerrit.wikimedia.org/r/117134
Comment 14 Gerrit Notification Bot 2014-03-06 01:43:03 UTC
Change 117134 merged by Jdlrobson:
Removing negative margin for headers since it breaks Chrome :(

https://gerrit.wikimedia.org/r/117134
Comment 15 Steven Walling 2014-03-31 00:46:17 UTC
The beta feature for this no longer exists on master, and will be turned off next Tuesday/Thursday with the MediaWiki release. In the version committed to Vectore in core, the TOC change is not present.
Comment 16 James Forrester 2014-04-15 20:17:59 UTC
FIXED -> INVALID as it… wasn't.
Comment 17 Ryan Kaldari 2014-04-15 20:49:44 UTC
Just for the records...

The Chrome bug was reported at:
http://code.google.com/p/chromium/issues/detail?id=350473

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links