Page MenuHomePhabricator
Create Task
Maniphest T15650

Decimal fractions in image sizes are parsed incorrectly (ignored entirely instead of truncated)
Open, LowPublic
Actions

Description

Images with a width containing a decimal dot are parsed incorrectly, width the dot being disregarded (so that [[Image:Foo|3.25px]] will be 325 pixel wide). See for example: http://hu.wikipedia.org/w/index.php?title=User_vita:Tgr&diff=prev&oldid=3050910

Using fractions in image sizes does not make much sense, nevertheless it is used by a number of people (especially in signatures), and when a lot of images suddenly increase by one or two orders of magnitude, the result is not pretty. (Note also that CSS does allow for such sizes: http://www.w3.org/TR/REC-CSS2/syndata.html#value-def-length )

Version: unspecified
Severity: normal

Details

Reference
bz13650

Event Timeline

• bzimport raised the priority of this task from to Low.Nov 21 2014, 10:07 PM
• bzimport added a project: MediaWiki-File-management.
• bzimport set Reference to bz13650.
• bzimport added a subscriber: Unknown Object (MLST).
Tgr created this task.Apr 8 2008, 9:47 AM
Tgr added a comment.Apr 8 2008, 12:23 PM

Correction: it does not only disregard the decimal dot, but the whole width parameter. The image appears at its original size.

• brooke added a comment.Apr 9 2008, 8:22 PM

Well, that'd be because it's invalid input -- now that we have improved the input validation, it's rejected.

We could perhaps allow them; they'd still get rounded off of course.

• Gilles added a project: Multimedia.Nov 24 2014, 3:20 PM
matmarex renamed this task from decimal fractions in image sizes are parsed incorrectly to Decimal fractions in image sizes are parsed incorrectly (ignored entirely instead of truncated).Aug 30 2015, 10:37 PM
matmarex edited projects, added MediaWiki-Parser; removed Multimedia, MediaWiki-File-management.
matmarex set Security to None.
matmarex removed a subscriber: • wikibugs-l-list.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 30 2015, 10:37 PM
matmarex subscribed.Aug 30 2015, 10:38 PM

This is still valid, although the linked example doesn't exhibit the issue because the file was changed since this bug was filed.

• gpaumier unsubscribed.Jul 18 2018, 6:02 PM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits