Last modified: 2012-09-27 01:10:50 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 T25453, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 23453 - IRC icon in Vector gets cut off at the top
IRC icon in Vector gets cut off at the top
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Interface (Other open bugs)
1.17.x
All All
: Normal minor (vote)
: ---
Assigned To: Trevor Parscal
http://meta.wikimedia.org/wiki/Talk:S...
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-05-09 21:09 UTC by Mike.lifeguard
Modified: 2012-09-27 01:10 UTC (History)
5 users (show)

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


Attachments
Screenshot of the error (169.90 KB, image/png)
2010-05-09 21:09 UTC, Mike.lifeguard
Details

Description Mike.lifeguard 2010-05-09 21:09:23 UTC
Created attachment 7359 [details]
Screenshot of the error

The top 1px gets cut off on both Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.2.3) Gecko/20100423 Ubuntu/10.04 (lucid) Firefox/3.6.3 and Google Chrome 5.0.307.11 beta.

On my wiki, I set:
#content a[href^="irc://"], #content a.extiw[href^="irc://"], .link-irc {
    padding: 1px 18px 0 0;
}
Comment 1 Trevor Parscal 2010-05-10 17:06:54 UTC
Yes, the icons need to be lower sizes to be able to be shown inline with text in most cases.
Comment 2 Trevor Parscal 2010-05-10 18:34:12 UTC
Solved in r66151.
Comment 3 Mike.lifeguard 2010-05-10 19:07:58 UTC
This makes the icons tiny, which means all the detail inside the icons is gone. Furthermore, monobook somehow manages to get the icons to not be cut off when they're taller than line-height, can't vector do the same? It is 1px off, no need to butcher the icons by 6px. Is there some reason adjusting padding by 1px isn't good enough?
Comment 4 Trevor Parscal 2010-05-10 19:55:23 UTC
The larger icons were stock 16px icons pulled from the tango library which are actually only 14px since they had a 1px clear border around them. If you look at the sizes of the actual graphics in the Monobook icons, you will see that they are all between 10 and 12 pixels in height, so we're really talking about somewhere between 0 and 2 pixels difference between Vector and Monobook.

Vector's icons are more consistently sized than those of Monobook, which is essentially the justification for the 2px differences where they occur.

Additionally, the sizes we're using here flow more consistently with the text, given they are more consistently similar to the height of the rendered text in most situations.

So, sure - span padding is clever, but it's not really going to solve the real problem, that the icons were actually too large to begin with.

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


Navigation
Links