Skip to content
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

MQ5 User Prefs: prefers-reduced-transparency #1709

Merged
merged 2 commits into from
Aug 11, 2017
Merged

MQ5 User Prefs: prefers-reduced-transparency #1709

merged 2 commits into from
Aug 11, 2017

Conversation

cookiecrook
Copy link
Contributor

Discussed at recent F2F meeting, and in CSS Issue #443

@inoas
Copy link

inoas commented Aug 10, 2017

Aside that … the MQ feature name never (as in not here nor with prefers-reduced-motion) should have included the direction of the scale and an implication that you cannot request prefers-disabled-transparency etc. … this is great and will mean authors can help those visually impaired.

@cookiecrook
Copy link
Contributor Author

cookiecrook commented Aug 10, 2017

@inoas wrote:

the MQ feature name never should have included the direction of the scale and an implication that you cannot request prefers-disabled-transparency

We talked about this in the other thread. It could always be expanded at a later date with values that still make sense: For example…

prefers-reduced-transparency: [ none |  reduce | force | opaque | bikeshed]

this is great and will mean authors can help those visually impaired.

👍

Copy link
Collaborator

@frivoal frivoal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generally looks good, thanks. Will merge as soon as the editorial change I requested on the section title is addressed.

@@ -478,6 +478,36 @@ Detecting the desire for less motion on the page: the 'prefers-reduced-motion' f

</dl>

<h3 id="prefers-reduced-transparency">
Detecting the desire to reduce the use of transparent or translucent layering effects on the page: the 'prefers-reduced-transparency' feature</h3>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you shorten this title a bit? This a bit unwieldy, and does not need to reproduce the whole content of the section. How about:

Detecting the desire for reduced transparency on the page: the 'prefers-reduced-transparency' feature

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Transparency is a boolean concept; translucency and opacity are not. If the feature name contains transparency for the sake of easy vs more correct language, then the docs can easily contain translucent and/or opacity if authors are search the docs for those (more correct) terms.

But - as long as @cookiecrook hadn't had more specifics in mind - skipping the layering effects vs just page 👍

Detecting the desire for reduced transparency or translucency on the page: the 'prefers-reduced-transparency' feature

Copy link
Collaborator

@frivoal frivoal Aug 10, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Transparency is a boolean concept; translucency and opacity are not.

I'm affraid I'll disagree on this one. I check the dictionary, and it gave not hit of this nuance. https://encrypted.google.com/search?q=partially+transparent&tbm=isch gives plenty of relevant results. Apple used this phrasing in their System Preferences to express the same concept. Using the word transparency is fine.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, but clearly many UI effects nowadays combine a certain levels of opacity/transparency with certain levels of translucency (see example graphics on https://en.wikipedia.org/wiki/Transparency_and_translucency) and as long as this MQ should detect a combined preference then transparency for the name and translucency as part of the docs seems reasonable. My guess is still that apple picked "transparency" as that is an easy concept to get to every user.

opacity is btw the key word in CSS used to describe (non boolean shades of) transparency of layers, not transparency.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agree that the standard English usage of all of these related terms are all roughly equivalent, and all encompass degrees.

For example, I was taught that opaque/translucent/transparent refer to the degree of opacity: something is translucent when light passes thru, but it's not clear enough to actually see details on the far side; something is transparent when you can actually see thru it to at least some degree. (The boundary is obviously very fuzzy, no pun intended, but the categories are clear enough, no pun intended. "Privacy glass" is more on the translucent side, cloudy plastic is translucent, etc. Normal windows are transparent. Rice paper is translucent.)

In other words, the definitions are definitely intertwined and uncertain enough that we can really use any of them here; there's no way to defend any particular definition as Definitely Correct.

@frivoal frivoal self-assigned this Aug 10, 2017
Copy link
Collaborator

@frivoal frivoal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@frivoal frivoal merged commit 1d7f755 into w3c:master Aug 11, 2017
@frivoal
Copy link
Collaborator

frivoal commented Aug 11, 2017

@grorg You may be interested in this thing I just merged here. If something's wrong, please open a new issue, thx.

@cookiecrook cookiecrook deleted the mq5/prefers-reduced-transparency branch August 11, 2017 23:24
@cookiecrook
Copy link
Contributor Author

Thanks @frivoal

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants