GrenderG/Toasty

not showing properly in phones with smaller display

Open

#94 ouverte le 11 déc. 2018

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)Java (822 forks)batch import
can't reproducehelp wanted

Métriques du dépôt

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

Description

First of all, thanks for this amazing library. But its not showing properly in smaller display phones. Here's some screenshots:

screenshot_20181211-125645 This was the code: Toasty.error(this, "Failed GsignIn", Toast.LENGTH_SHORT, true).show();

and

screenshot_20181211-125927 This was the code: Toasty.info(this, "Loading. Please wait...", Toast.LENGTH_SHORT, true).show();

Tested in Sony Ericsson Xperia Mini (160dpi) Android 7.1.2

Thanks in advance...

Guide contributeur

Direction de recherche
Examinez le code source de Toasty, en particulier la mise en page de la vue toast personnalisée (probablement dans Toasty.java ou les fichiers de mise en page XML). Le problème est que le texte ne tient pas sur les petits écrans (160 dpi). Une correction pourrait impliquer de rendre la mise en page du toast réactive en utilisant des dimensions indépendantes de la densité (dp, sp) ou d'ajuster la taille du texte en fonction de la largeur de l'écran. Vérifiez si des commentaires ou PR précédents abordent ce problème. Implémentez une solution qui garantit que le toast reste entièrement visible sur les petits écrans.
Stack technique
javaandroid
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.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Android SDK basicsJavaunderstanding of screen densities
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65