Papillon icon indicating copy to clipboard operation
Papillon copied to clipboard

fix: unavailable android platform color (#476)

Open lumaa-dev opened this issue 1 month ago • 7 comments

Contribution

Règles de contribution

[!CAUTION] Afin de garantir une application stable et pérenne dans le temps, nous t'invitons à vérifier que tu as bien respecté les règles de contribution. Sans cela, ta Pull Request ne pourra pas être examinée.

  • [x] Cette Pull Request porte sur une seule fonctionnalité ou un seul correctif.
  • [ ] Cette Pull Request n'est pas faite essentiellement avec de l'IA.
  • [ ] Pour tout changement majeur, j’ai créé une issue afin d’échanger avec les mainteneurs de Papillon sur la meilleure façon de l’intégrer.
  • [x] Ma Pull Request respecte les conventions Conventional Commits et Conventional Branch ainsi que les conventions de codage de l'application.
  • [ ] J’ai testé mes modifications sur iOS et Android, et l’application fonctionne correctement.
  • [x] J’emploie un langage informel, clair et concis dans mes messages.
  • [x] J’ai documenté mes changements de manière appropriée, soit dans la description de la Pull Request, soit dans le GitBook.
  • [ ] J’ai ajouté les traductions nécessaires dans au moins un fichier de langue.

Résumé des changements

  • Remplacé la couleur du Papicons cross de PlatformColor('labelColor') pour colors.text. La PlatformColor pour labelColor n'existe pas sur Android, ce qui causait un crash.
  • Enlevé l'import de PlatformColor de react-native.

Informations supplémentaires

Issues concernées : #476

lumaa-dev avatar Nov 07 '25 17:11 lumaa-dev

my bad j'ai mal lu les changements 💀

raphckrman avatar Nov 07 '25 17:11 raphckrman

La PlatformColor('labelColor') existe pas sur Android, faudrait utiliser un équivalent ou utiliser autre chose pour pas que ça crash

lumaa-dev avatar Nov 07 '25 17:11 lumaa-dev

La PlatformColor('labelColor') existe pas sur Android, faudrait utiliser un équivalent ou utiliser autre chose pour pas que ça crash

Et ça ne break pas le style sur iOS 26 ?

raphckrman avatar Nov 07 '25 17:11 raphckrman

il me semble que PlatformColor est utilisé pour iOS 26, c'est donc plus intéressant dans ce cas de vérifier la platform avant et ensuite d'appliquer le style qui correspond

raphckrman avatar Nov 07 '25 17:11 raphckrman

Juste pour confirmer, tu suggères de faire un truc genre

if (Platform.OS == "ios") {
     PlatformColor('labelColor')
} else {
     colors.text
}

(c'est pas le truc final je précise quand même)

lumaa-dev avatar Nov 07 '25 18:11 lumaa-dev

plus simple, utilise une opération ternaire sinon, (ps: y'a une fonction pour savoir si le device run sous iOS 26)

raphckrman avatar Nov 07 '25 18:11 raphckrman

Ok je vois, je refais un commit 👌

lumaa-dev avatar Nov 07 '25 18:11 lumaa-dev