Issue metadata
Sign in to add a comment
|
Should not apply `minimum font size` to <canvas>
Reported by
zcyzcy88...@gmail.com,
Jun 7 2017
|
||||||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:53.0) Gecko/20100101 Firefox/53.0 Example URL: Steps to reproduce the problem: 1. turn on `minimum font size` in setting 2. visit <canvas> text (example: https://codepen.io/anon/pen/bRVdGV) What is the expected behavior? What went wrong? <canvas> text applied `minimum font size`, so they inflated. Although a11y is important, it is break the expect font size. Imagine here is a <canvas> Photoshop, I try add text to picture, I can't make text small than `minimum font size`. Does it occur on multiple sites: Yes Is it a problem with a plugin? No Did this work before? No Does this work in other browsers? Yes Chrome version: all Channel: n/a OS Version: all Flash Version: This also cause Issue 720207 .
,
Jun 7 2017
,
Jun 8 2017
,
Jun 8 2017
Able to reproduce the issue on Windows 10, Ubuntu 14.04 and Mac 10.12.5 with chrome #59.0.3071.86 Canary #61.0.3123.0 and also in earlier version #30.0.1549.0, hence marking it as untriaged. Attaching a screen-cast for reference
,
Jun 12 2017
,
Jun 12 2017
This is technically a feature request, and the only way we would implement this, I think, is to somehow be able to disable the minimum font. There are use cases where the minimum font size should still kick in. Accessibility, could you indicate whether you think the original request is an acceptable thing to do? Marking Needs-Feedback until Accessibility weigh in.
,
Jun 12 2017
Hmmm, this just seems like a bug to me. Canvas is a pixel API, I feel like it should be more deterministic and shouldn't be affected by system settings, especially a minimum like this that can't be overridden by the author. I think that <canvas> is already outside the scope of what we can control accessibility for. Authors are certainly able to make a canvas-based UI accessible with some extra effort, but there's almost nothing we can do if the author doesn't want to do that. Note that text drawn into a canvas by default won't be selectable, won't be accessible to screen readers, won't be speakable via select-to-speak features (macOS and Chrome OS both have this), and won't be affected by high contrast mode on browsers that respect system colors like Edge. I'm okay with that because it's a low-level API and authors shouldn't be using it for accessible text just like they shouldn't be using Flash or images for text.
,
Jun 13 2017
,
Jun 13 2017
I agree with dmazzoni@. Besides the notion of enforcing a minimum font size is sort of futile in the first place because font style resolution (where the minimum is applied) happens upstream of the application of the 2D context's transform, so text can be scaled arbitrarily anyways.
,
Jun 14 2017
Issue 727428 has been merged into this issue.
,
Jun 14 2017
Fixed it! There's still the possibility of min font being applied in the case that the font size specified on Canvas is proportional to the parent element. In that case, the parent element will still be bounded by min font size, in which case we will inherit it. That said, for most cases - where the size is directly specified - we will respect that and completely ignore min font size. :) The CL is under review. Once it gets on, you can probably try it on canary.
,
Jun 20 2017
Gerrit seems to be a bit shy on posting bug emails, but here it is: https://chromium-review.googlesource.com/c/535275/ Feel free to check on canary in a couple days to make sure it's working. But it's working. :) it's interesting that this bug has been around for so long but never reported. But here we are. Donerino! Thanks for reporting.
,
Jun 26 2017
The NextAction date has arrived: 2017-06-26 |
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by dtapu...@chromium.org
, Jun 7 2017