-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
pHYs is metric-only #5
Comments
Looking at 11.3.5.3 pHYs Physical pixel dimensions I suspect this comment has misunderstood the use of
Also, the resolution in My feeling is that this chunk is not used very much, and when it is, it is to capture non-square pixels (for example, to archive images from some other format). However, the inability to accurately capture a physical resolution in Imperial units is well put, and possibly should be corrected or another chunk introduced to do this, if there is a need. |
I believe you are correct that it is perhaps rarely used. And when it is, it is probably mostly used to specify non-square pixels. But that linked spec does say:
(emphasis my own) If I think I am leaning toward adding an inch unit. What are your thoughts? |
We probably should (and add required handling for unknown unit specifiers). In addition that section really needs a worked example, like: 72dpi 72 ÷ 25.4 × 1,000 = 2,834.64566929 = 2835 Before doing so it would be good to run some tests to see what image display systems do with non-square pixels or with physical dimensions. PngSuite has a few tests:
results show browsers ignore non-square pixels and ignore physical dimensions. Need to check against CSS and HTML specs to see correct handling there. |
Raised by: Dirk Schenkewitz
Originally raised: 21 Oct 2005
The text was updated successfully, but these errors were encountered: