clementwzk/OpenCalc

Outdated and inconsistent dialogs in settings

Open

#299 ouverte le 29 juil. 2023

Voir sur GitHub
 (7 commentaires) (0 réactions) (1 assigné)Kotlin (182 forks)auto 404
enhancementhelp wanted

Métriques du dépôt

Stars
 (1 480 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

Describe the bug Some of the dialog menus used in the settings are outdated and look out of place. They don't follow the Material You color palette and instead have a Material 2 design.

To Reproduce Steps to reproduce the behavior:

  1. Go to 'Settings'
  2. Click on 'Number of decimal places' or 'Results saved in history'
  3. See the old dialog/fragment

Expected behavior They should be similar to the dialog menu used for the 'Theme'.

Screenshots/videos Screenshot_20230728_211853_Calculator_1 Screenshot_20230728_211900_Calculator_1

Smartphone (please complete the following information):

  • Device: Galaxy S20+
  • OS: Android 13
  • Version: 3.0.0-beta3
  • Language: English

Additional context I saw this issue in only 2 of the 3 menus found in the app which are mentioned above.

Guide contributeur

Direction de recherche
Examinez le style de dialogue utilisé pour l'option Thème et reproduisez le pour les dialogues Nombre de décimales et Résultats enregistrés dans l'historique. Comparez les fichiers de mise en page et appliquez la palette de couleurs Material You.
Stack technique
kotlin
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.
Claire
Prérequis
Android developmentKotlin
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75