api review feedback Nov 11th, 2026

オープン
#44,062 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

@luizpaulo-antunes がすでに取り組んでいます。

2026年11月12日 から。

評価

難易度
3/5
見積もり時間
1〜2日
初心者へのやさしさ
70/100
issue の種類
ドキュメント
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
markdown, playwright, typescript
領域
documentation

調査の方向性

Update the API descriptions in docs/src/api/params.md for screenshot animations, locateOnlyVisible, and conditional retries, plus the locators guide's “Matching only visible elements” section and release notes. Start by reading the linked PRs and the existing documentation sections to confirm the intended behavior. Done when all three documentation requests and the screenshot baseline release note are covered.

索引モデルが issue の本文から書いたものです。

説明

API review items

Playwright 1.66 API review, range v1.65.0..main (head 3b438229c), snapshot 2026-11-11.

1. Screenshots with animations: "disabled" leave scroll-driven animations unchanged

PR #44022, Issue #43986

  • instanceof ScrollTimeline
    https://github.com/microsoft/playwright/pull/44061
  • Docs: The animations option text still says that finite animations are fast-forwarded and infinite animations are canceled. Add to both variants in docs/src/api/params.md that scroll-driven and view-driven animations are left unchanged, and add a release notes line that screenshot baselines of such pages can change.

3. locateOnlyVisible context option

PR #43903, Issue #43855

  • apply auto-visible before nth(), but after has:

  • check that we can opt-out with .visible(false|true|'any')

  • Docs: Add to the option description in docs/src/api/params.md that first(), last() and nth() select the element before the visibility filter applies, with the .visible().first() pattern, and mention the option in the "Matching only visible elements" section of the locators guide.

5. Conditional retries: retries accepts a list of { count, condition }

PR #43929, Issue #43917

  • make sure that TestCase.retries is updated before onTestEnd() is called

  • Docs: State in the "Conditional retries" section and in TestConfig.retries that the condition is matched against the error message with its color codes, unless the code is changed to remove them first. State in the same section that the --retries command line option replaces the list.

主要言語
TypeScript
スター
99.7k
フォーク
6.8k
平均マージ
16時間 10分
マージ済み PR(30日)
301

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

microsoft/playwright のほかの issue

microsoft/playwright の issue をすべて見る

似ている issue

TypeScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。