browsh-org/browsh

Stub `alert()`

Open

#26 ouverte le 12 juin 2018

Voir sur GitHub
 (4 commentaires) (0 réactions) (0 assignés)JavaScript (455 forks)batch import
bughelp wanted

Métriques du dépôt

Stars
 (15 996 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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

Guide contributeur

Direction de recherche
Étudiez comment le moteur de navigateur dans browsh gère d'autres méthodes DOM comme confirm() et prompt(), et recherchez des stubs ou implémentations existants. Le problème se situe dans la base de code gérant l'interopérabilité JavaScript pour le moteur de rendu TTY. Un stub pour alert() devrait probablement correspondre au modèle utilisé pour d'autres boîtes de dialogue non implémentées. Consultez l'issue #26 du dépôt pour toute discussion antérieure ou PR liés pouvant fournir du contexte.
Stack technique
javascript
Domaine
frontend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
basic JavaScriptunderstanding of browser events
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60