Last modified: 2012-10-24 11:32:56 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 T37430, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 35430 - Increase line height in headings for indic languages at multilingual projects
Increase line height in headings for indic languages at multilingual projects
Status: NEW
Product: MediaWiki
Classification: Unclassified
Interface (Other open bugs)
unspecified
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on: 34514
Blocks:
  Show dependency treegraph
 
Reported: 2012-03-23 00:06 UTC by Siddhartha Ghai
Modified: 2012-10-24 11:32 UTC (History)
7 users (show)

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


Attachments

Description Siddhartha Ghai 2012-03-23 00:06:25 UTC
OldWikisource contains pages in indic scripts. CSS rules were recently applied to indic language wikipedias to increase line-height for headings due to the upper and lower diacritics used in indic scripts. Line height needs to be increased for oldWS too (probably using the same patches).

The same patches need to be done for commons since they contain such pages too.

Note: This already seems to be in effect at meta.

Refer bug 32826 and bug 30809
Comment 1 Mark A. Hershberger 2012-03-26 16:05:26 UTC
These changes were part of the core CSS and when I check oldwikisource, I see the CSS changes you request are there.

The problem you are likely seeing is  that "lang" attributes aren't being set correctly.  Can you point to a page where this works on meta as expected and a page where it doesn't work on commons?

(Adding Niklas and Siebrand since they probably have more clue about this.)
Comment 2 Siddhartha Ghai 2012-04-08 22:23:12 UTC
(In reply to comment #1)
> The problem you are likely seeing is  that "lang" attributes aren't being set
> correctly.  Can you point to a page where this works on meta as expected and a
> page where it doesn't work on commons?

Working on meta: http://meta.wikimedia.org/w/index.php?title=%E0%A4%AE%E0%A5%81%E0%A4%96%E0%A5%8D%E0%A4%AF_%E0%A4%AA%E0%A5%83%E0%A4%B7%E0%A5%8D%E0%A4%A0&uselang=hi
Not working on commons: http://commons.wikimedia.org/wiki/File:Hollow_Horn_Bear_(%E0%A4%96%E0%A5%8B%E0%A4%96%E0%A4%B2%E0%A4%BE_%E0%A4%B8%E0%A5%80%E0%A4%82%E0%A4%97_%E0%A4%AD%E0%A4%BE%E0%A4%B2%E0%A5%82).jpg
Not working at oldWS: http://wikisource.org/wiki/%E0%A4%B9%E0%A4%A8%E0%A5%81%E0%A4%AE%E0%A4%BE%E0%A4%A8%E0%A4%9A%E0%A4%BE%E0%A4%B2%E0%A5%80%E0%A4%B8%E0%A4%BE

Note: Setting uselang as hi (or any other language with the lang-specific height increase) makes the pageheading appear correctly. See the difference between http://wikisource.org/wiki/%E0%A8%AC%E0%A9%81%E0%A9%B1%E0%A8%B2%E0%A9%8D%E0%A8%B9%E0%A9%87_%E0%A8%A8%E0%A9%82%E0%A9%B0_%E0%A8%B8%E0%A8%AE%E0%A8%9D%E0%A8%BE%E0%A8%B5%E0%A8%A3_%E0%A8%86%E0%A8%88%E0%A8%AF%E0%A8%BE%E0%A8%82 and http://wikisource.org/wiki/%E0%A8%AC%E0%A9%81%E0%A9%B1%E0%A8%B2%E0%A9%8D%E0%A8%B9%E0%A9%87_%E0%A8%A8%E0%A9%82%E0%A9%B0_%E0%A8%B8%E0%A8%AE%E0%A8%9D%E0%A8%BE%E0%A8%B5%E0%A8%A3_%E0%A8%86%E0%A8%88%E0%A8%AF%E0%A8%BE%E0%A8%82?uselang=hi for example.

It appears this is also the case for single-language projects such as hi-wp. Since the heights have been specified per the lang attributes, going to http://hi.wikipedia.org/wiki/%E0%A4%AE%E0%A5%81%E0%A4%96%E0%A4%AA%E0%A5%83%E0%A4%B7%E0%A5%8D%E0%A4%A0?uselang=en shows that the height-increase is not to be seen and the diacritics are again hidden.

Sorry for the late response.
Comment 3 Siddhartha Ghai 2012-04-24 02:55:45 UTC
Does this problem basically boil down to Bug 34514 ? If yes, should this be marked as a dupe of that?
Comment 4 Fomafix 2012-05-05 09:52:13 UTC
Long letters gets cut because of

 h1, h2, h3, h4, h5, h6 { overflow: hidden; }

This was set because of Bug 26449. Bunching cannot appear on first heading, so overflow:hidden isn't needed here. The default value overflow:visible can be set by

 #firstHeading { overflow: visible; }

in skins/common/commonInterface.css
Comment 5 Siddhartha Ghai 2012-05-07 13:09:37 UTC
(In reply to comment #4)
> Long letters gets cut because of
> 
>  h1, h2, h3, h4, h5, h6 { overflow: hidden; }
> 

I'm not sure, but I think that isn't the entire issue. Even if the overflow is set to visible, the characters will probably cut through the line below due to font renderings, and for lower headings, they might even cover the plaintext line below. This problem was previously handled in bug 32826 and bug 30809 via setting the line-height for langs that needed them. However, that doesn't work on multilingual projects since the lang-tags are not there for each heading individually (this is Bug 34514 ). I don't think simply setting the overflow will solve the problem if the line-height isn't set too.
Comment 6 Fomafix 2012-05-07 14:38:56 UTC
Of course
 #firstHeading { overflow: visible; }
doesn't prevent cutting letters entire. But it reduces the cutting.

The existing h1#firstHeading { overflow: hidden; } is not necessary and can be certainly reseted to the default h1#firstHeading { overflow: visible; } without disadvantages.

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


Navigation
Links