Improve Alert Behavior on Sketch Collection Deletion
メンテナーはふだん 3 日以内に返信
@SinhaS12 がすでに取り組んでいます。
2026年2月26日 から。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 48/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- javascript
調査の方向性
コレクションビューの「…」メニューと削除アクションから始め、ネイティブブラウザー確認または window.confirm() が呼び出されている箇所をたどります。プロジェクトに既にあるモーダル／ダイアログコンポーネントを読み、そのアクセシビリティの挙動を確認します。削除が、安全なデフォルトフォーカス、キーボード操作、フィードバック、重複送信に対する保護を備えた、アクセシブルでコンテキストに応じた確認モーダルを使用するようになれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Increasing Access
Increasing Access
Using a native browser alert for collection deletion creates an inconsistent and sometimes less accessible experience for keyboard and screen-reader users. Replacing it with an in-app confirmation modal allows proper focus management, clearer context about what will be deleted, and more consistent messaging across the editor.
Feature request details
Increasing Access
Currently, when deleting a sketch collection, the editor shows a native browser confirmation alert (e.g. window.confirm()).
This creates multiple issues:
- The alert UI is inconsistent with the editor’s design system.
- The message provides limited context (what exactly is being deleted, and what the impact is).
- The interaction can be less accessible than an in-app dialog (focus management, keyboard flow, screen reader context).
- The text may be harder to localize/standardize compared to app UI components.
Current Behaviour
- Navigate to a collection / sketches list.
- Trigger delete from the “…” menu.
- A native browser alert appears with Cancel / OK.
Image
(attach screenshot)
Feature enhancement details
- Replace native browser alert with an in-app confirmation modal
- Use the project’s modal/dialog component styling.
- Clear, contextual confirmation copy
- Example: “You are about to delete the collection ‘’. This action cannot be undone.”
- If applicable, include counts (e.g. number of sketches affected).
- Accessibility
- Proper dialog semantics (
role="dialog"/aria-modal="true") - Focus moves into the dialog and returns to the trigger on close
- Keyboard support (Tab/Shift+Tab, Esc to close)
- Default focus on the safest action (Cancel)
- Proper dialog semantics (
- Confirmation & feedback
- Show success/error feedback (toast or inline message) after completion.
- Disable buttons while delete is in progress to prevent double-submit.
Expected Behaviour
Deleting a sketch collection should use an accessible in-app confirmation modal with clear context and consistent UX (instead of a native browser alert).
Steps to reproduce
- Open the editor and go to My sketches / collections view.
- Open the “…” menu for a collection.
- Click Delete.
- Observe the native browser alert prompt.
- 主要言語
- JavaScript
- スター
- 1.7k
- フォーク
- 1.7k
- 平均マージ
- 3日 18時間
- マージ済み PR（30日）
- 7
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
processing/p5.js-web-editor のほかの issue
-
`PATCH /editor/project/visibility` returns 200 with `null` and doesn't update when `projectId` is a slug対応中かも @Prbhtsgh が今日担当しました。 オープンAwaiting Maintainer Approval Bug
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
processing/p5.js-web-editor#4348 · コメント 3 件 ·
メンテナーはふだん 3 日以内に返信
-
Fix: example.js stores defaultHTML function reference instead of calling it, causing examples to display raw JavaScript source in preview対応中かも @syedbarkath980 が 5 日前に担当しました。 オープンAwaiting Maintainer Approval Bug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
processing/p5.js-web-editor#4344 · コメント 1 件 ·
メンテナーはふだん 3 日以内に返信
-
signup form gets stuck when signup request fails due to network error対応中かも @PS01K が 34 日前に担当しました。 オープンAwaiting Maintainer Approval Bug
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
processing/p5.js-web-editor#4285 · コメント 3 件 ·
メンテナーはふだん 3 日以内に返信
-
saveProject throws an error when a network request fails対応中かも @dyk1454683243-sudo が 6 日前に担当しました。 オープンBug
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
processing/p5.js-web-editor#4276 · コメント 3 件 · 担当者 1 名 ·
メンテナーはふだん 3 日以内に返信
-
Awaiting Maintainer Approval Enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
processing/p5.js-web-editor#4270 · コメント 1 件 ·
メンテナーはふだん 3 日以内に返信
processing/p5.js-web-editor の issue をすべて見る
似ている issue
-
check:passed feeds:remove
難易度 1/5 1時間未満 初心者へのやさしさ 65/100
iptv-org/database#37176 · コメント 1 件 · リアクション 1 件 ·
メンテナーはふだん 9 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
hawk-digital-environments/HAWKI#443 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
メンテナーはふだん 1 日以内に返信
-
feedback simulation workshop
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
githubnext/gh-aw-workshop#4370 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
AltimateAI/vscode-dbt-power-user#2089 ·