[HOME PAGE] [STORES] [CLASSICISTRANIERI.COM] [FOTO] [YOUTUBE CHANNEL]

Bug 31993 – 1.18 regression: Monobook skin left bar has very large spacing in IE 7 (and IE 8 & IE 9's compatibility mode)

Last modified: 2011-11-03 02:46:18 UTC

Wikimedia Bugzilla is closed!

Wikimedia has migrated from Bugzilla to Phabricator. Bug reports should be created and updated in Wikimedia Phabricator instead. Please create an account in Phabricator and add your Bugzilla email address to it.
Wikimedia Bugzilla is read-only. If you try to edit or create any bug report in Bugzilla you will be shown an intentional error message.
In order to access the Phabricator task corresponding to a Bugzilla report, just remove "static-" from its URL.
You could still run searches in Bugzilla or access your list of votes but bug reports will obviously not be up-to-date in Bugzilla.

Bug 31993 - 1.18 regression: Monobook skin left bar has very large spacing in IE 7 (and IE 8 & IE 9's compatibility mode)


Summary:	1.18 regression: Monobook skin left bar has very large spacing in IE 7 (and I...

Status:	RESOLVED FIXED

Product:	MediaWiki
Classification:	Unclassified
Component:	Interface (Other open bugs)
Version:	1.18.x
Hardware:	All Windows 7

Importance:	High normal (vote)
Target Milestone:	1.18.0 release
Assigned To:	Nobody - You can work on this!

URL:
Whiteboard:
Keywords:

Duplicates:	32166 (view as bug list)
Depends on:
Blocks:	29876
	Show dependency tree / graph

Reported:	2011-10-27 15:43 UTC by snews
Modified:	2011-11-03 02:46 UTC (History)
CC List:	3 users (show)

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

Attachments
Screentshot of left bar spacing problem (227.75 KB, application/vnd.openxmlformats-officedocument.wordprocessingml.document) 2011-10-27 15:43 UTC, snews	Details
Re-uploaded screenshot (196.78 KB, image/png) 2011-10-28 08:30 UTC, Roan Kattouw	Details
Screenshot of left bar spacing problem (204.66 KB, image/png) 2011-10-28 08:30 UTC, Michael M.	Details
Show Obsolete (1) Add an attachment (proposed patch, testcase, etc.)

Description snews 2011-10-27 15:43:55 UTC

Created attachment 9296 [details]
Screentshot of left bar spacing problem

There are very large spaces between the navigation box, the search box, the interaction box, the toolbox, the pring/export box, and the languages box in the left bar of the classic view.  This occurs only in the English-language version.

Attached is a screenshot.

Comment 1 Roan Kattouw 2011-10-28 08:30:00 UTC

Created attachment 9308 [details]
Re-uploaded screenshot

Re-uploading screenshot as PNG.

In the future, please don't upload screenshots embedded in Word documents, upload them as image files instead.

Comment 2 Michael M. 2011-10-28 08:30:54 UTC

Created attachment 9309 [details]
Screenshot of left bar spacing problem

Replaced screenshot in .docx format by a PNG.
If I remember correctly the space is caused by the "Compatibility View" of Internet Explorer.

Comment 3 Lupo 2011-10-28 12:31:19 UTC

Yes. Original reports at

https://en.wikipedia.org/w/index.php?title=Wikipedia:Village_pump_%28technical%29&oldid=457793423#Monobook_Sidebar_has_spaces_between_boxes

and at

https://en.wikipedia.org/w/index.php?title=Wikipedia:Village_pump_%28technical%29&oldid=457793423#Layout_is_screwed_up_with_compatibility_mode_.28on_IE9_at_least.29

Appears to happen on IE9 and IE8, both in compatibility mode, as well IE7. I could not reproduce this in IE6.

Comment 4 Brion Vibber 2011-11-01 17:55:27 UTC

I can't reproduce this on IE 7/XP, IE 8/XP (regular or compatibility), or IE 9/Win7 (regular or compatibility).

Could be something that was specific to the styles and images that were on the main page that particular day, perhaps?

Comment 5 Brion Vibber 2011-11-01 17:56:28 UTC

Ah, this is Monobook-only. Adding to the summary so people might be able to repro. :)

Comment 6 Brion Vibber 2011-11-01 18:17:47 UTC

Confirmed this is a regression in 1.18's Monobook skin and can reproduce on my local 1.18 and trunk installs while 1.17 is fine.

Primary problem is with IE 7; IE 8 & 9's "compatibility mode" emulates IE 7's rendering behavior, so those modes showing the bug too is not surprising. :) Regular mode *should* be the default (and is for me at least) and appears fine.

Comment 7 Brion Vibber 2011-11-01 18:34:28 UTC

Removing the 'float: left' from the "HTML > BODY .portlet" rule seems to resolve this.

Rule was added in r74966 by Roan, while porting MonoBook styles to the Vector infrastructure. There's no specific comment explaining what its purpose is...

http://www.mediawiki.org/wiki/Special:Code/MediaWiki/74966#c13415 questions the purpose of the block; the rule after it seems to be a hack for IE/Mac (which is so broken these days we don't care about supporting it) while the actual rule here, just not sure why it's there.

Comment 8 Brion Vibber 2011-11-01 22:12:48 UTC

Fixed this on trunk in r101540 by removing the suspicious float/clear bits.

Roan confirmed that code was ported from Monobook's old rtl.css, but the float/clear is no longer needed. (Oddly, on RTL there was no spacing issue with the floats. Bah!)

Marked in CR for merge to 1.18 & 1.18wmf1 deployment as a regression fix.

Comment 9 David Hyland 2011-11-03 02:46:18 UTC

*** Bug 32166 has been marked as a duplicate of this bug. ***

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

Search

Personal tools

Navigation

Links

Contents Listing Alphabetical by Author:

A B C D E F G H I J K L M N O P Q R S T U V W X Y Z Unknown Other

Contents Listing Alphabetical by Title:

# A B C D E F G H I J K L M N O P Q R S T U V W Y Z Other

Medical Encyclopedia

Browse by first letter of topic:

A-Ag	Ah-Ap	Aq-Az	B-Bk	Bl-Bz	C-Cg	Ch-Co
Cp-Cz	D-Di	Dj-Dz	E-Ep	Eq-Ez	F	G
H-Hf	Hg-Hz	I-In	Io-Iz	J	K	L-Ln
Lo-Lz	M-Mf	Mg-Mz	N	O	P-Pl	Pm-Pz
Q	R	S-Sh	Si-Sp	Sq-Sz	T-Tn	To-Tz
U	V	W	X	Y	Z	0-9