api review feedback Nov 11th, 2026

Aperta
  • Qualcosa non va?
#44,062 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

@luizpaulo-antunes ci sta già lavorando.

Dal 12/11/2026.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
70/100
Tipo di issue
Documentazione
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
markdown, playwright, typescript
Ambito
documentation

Direzione di ricerca

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.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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.

Lingua principale
TypeScript
Stelle
99.7k
Fork
6.8k
Merge medio
16h 10m
PR unite (30g)
301

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di microsoft/playwright

Tutte le issue di microsoft/playwright

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.