Last modified: 2013-07-25 23:11:08 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 T52058, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 50058 - VisualEditor: Save dialog is far too small in Monobook, making edit summaries hard to write
VisualEditor: Save dialog is far too small in Monobook, making edit summaries...
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
MediaWiki integration (Other open bugs)
unspecified
All All
: High normal
: VE-deploy-2013-07-25
Assigned To: Ed Sanders
:
: 51034 51315 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-06-23 16:41 UTC by Oliver Keyes
Modified: 2013-07-25 23:11 UTC (History)
9 users (show)

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


Attachments
Screenshot (33.36 KB, image/png)
2013-06-23 16:41 UTC, Oliver Keyes
Details
Review changes popup (15.01 KB, image/png)
2013-07-22 14:00 UTC, Kyle G
Details

Description Oliver Keyes 2013-06-23 16:41:00 UTC
Created attachment 12615 [details]
Screenshot

See the screenshot; it appears in Monobook on enwiki and mediawiki.org as of the latest release. Kiiinda a problem.

(the other interstitials there - review your changes/save page - are also shrunk).
Comment 1 James Forrester 2013-06-23 16:44:08 UTC
Timo, what have you wrought?! ;-)
Comment 2 Krinkle 2013-06-23 22:45:47 UTC
This is due to Monobook's "interesting" way of resetting the font-size to a consistent value cross-browser. It sets the <body> font-size to x-small, and then turn it back up to regular size with "font-size: 127%;" from their globalWrapper div.

Due to this odd construct, any new elements attached to the <body> have an exceptionally small font size.

Short of hardcoding a larger font-size for the overlay wrappers in Monobook or appending to #globalWrapper in monobook (which might cause others bugs due to not being in the <body> directly) - short of that this is a tough one to get right. Ideally core would fix this in Monobook by using the same font-size technique used by Vector.

In Monobook's author's defence (Gabriel), the browsers we had to support when Monobook was being developed made it quite difficult to do right so this was probably the best way to do it at the time – but we no longer need to worry about IE5.5, IE4-for-Mac, Opera<7 etc.
Comment 3 Gabriel Wicke 2013-06-24 19:58:26 UTC
You could add a CSS rule that uses the same technique for the dialog.

Setting a simple relative font size (0.7em or something like that) makes it impossible to set the browser's default size to something sensible, as 0.7 * something sensible is usually too small. The non-linear nature of keyword scaling gives a good compromise between honoring a browser's font size and providing sane defaults despite 16px browser defaults.

So if there are other JS-free solutions to this issue these days then that would be great.
Comment 4 This, that and the other (TTO) 2013-07-10 07:46:46 UTC
*** Bug 51034 has been marked as a duplicate of this bug. ***
Comment 5 This, that and the other (TTO) 2013-07-14 12:38:37 UTC
*** Bug 51315 has been marked as a duplicate of this bug. ***
Comment 6 Spinningspark 2013-07-14 13:05:16 UTC
Can I suggest that the title of this bug is changed to something more meaningful?  If the words "edit summary" were included there would probably be fewer duplicatates reported.
Comment 7 James Forrester 2013-07-14 13:07:33 UTC
(In reply to comment #6)
> Can I suggest that the title of this bug is changed to something more
> meaningful?  If the words "edit summary" were included there would probably
> be fewer duplicatates reported.

Done.
Comment 8 Kyle G 2013-07-22 14:00:53 UTC
Created attachment 12915 [details]
Review changes popup

The review changes popup also has the same sizing issues...

My eyes strain so hard trying to read the lines!
Comment 9 Gerrit Notification Bot 2013-07-24 23:13:39 UTC
Change 75814 had a related patch set uploaded by Esanders:
Fix font size for overlays in Monobook

https://gerrit.wikimedia.org/r/75814
Comment 10 Elitre 2013-07-24 23:20:07 UTC
I'd add that even on en.wp Monobook is used by many (at least if I read correctly  http://en.wikipedia.org/wiki/Wikipedia:Database_reports/User_preferences#Skin ).
I had quite a few complaints about this now at it.wp, so it would really great if we managed to fix this ASAP. Thanks!

(confl. by Gerrit: fixed now?)
Comment 11 Gerrit Notification Bot 2013-07-25 22:38:46 UTC
Change 75814 merged by jenkins-bot:
Fix font size for overlays in Monobook

https://gerrit.wikimedia.org/r/75814
Comment 12 James Forrester 2013-07-25 23:11:08 UTC
Now fixed in master and will be deployed in a few minutes' time.

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


Navigation
Links