Last modified: 2012-09-27 01:10:42 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 T21567, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 19567 - White space at the top of the article
White space at the top of the article
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Interface (Other open bugs)
1.16.x
All All
: Normal normal with 1 vote (vote)
: ---
Assigned To: Trevor Parscal
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-07-07 19:13 UTC by triatrea
Modified: 2012-09-27 01:10 UTC (History)
5 users (show)

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


Attachments
A picture of the white space at the top of the article (16.52 KB, image/png)
2009-07-07 19:15 UTC, triatrea
Details

Description triatrea 2009-07-07 19:13:06 UTC
There is a padding at the top of the article, right above the <h1> tag. It is caused by <code>padding-top: 0.9em;</code> in <code>#siteNotice</code>. This bug was fixed in Monobook a while ago.

See [[19219]] for more information.
Comment 1 triatrea 2009-07-07 19:15:44 UTC
Created attachment 6304 [details]
A picture of the white space at the top of the article
Comment 2 Gurch 2009-07-07 19:50:49 UTC
In short, "0 .9em" was mis-corrected to "0.9em", when the intention was "0 0.9em"

Didn't check the other skins at the time because the change was to Monobook, but of course Vector has copied some stuff from there.
Comment 3 Trevor Parscal 2009-07-07 20:26:54 UTC
This is working well now afai can test.
Comment 4 Derk-Jan Hartman 2009-07-07 20:38:16 UTC
In vector this margin was recently changed from a margin-top to a margin-bottom. But sitenotice is not in sync with monobook at the moment.
Comment 5 Roan Kattouw 2009-07-07 20:42:58 UTC
(In reply to comment #3)
> This is working well now afai can test.
> 

For reference, this was done in r52855.
Comment 6 Derk-Jan Hartman 2009-07-18 10:42:38 UTC
Still broken, as can be seen on:

http://test.wikipedia.org/w/index.php?title=Main_Page&useskin=vector
after you hide the board notice.

The css adds a maring to the siteNotice div, but this div is almost ALWAYS present on foundation websites due to the perpetual deployment of the centralnotice javascript loader. Even though in many cases this loader produces no content, or hidden content, the loader itself is considered to be content for the siteNotice and THUS the siteNotice introduces an extra margin even though there is no user visible content to be presented.

Basically, this is why the centralNotice implementation sucks :D
Comment 7 Trevor Parscal 2009-07-21 22:44:55 UTC
Solved, I believe properly, in r53629.
Comment 8 Derk-Jan Hartman 2009-08-05 23:06:41 UTC
The problem still exists. Now the problem shows when there is a sitenotice, but you have chosen "hide".
Safari 4.

I have "#siteNotice div.expanded table.siteNoticeUser { margin-bottom: 1em; }" deployed on en.wp to "fix" this, but to be honest, i'm not quite sure how it acts when there are local sitenotices active, and how it works for anonymous users... 
Comment 9 Derk-Jan Hartman 2010-03-29 15:05:39 UTC
This issue is still present on all Wikimedia sites (except english wikipedia) that now run the scholarship centralnotice.
Comment 10 Derk-Jan Hartman 2010-06-10 20:42:26 UTC
For any other Wikis that are now seeing this problem, English wikipedia and Dutch Wikipedia have the following CSS in MediaWiki:Vector.css to correct this problem:

/* Temporary fix until [[bugzilla:19567]] is addressed */
#siteNotice div { margin: 0; }
#siteNotice div.expanded table.siteNoticeUser { margin-bottom: 1em; }
Comment 11 Ryan Kaldari 2010-06-22 01:24:28 UTC
The essential problem here is that there is no consistent way of determining if any content is actually being shown by the centralNotice or not. toggleNotice() switches the centralNotice div to have a class of either "expanded" or "collapsed", but collapsed can mean "hidden completely" or "collapsed to a smaller size notice" depending on how things are defined in the banner/template itself. In one case, we want a margin-bottom; in the other case, we don't. Is the best solution to always leave the margin-setting to the banner/template builder and remove it from the skin and extension css (as has been done with the local hack above)? The down side of this is that it means that you can't just create a simple centralNotice banner with no custom CSS and expect it to look right on the site. Is there a better solution?
Comment 12 Ryan Kaldari 2010-10-14 00:14:49 UTC
This should be fixed now. We now set margins for CentralNotice to 0 by default, and the margin is set on the actual banner div within each centralNotice (the same div that gets set to display:none when you click hide).

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


Navigation
Links