"Try again" on a report run that failed to load is a span, not a button
メンテナーはふだん 1 日以内に返信
関連するプルリクエストがすでにマージされています。
- #181 @CRYPTONIKAV による — マージ済み
評価
- 難易度
- 1/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 91/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- javascript, react
調査の方向性
website/src/ReportsView.jsxで、issueの指定どおり「Try again」要素を更新してください。website/test/clickableContent.test.mjsにある既存のアサーションパターンを読み、ボタンとしてのセマンティクスとフォーカス可能性を確認するアサーションを追加してください。該当するテストを実行してください。テストが成功し、コントロールが説明どおりの視覚的なスタイルを維持すれば完了です。
索引モデルが issue の本文から書いたものです。
説明
When a report run's details fail to load, Reports shows "This run's details could not be loaded. Try again". That "Try again" is a <Box component="span"> with an onClick, so you cannot reach it with the keyboard. Every other "Try again" in website/src is already a real button. This is the only span.
Where: website/src/ReportsView.jsx:
<Box component="span" onClick={() => setMissed(null)} sx={{ ... cursor: "pointer", textDecoration: "underline" }}>Try again</Box>
Done when it is component="button" type="button" with border: 0, bgcolor: "transparent", p: 0, font: "inherit" added to sx, so it looks the same but takes focus.
Test: an assertion in website/test/clickableContent.test.mjs.
Getting started: pip install -e ".[dev]", then python -m pytest -q (offline). UI work: cd website && npm ci && npm test. See CONTRIBUTING.md. One small PR with the change (and its test, where there is one) is perfect.
- 主要言語
- Python
- スター
- 116
- フォーク
- 22
- 平均マージ
- 14時間 6分
- マージ済み PR(30日)
- 50
環境構築
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
ldbumble/taskuary のほかの issue
-
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
メンテナーはふだん 1 日以内に返信
-
enhancement help wanted
難易度 5/5 1週間以上 初心者へのやさしさ 20/100
メンテナーはふだん 1 日以内に返信
-
bug
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
メンテナーはふだん 1 日以内に返信
-
bug connector
難易度 5/5 1週間以上 初心者へのやさしさ 20/100
メンテナーはふだん 1 日以内に返信
-
bug connector
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
メンテナーはふだん 1 日以内に返信
ldbumble/taskuary の issue をすべて見る
似ている issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
conda-forge/conda-forge-bot#7025 ·
メンテナーはふだん 1 日以内に返信
-
area:ci
難易度 1/5 1時間未満 初心者へのやさしさ 74/100
catincloud-labs/dagster-dex#149 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
home-assistant/core#188424 ·
メンテナーはふだん 1 日以内に返信
-
Review backlog (docs-currency): README 'CI run' link lands on a run whose overall badge is cancelledオープンagent/reviewer needs-kind needs-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 63/100
projectbluefin/utah#880 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
LSan reports obj_gen leak on post-parse usage() exits (aarch64)対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
redis/memtier_benchmark#850 ·
メンテナーはふだん 1 日以内に返信