browsh-org/browsh

Stub `alert()`

Open

#26 geöffnet am 12. Juni 2018

Auf GitHub ansehen
 (4 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (455 Forks)batch import
bughelp wanted

Repository-Metriken

Stars
 (15.996 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Alerts are fairly common, but they are not rendered or interactable in the TTY.

Contributor Guide

Research-Richtung
Untersuche, wie die Browser Engine in browsh andere DOM Methoden wie confirm() und prompt() behandelt, und suche nach vorhandenen Stubs oder Implementierungen. Das Problem befindet sich in der Codebasis, die die JavaScript Interoperabilität für den TTY Renderer behandelt. Ein Stub für alert() sollte wahrscheinlich dem Muster entsprechen, das für andere nicht implementierte Dialoge verwendet wird. Überprüfe Issue #26 des Repositorys auf frühere Diskussionen oder verknüpfte PRs, die Kontext liefern könnten.
Tech Stack
javascript
Domain
frontend
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
basic JavaScriptunderstanding of browser events
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60