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

fix: silence autofocus a11y warning inside<dialog>#16341

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
Ocean-OS merged 1 commit intosveltejs:mainfromnavorite:dialog-a11y
Jul 11, 2025

Conversation

@navorite
Copy link
Contributor

@navoritenavorite commentedJul 11, 2025•
edited
Loading

closes#9225

Before submitting the PR, please make sure you do the following

  • It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC:https://github.com/sveltejs/rfcs
  • Prefix your PR title withfeat:,fix:,chore:, ordocs:.
  • This message body should clearly illustrate what problems it solves.
  • Ideally, include a test that fails without this PR but passes with it.
  • If this PR changes code withinpackages/svelte/src, add a changeset (npx changeset).

Tests and linting

  • Run the tests withpnpm test and lint the project withpnpm lint

shadow-identity reacted with rocket emoji
@changeset-bot
Copy link

🦋 Changeset detected

Latest commit:c02e9a5

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
NameType
sveltePatch

Not sure what this means?Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions
Copy link
Contributor

Playground

pnpm add https://pkg.pr.new/svelte@16341

@Ocean-OSOcean-OS mentioned this pull requestJul 11, 2025
6 tasks
@Ocean-OSOcean-OS merged commit96c1a10 intosveltejs:mainJul 11, 2025
10 checks passed
@Ocean-OS
Copy link
Member

thank you!

@github-actionsgithub-actionsbot mentioned this pull requestJul 11, 2025
@navoritenavorite deleted the dialog-a11y branchJuly 11, 2025 18:31
Ocean-OS added a commit that referenced this pull requestJul 11, 2025
Ocean-OS added a commit that referenced this pull requestJul 14, 2025
* chore: clean up a11y analysis code* lint* apply changes from#16340 and#16341* `utils.js` -> `index.js`* remove unused exports* remove unused exports* newlines are free :)* consolidate loops* put the exported function up top---------Co-authored-by: Rich Harris <rich.harris@vercel.com>
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@Ocean-OSOcean-OSOcean-OS approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Wrong a11y wargning on <dialog> with autofocus attribute

2 participants

@navorite@Ocean-OS
[8]ページ先頭
©2009-2025 Movatter.jp