Page MenuHomePhabricator

VisualEditor: [Regression pre-wmf17] Please round image sizes to whole numbers
Closed, ResolvedPublic


Wikitext code: [[File:Hedrick (1921) - Bartlett.jpg|thumb|Caption]]

Original image dimensions: 920 × 1,216 pixels.

VisualEditor says: 166.32302405498282 × 220 pixels.

Please round to whole numbers of pixels.

Version: unspecified
Severity: minor



Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 3:02 AM
bzimport set Reference to bz62279.

Created attachment 14746
Screen shot of VisualEditor's image dialog, under advanced settings


I can't manage to replicate this, not even in betalabs.

I get 166x220 in the media size widget...

I still can't manage to replicate the bug on master or on betalabs, but in any case, height calculations are rounded in this patch:

This should fix instances of this bug.

Apologies, I mean this fix:

(Added Math.round() in the default size conversion in the dm)

Change 117161 had a related patch set uploaded by Mooeypoo:
Enforce width-only default size only on larger images

Change 117161 merged by jenkins-bot:
Enforce width-only default size only on larger images