mozilla/send

[Send 2][Enhancement] Clicking inside the “Privacy” overlay should dismiss it

Open

#1 304 ouverte le 22 avr. 2019

Voir sur GitHub
 (6 commentaires) (0 réactions) (0 assignés)JavaScript (1 239 forks)batch import
[QA]:Enhancementgood first issueskill:js

Métriques du dépôt

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

Description

[Notes]:

  • There should be consistency regarding the dismissal of the overlay considering that the upload complete and Sign in/up overlays are dismissed when clicking inside them.

[Affected versions]:

  • Firefox 66.0.3 and up
  • Chrome Version 73.0

[Affected Platforms]:

  • All Mac
  • All Windows
  • All Linux

[Steps to reproduce]:

  1. Open the browser and navigate to the https://send.firefox.com/ website.
  2. Click the “Privacy” button.
  3. Click inside the overlay but not on any actionable buttons.

[Expected result]:

  • The overlay is dismissed.

[Actual result]:

  • The overlay is still displayed.

[Additional Notes]:

rec of issue - overlay dismissal

Guide contributeur

Direction de recherche
Localisez le composant de superposition Confidentialité dans la base de code. Étudiez comment les autres superpositions (téléchargement terminé, connexion/inscription) sont fermées lorsqu'on clique à l'intérieur. Implémentez un comportement similaire pour la superposition Confidentialité.
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.
Claire
Prérequis
GitJavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75