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

first draft of accessibility device section #158

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

pes10k
Copy link
Contributor

@pes10k pes10k commented May 21, 2024

fixes ##157

First go at adding a new question to discuss risks of exposing accessibility devices, and that spec authors should minimize as much as possible this kind of information exposure


Preview | Diff

@pes10k pes10k requested a review from hober May 21, 2024 21:49
@pes10k pes10k self-assigned this May 21, 2024
@pes10k
Copy link
Contributor Author

pes10k commented May 21, 2024

@hober heres a first go at text for this question. I fought with bikeshed locally for a while, and can't even seem to get current main HEAD to build, so this might need bikeshed tweaking. But grateful for any thoughts you might have on the content

@hober
Copy link
Contributor

hober commented May 24, 2024

I fought with bikeshed locally for a while, and can't even seem to get current main HEAD to build, so this might need bikeshed tweaking.

Fixed on main. Please rebase! :)

@pes10k pes10k force-pushed the add-accessibility-device-question branch from 7f12486 to 69b9e4d Compare May 24, 2024 21:16
@pes10k pes10k force-pushed the add-accessibility-device-question branch from 69b9e4d to 1ee702f Compare May 24, 2024 21:17
@pes10k
Copy link
Contributor Author

pes10k commented May 24, 2024

done and done!

Also, should we be tracking index.html? Guessing not since there isn't one in the repo. If thats right, mind if I .gitignore it?

@hober
Copy link
Contributor

hober commented May 31, 2024

done and done!

Also, should we be tracking index.html? Guessing not since there isn't one in the repo. If thats right, mind if I .gitignore it?

the Makefile lands index.html in the build/ subdirectory, which is already in the .gitignore file.

@pes10k
Copy link
Contributor Author

pes10k commented May 31, 2024

Ah, not sure why but somehow i got a index.html in the root of the repo. I assumed it was from the build script (maybe a previous version?), but i see thats not the case when i check now. Anyway, all good, and nvm

Copy link
Contributor

@hober hober left a comment

Choose a reason for hiding this comment

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

looks good! here are some nits.

@@ -810,6 +810,52 @@ You may follow the guidelines for <a href="bfcache">BFCache</a> mentioned above,
as we expect BFCached and detached documents to be treated the same way,
with the only difference being that BFCached documents can become [=Document/fully active=] again.

<h3 class=question id="accessibility-devices">
Does your feature allow sites to learn about the users use of accessibility devices?
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Does your feature allow sites to learn about the users use of accessibility devices?
Does your feature allow sites to learn about the users use of assistive technology?

Does your feature allow sites to learn about the users use of accessibility devices?
</h3>
The Web is designed to work for everyone, and Web standards should be designed
for people using assistive devices just as much as for users relying
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
for people using assistive devices just as much as for users relying
for people using assistive technology (<abbr title="assistive technology">AT</abbr>) just as much as for users relying

are core to the W3C's mission.

Standards authors though should keep in mind that Web users that rely on
assistive devices and technologies face some unique risks when using the Web.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
assistive devices and technologies face some unique risks when using the Web.
assistive technology face some unique risks when using the Web.

The use of assistive technologies may cause those Web users to stand
out among other Web users, increasing the risk of unwanted reidentification
and privacy harm. Similarly, some Web site operators may try to
discriminate against Web users who rely on assistive devices.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
discriminate against Web users who rely on assistive devices.
discriminate against Web users who rely on assistive technology.

and privacy harm. Similarly, some Web site operators may try to
discriminate against Web users who rely on assistive devices.

Feature designers and standards authors should therefor be thoughtful and
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Feature designers and standards authors should therefor be thoughtful and
Feature designers and <abbr title=specification>spec</abbr> authors should therefore be thoughtful and

mouse, keyboard, or touch screen.

<p class=example>
The [[wai-aria-1.3]] defines additional markup site authors can use to make
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
The [[wai-aria-1.3]] defines additional markup site authors can use to make
The [[wai-aria-1.3]] defines additional markup authors can use to make


<p class=example>
The [[wai-aria-1.3]] defines additional markup site authors can use to make
their pages easier to navigate with accessibility devices. The standard
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
their pages easier to navigate with accessibility devices. The standard
their pages easier to navigate with assistive technology. The <abbr>spec</abbr>

<p class=example>
The [[wai-aria-1.3]] defines additional markup site authors can use to make
their pages easier to navigate with accessibility devices. The standard
includes the an [`aria-hidden`](https://w3c.github.io/aria/#aria-hidden)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
includes the an [`aria-hidden`](https://w3c.github.io/aria/#aria-hidden)
includes the [`aria-hidden`](https://w3c.github.io/aria/#aria-hidden)

their pages easier to navigate with accessibility devices. The standard
includes the an [`aria-hidden`](https://w3c.github.io/aria/#aria-hidden)
attribute, that site authors can use to indicate that certain content
should be hidden from assistive devices.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
should be hidden from assistive devices.
should be hidden from assistive technology.


A malicious site author might
abuse the `aria-hidden` attribute to learn if a user is using assistive
technology, possibly by revealing certain page content to assistive devices,
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
technology, possibly by revealing certain page content to assistive devices,
technology, possibly by revealing certain page content to assistive technology,

A malicious site author might
abuse the `aria-hidden` attribute to learn if a user is using assistive
technology, possibly by revealing certain page content to assistive devices,
while showing very different page content to other users. A malicious

Choose a reason for hiding this comment

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

Similar things are true for CSS feature detection for reduced motion or high contrast. Worth noting?

Copy link

@csarven csarven left a comment

Choose a reason for hiding this comment

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

An example of explicit information about the accessibility device being revealed is through the User-Agent header field. This is a broader concern:

#160

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

Successfully merging this pull request may close these issues.

None yet

4 participants