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

Amend definition of motion animation to *not* exclude blurring #4040

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

patrickhlauke
Copy link
Member

@patrickhlauke patrickhlauke commented Aug 25, 2024

Closes #3949


Preview | Diff

Copy link

netlify bot commented Aug 25, 2024

Deploy Preview for wcag2 ready!

Name Link
🔨 Latest commit 9ba45ac
🔍 Latest deploy log https://app.netlify.com/sites/wcag2/deploys/66db196ab22f2f0008d02f0d
😎 Deploy Preview https://deploy-preview-4040--wcag2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@bruce-usab
Copy link
Contributor

Discussed on TF call. IMHO this is a nice minimal edit for the issue raised.

@detlevhfischer
Copy link
Contributor

This is a bit odd in that the blurring from the PR title is nowhere mentioned in the amended text (opacity and color change is). As to blurring, one could also argue that transitions from sharp to strong blur to sharp and back to sharp etc. could be quite distracting if not just an initial effect, the more so the large the image is. Admittedly haven't seen that out there yet.

@patrickhlauke
Copy link
Member Author

This is a bit odd in that the blurring from the PR title is nowhere mentioned in the amended text (opacity and color change is)

before, the list of excluded effects included blurring. this PR removes the mention of blurring, so does what the PR title says?

@detlevhfischer
Copy link
Contributor

@patrickhlauke argh... not sure how I could misread this.

@mraccess77
Copy link

My two cents - blurred images give me a headache and I am not able to look at them. This is a big issue with blurred remote meeting backgrounds but also some styles of photography.

@patrickhlauke
Copy link
Member Author

My two cents - blurred images give me a headache and I am not able to look at them. This is a big issue with blurred remote meeting backgrounds but also some styles of photography.

@mraccess77 while that is a wider, more general topic, this proposed change here should be welcome for you then because it does now mean that where blurring is used on interaction, it can now fail Animation from Interactions (while before it seemed to imply that blurring was exempt, just like changes in colour and opacity)

one word change to strengthen phrase
Copy link

@cookiecrook cookiecrook left a comment

Choose a reason for hiding this comment

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

Adding in my original proposed text that clarifies the it's not about screen motion in a vacuum, but about the perspective of the primary user. IMO, clarifying this is critical in the context of emerging modalities like XR, in which motion may be perceivable from another observable camera angle, but not the primary.

<p>addition of steps between conditions to create the illusion of movement or to give a sense of a smooth transition</p>
<aside class="example"><p>For example, an element which moves into place or changes size while appearing is considered to be animated. An element which appears instantly without transitioning is not using animation. Motion animation does not include changes of color, blurring, or opacity which do not change the perceived size, shape, or position of the element.</p></aside>

<aside class="example"><p>For example, an element which moves into place or changes size while appearing is considered to be animated. An element which appears or changes instantly, without transition/animation steps, is not using animation.</p><p>Motion animation does not include changes – such as changes of color or opacity – that do not alter the perceived size, shape, position, or distance/depth of the element.</p></aside>

Choose a reason for hiding this comment

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

Suggested change
<aside class="example"><p>For example, an element which moves into place or changes size while appearing is considered to be animated. An element which appears or changes instantly, without transition/animation steps, is not using animation.</p><p>Motion animation does not include changes – such as changes of color or opacity – that do not alter the perceived size, shape, position, or distance/depth of the element.</p></aside>
<aside class="example"><p>For example, an element which moves into place or changes size while appearing is considered to be animated. An element which appears or changes instantly, without transition/animation steps, is not using animation.</p><p>Motion animation does not include changes – such as changes of color or opacity – that do not alter the perceived size, shape, position, or distance/depth of the element to the viewer.</p></aside>

Copy link
Member Author

Choose a reason for hiding this comment

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

i'll admit, that reads a bit stilted to me. isn't it already implicit in the fact that this talks about "perceived..." as it's clearly being perceived by a user/viewer?

Choose a reason for hiding this comment

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

Agree, the use of perceived makes "to the viewer" redundant.

Copy link
Contributor

Choose a reason for hiding this comment

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

In the context of cookiecrook's comment: "this is critical in the context of emerging modalities like XR, in which motion may be perceivable from another observable camera angle, but not the primary."
I can see why it isn't as redundant as it appears from just the code change.

@bruce-usab
Copy link
Contributor

Discussed on backlog call 9/6.

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