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

Rustdoc accessibility: use real headers for doc items #87390

Closed

Conversation

notriddle
Copy link
Contributor

@notriddle notriddle commented Jul 22, 2021

This is a pretty annoying workaround: the specifications literally
contradict each other on the matter of whether headers may be nested
inside of summaries. We need the headers for people who use outline
mode to browse rustdoc, mostly in screen readers, and the headers
need to be inside the summary tag so that they're visible when the
details tag is collapsed.

HTML5 says it's okay, but the ordinary button semantics in ARIA say
that headers nested in buttons are ignored, and summary is a button.
Which means some screen readers, like NVDA, work, while others,
like JAWS, don't.
@rust-highfive
Copy link
Collaborator

Some changes occurred in HTML/CSS/JS.

cc @GuillaumeGomez

@rust-highfive
Copy link
Collaborator

r? @CraftSpider

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 22, 2021
@notriddle notriddle force-pushed the notriddle/rustdoc-headers-patch branch from 1b7232f to 0147ce7 Compare July 22, 2021 22:48
@camelid camelid added the T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. label Jul 23, 2021
@ahicks92
Copy link
Contributor

To get ahead of the "can you check it please": yes, this works at least with my setup, and it looks like you just made my day.

Also apparently I missed the table of contents headers one if that's not part of this one, but that's also very nice.

@GuillaumeGomez
Copy link
Member

The notable tooltip is broken too:

Screenshot from 2021-07-23 11-52-17

@jyn514 jyn514 added the beta-nominated Nominated for backporting to the compiler in the beta channel. label Jul 23, 2021
notriddle and others added 2 commits July 23, 2021 07:10
Co-authored-by: Guillaume Gomez <guillaume1.gomez@gmail.com>
Co-authored-by: Guillaume Gomez <guillaume1.gomez@gmail.com>
@rust-log-analyzer

This comment has been minimized.

@notriddle
Copy link
Contributor Author

@GuillaumeGomez okay, I fixed the tooltip and all of the test cases.

@bors
Copy link
Contributor

bors commented Jul 25, 2021

@notriddle: 🔑 Insufficient privileges: Not in reviewers

@GuillaumeGomez
Copy link
Member

@bors: r+ squash

@bors
Copy link
Contributor

bors commented Jul 25, 2021

📌 Commit 7e888bd has been approved by GuillaumeGomez

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 25, 2021
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 25, 2021
…atch, r=GuillaumeGomez

Rustdoc accessibility: use real headers for doc items

Part of rust-lang#87059

Partially reverts rust-lang#84703

Preview at: https://notriddle.com/notriddle-rustdoc-test/real-headers/std/index.html
@bors
Copy link
Contributor

bors commented Jul 25, 2021

⌛ Testing commit 7e888bd with merge 5782f01...

@bors
Copy link
Contributor

bors commented Jul 26, 2021

☀️ Test successful - checks-actions
Approved by: GuillaumeGomez
Pushing 5782f01 to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Jul 26, 2021
@pietroalbini
Copy link
Member

This has been merged.

@GuillaumeGomez GuillaumeGomez added the beta-accepted Accepted for backporting to the compiler in the beta channel. label Jul 26, 2021
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 26, 2021
…lacrum

Prepare 1.54.0 release

This PR builds the stable artifacts for the 1.54.0 release. Backports included:

* rust-lang#86696
*  rust-lang#87167
* rust-lang#87210

I was *not* able to cherry-pick rust-lang#87390 as that didn't apply cleanly to the stable branch. `@GuillaumeGomez` `@notriddle` could it be possible to get a PR targeting `stable` backporting that fix?

Also, this **enables** incremental compilation on the stable channel.

r? `@Mark-Simulacrum`
cc `@rust-lang/release`
@pietroalbini pietroalbini added beta-accepted Accepted for backporting to the compiler in the beta channel. beta-nominated Nominated for backporting to the compiler in the beta channel. and removed beta-accepted Accepted for backporting to the compiler in the beta channel. beta-nominated Nominated for backporting to the compiler in the beta channel. labels Jul 27, 2021
@pietroalbini
Copy link
Member

Actually, this will also need to be backported to 1.55.0 beta as it was merged after the beta cutoff.

@pietroalbini
Copy link
Member

Included in #87508

@pietroalbini pietroalbini removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label Jul 27, 2021
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 28, 2021
…crum

Prepare beta 1.55.0

Included backports:

* rust-lang#86696
* rust-lang#87390 (squashed)

Reverted:

* rust-lang#86623

cc `@rust-lang/release` `@Mark-Simulacrum`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beta-accepted Accepted for backporting to the compiler in the beta channel. merged-by-bors This PR was explicitly merged by bors. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants