Last modified: 2009-07-25 21:16:07 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 T21225, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 19225 - width for td.mw-label causes issues
width for td.mw-label causes issues
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Interface (Other open bugs)
1.16.x
All All
: Normal trivial (vote)
: ---
Assigned To: Andrew Garrett
http://en.wikipedia.org/wiki/Special:...
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-06-15 19:39 UTC by Ahmad Sherif
Modified: 2009-07-25 21:16 UTC (History)
3 users (show)

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


Attachments
screenshot of the problem (68.09 KB, image/png)
2009-06-15 19:39 UTC, Ahmad Sherif
Details
AllPages screenshot (82.63 KB, image/png)
2009-06-15 19:40 UTC, Ahmad Sherif
Details
Prefs with 20% width (46.01 KB, image/png)
2009-06-15 19:56 UTC, Siebrand Mazeland
Details
Prefs with 30% width (47.15 KB, image/png)
2009-06-15 19:56 UTC, Siebrand Mazeland
Details
proposed patch (533 bytes, patch)
2009-06-18 09:45 UTC, Ahmad Sherif
Details
Special:Upload screenshot (35.22 KB, image/png)
2009-06-21 09:49 UTC, Tisza Gergő
Details

Description Ahmad Sherif 2009-06-15 19:39:13 UTC
Created attachment 6225 [details]
screenshot of the problem

in r51868, the value of width was changed from 20% to 30% for some adjustment and stuff. Unfortunately, this cases some labels as in Special:RecentChangesLinked to look over-margined as you can see in this attachment.

Another thing, changing this value was because 'at' in "Display pages starting at:" (Special:AllPages) appears in new line. Well, 'at' still there in a new line on my machine. And when changing the value to 25% the sentence appears in single line.

So, 'at' problem seems to be machine-dependent somehow, and it causes some visual problems the same time.

I suggest reverting to the old value.
Comment 1 Ahmad Sherif 2009-06-15 19:40:41 UTC
Created attachment 6226 [details]
AllPages screenshot

Screenshot of Special:AllPages to show the position of 'at'
Comment 2 Siebrand Mazeland 2009-06-15 19:45:40 UTC
It seems neither is a good solution, nor is changing it to 25%, because other forms will get issues. Getting rid of it altogether, as it was before r49790 is also an option[1]. According to the designer of the revised preferences, Special:Preferences may suffer, though. Which is the least of all evils, or what is a proper solution?

Adding Andrew to cc.

[1] r1=49544&r2=49790&pathrev=51868">http://svn.wikimedia.org/viewvc/mediawiki/trunk/phase3/skins/common/shared.css?r1=49544&r2=49790&pathrev=51868
Comment 3 Andrew Garrett 2009-06-15 19:48:16 UTC
The columns should be the same width, and it should be reasonably narrow. It is hideous on some pages when there are multiple form tables and the columns have different widths. It is also hideous when there is a long description.
Comment 4 Siebrand Mazeland 2009-06-15 19:56:06 UTC
Created attachment 6227 [details]
Prefs with 20% width

Prefs with 20% width. Also uploading 30% width. Neither will win a design competition. Screenshot in Windows Xp with Firefox 3.0.11
Comment 5 Siebrand Mazeland 2009-06-15 19:56:30 UTC
Created attachment 6228 [details]
Prefs with 30% width
Comment 6 Siebrand Mazeland 2009-06-18 08:52:24 UTC
Is there a way to change the mw-label to mw-label-preference use 20/25/30% in preferences only, and revert to the old behaviour in the rest of core (no width)? Also got a report that Special:Email is affected.
Comment 7 Ahmad Sherif 2009-06-18 09:45:14 UTC
Created attachment 6238 [details]
proposed patch

(In reply to comment #6)
> Is there a way to change the mw-label to mw-label-preference use 20/25/30% in
> preferences only, and revert to the old behaviour in the rest of core (no
> width)? Also got a report that Special:Email is affected.
> 

Or something like this?
Comment 8 Tisza Gergő 2009-06-21 09:49:30 UTC
Created attachment 6243 [details]
Special:Upload screenshot

Special:Upload looks quite horrible too. It is a bad idea to apply the same spacing to a number of different interfaces.
Comment 9 Siebrand Mazeland 2009-06-21 10:20:08 UTC
Assigning to Andrew. Proposed patch looks OK, doesn't it?
Comment 10 Andrew Garrett 2009-07-03 13:32:29 UTC
Patch applied with modifications in r52737.
Comment 11 RockMFR 2009-07-25 21:16:07 UTC
Why do tables have width: 100% applied to them? This broke stuff on enwiki's prefs page (see [[en:MediaWiki:Gadgets-prefstext]]).

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


Navigation
Links