hyperoslo/ImagePicker

Do you think imageDidDrop should calls dismissIfNeeded?

Ouverte

#432 ouverte le 10 sept. 2018

Voir sur GitHubGuide contributeur
 (7 commentaires) (0 réaction) (0 personne assignée)Swift (619 forks)batch import
help wanted

Métriques du dépôt

Stars
 (4 400 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

When imageLimit is set to 1 and touches the photo already selected, the imagePicker dismisses.

Is that what you expected?

I wonder why just touching the selected photo does not deselect the photo.

Guide contributeur

Direction de recherche
Examinez la méthode dismissIfNeeded dans le code d'ImagePicker pour comprendre pourquoi la sélection d'une photo déjà sélectionnée lorsque imageLimit est à 1 provoque la fermeture au lieu de la désélection. Vérifiez la logique de gestion de imageLimit et des événements tactiles sur les images sélectionnées.
Stack technique
swift
Domaine
mobile
Type d'issue
Bug
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.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Swift
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75