Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Correction: revise wbr and br element allowances#353

Merged
scottaohara merged 10 commits intogh-pagesfrom
332-wbr-and-br-element
Dec 8, 2021
Merged

Correction: revise wbr and br element allowances#353
scottaohara merged 10 commits intogh-pagesfrom
332-wbr-and-br-element

Conversation

@scottaohara
Copy link
Member

@scottaoharascottaohara commentedSep 23, 2021•
edited
Loading

closes#332

Revise the allowances for thebr element - only allowaria-hidden attribute, rather than all global attributes.

Revise allowances forwbr to matchbr. Neither of these elements allow for styling from authors, nor do they allow for child content (or content from CSS - again no styling). So allowing for any roles/attributes beyond presentation or for hiding would not be a wise idea.

Need at least two checkers to accept this change before we can merge.

html validator

  • role update
  • allowed attributes update

ibm equal access accessibility checker

  • role update
  • allowed attributes update

axe-core

  • role update
  • allowed attributes update

arc toolkit

  • role update
  • allowed attributes update

html validator and axe issues are still open in regards to limiting allowedaria-* attributes on these elements toaria-hidden alone.

Preview |Diff

@scottaoharascottaohara added the needs implementation commitmentCannot merge into spec until implementations in conformance checkers has been confirmed. labelSep 23, 2021
@scottaoharascottaohara added the Meets 2 implementations reqIndicates that a spec change has met the necessary 2 implementations needed to merge. labelOct 8, 2021
@scottaohara
Copy link
MemberAuthor

This can now be merged after the 1.0 spec officially meets Rec status.

@scottaoharascottaohara removed the Meets 2 implementations reqIndicates that a spec change has met the necessary 2 implementations needed to merge. labelOct 26, 2021
@scottaoharascottaohara changed the titlerevise wbr and br element allowancesCorrection: revise wbr and br element allowancesNov 8, 2021
@scottaoharascottaohara removed the needs implementation commitmentCannot merge into spec until implementations in conformance checkers has been confirmed. labelDec 8, 2021
@scottaohara
Copy link
MemberAuthor

As 3 of the 4 checkers have already updated the role allowances forwbr, and intent to implement the additionalaria-* allowances forbr andwbr has met no opposition, I'm going to merge this into the spec.

@scottaoharascottaohara merged commitcfab4e1 intogh-pagesDec 8, 2021
@scottaoharascottaohara deleted the 332-wbr-and-br-element branchDecember 8, 2021 01:06
scottaohara added a commit that referenced this pull requestMay 19, 2022
directly reference the following PRs in the changelog, and prepend each entry with the appropriate 'label'related to#419-#352-#367 (and follow on [commit](887c772))-#360-#353-#372-#402-#404-#403-#396-#391-#369-#381
pkra pushed a commit to w3c/aria that referenced this pull requestMay 20, 2024
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@marcoscaceresmarcoscaceresmarcoscaceres left review comments

@plehegarplehegarplehegar left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

wbr element

3 participants

@scottaohara@marcoscaceres@plehegar

Comments

[8]ページ先頭
©2009-2026 Movatter.jp