Last modified: 2014-05-02 23:21:29 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 T66599, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 64599 - VisualEditor: [Regression] Content has too small font-size and squished line-height in MediaWiki Vector
VisualEditor: [Regression] Content has too small font-size and squished line-...
Status: VERIFIED FIXED
Product: VisualEditor
Classification: Unclassified
MediaWiki integration (Other open bugs)
unspecified
All All
: High major
: VE-deploy-2014-05-01
Assigned To: Krinkle
: code-update-regression
Depends on: 63985
Blocks:
  Show dependency treegraph
 
Reported: 2014-04-29 11:53 UTC by Krinkle
Modified: 2014-05-02 23:21 UTC (History)
6 users (show)

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


Attachments
Screenshot of view and edit mode side-by-side (182.86 KB, image/png)
2014-04-29 11:53 UTC, Krinkle
Details

Description Krinkle 2014-04-29 11:53:27 UTC
Created attachment 15237 [details]
Screenshot of view and edit mode side-by-side

Ever since the typography update, the content looks malformed in the editor.

This is because the way the typography change in Vector was implemented, used rather specific selectors that cannot apply to VisualEditor's surface and thus it cascades in ways it wasn't supposed to.

The content line-height is squished, and the font size is too small.
Comment 1 Gerrit Notification Bot 2014-04-29 12:00:50 UTC
Change 129567 had a related patch set uploaded by Krinkle:
mw.ViewPageTarget: Use .mw-content for content styling

https://gerrit.wikimedia.org/r/129567
Comment 2 Gerrit Notification Bot 2014-04-29 22:24:00 UTC
Change 130507 had a related patch set uploaded by Jforrester:
mw.ViewPageTarget: Use .mw-body-content for content styling

https://gerrit.wikimedia.org/r/130507
Comment 3 Gerrit Notification Bot 2014-04-29 22:25:55 UTC
Change 129567 merged by jenkins-bot:
mw.ViewPageTarget: Use .mw-body-content for content styling

https://gerrit.wikimedia.org/r/129567
Comment 4 Gerrit Notification Bot 2014-04-29 23:26:05 UTC
Change 130507 merged by jenkins-bot:
mw.ViewPageTarget: Use .mw-body-content for content styling

https://gerrit.wikimedia.org/r/130507
Comment 5 James Forrester 2014-04-29 23:54:48 UTC
Still some mess here.
Comment 6 Gerrit Notification Bot 2014-04-30 00:42:01 UTC
Change 130541 had a related patch set uploaded by Krinkle:
mw.ViewPageTarget: Move .mw-body-content down to .ve-ce-surface

https://gerrit.wikimedia.org/r/130541
Comment 7 Krinkle 2014-04-30 20:21:52 UTC
Yeah, we fixed the page content font-size and line-height, but now those changes incorrectly apply to our overlays as well. This causes all our inspectors to have messed up line-heights and recursively smaller font-sizes inside e.g. link inspector suggestions and the context menu cursor.
Comment 8 Krinkle 2014-04-30 20:22:05 UTC
(In reply to Krinkle from comment #7)
> Yeah, we fixed the page content font-size and line-height, but now those
> changes incorrectly apply to our overlays as well. This causes all our
> inspectors to have messed up line-heights and recursively smaller font-sizes
> inside e.g. link inspector suggestions and the context menu cursor.

https://gerrit.wikimedia.org/r/130541 addresses this.
Comment 9 Gerrit Notification Bot 2014-04-30 21:16:51 UTC
Change 130541 merged by jenkins-bot:
mw.ViewPageTarget: Move .mw-body-content down to .ve-ce-surface

https://gerrit.wikimedia.org/r/130541
Comment 10 Gerrit Notification Bot 2014-04-30 21:29:39 UTC
Change 130745 had a related patch set uploaded by Catrope:
mw.ViewPageTarget: Move .mw-body-content down to .ve-ce-surface

https://gerrit.wikimedia.org/r/130745
Comment 11 Gerrit Notification Bot 2014-04-30 23:04:58 UTC
Change 130745 merged by Catrope:
mw.ViewPageTarget: Move .mw-body-content down to .ve-ce-surface

https://gerrit.wikimedia.org/r/130745
Comment 12 Rummana Yasmeen 2014-05-02 22:44:59 UTC
Verified the fix on Betalabs and test2
Comment 13 Rummana Yasmeen 2014-05-02 23:21:29 UTC
Verified the fix on production

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


Navigation
Links