[Bug]: Numéro de salle trop long

[Cleboost]

Description du bug

Si le numéro/nom de salle est trop long, cela coupe le texte de manière brutale

Étapes à reproduire

1. Avoir un cours avec un numéro de salle très long et regarder

Comportement attendu

Je penses que si ca rentre pas on peut split a la première salle ("," en tant de séparateur dans mon cas, je sais pas si c'est de partout pareil)

Appareil

Samsung S21Fe

Version du système d`exploitation

Android 14

Version

7.2.1

Environnement

Application native Android

Source de l'application

Stores (Play Store / App Store)

Service scolaire

🦋 Pronote

Captures d'écran / vidéo

Contexte supplémentaire

No response

[Kgeek33]

C'est un cas particulier de Pronote ça, où plusieurs salles sont attribuées sur un seul cours
À changer : https://github.com/PapillonApp/Papillon/blob/main/src%2Fviews%2Faccount%2FLessons%2FDocument.tsx#L83

[Gabriel29306]

J'ai ça. C'est ce qui est voulu ?

[Kgeek33]

Oula, vous avez que des comportements bizarres

[Gabriel29306]

Comment ça ?

[Kgeek33]

Ben c'est bizarre ce que ça fait
Soit ça met des virgules, soit un \n
Trop chelou

[Gabriel29306]

Bah j'ai remplacé les virgules par un retour à la ligne juste pour les classes

[Kgeek33]

D'accord c'est pas mal
Mais comment faire le rendu sur la page d'accueil (notamment le widget)

[Kgeek33]

Sur tes captures, intégrer une genre de liste pour faire comprendre qu'il y a plusieurs salles seraient pas mieux ?

[Kgeek33]

"absences de professeur" n'est pas en rouge aussi

[Gabriel29306]

Je vais aussi devoir corriger ça.
Je prévois une pr avec plusieurs petits fixes

[Kgeek33]

Tiens @Gabriel29306 , sur ta future pr, corrige en même temps l'issue #279 stp

[Gabriel29306]

Déjà fait

[Kgeek33]

@Gabriel29306 sur ton ancienne PR, tu l'avais corrigé ?

[Gabriel29306]

@Gabriel29306 sur ton ancienne PR, tu l'avais corrigé ?

Je crois que j'ai juste fait ça sur le document de cours

[Kgeek33]

Ah mince 😅
Faudra un test de @Cleboost

[Cleboost]

Yes je peux regarder, vois avez fix ou ?

[Kgeek33]

#325 :)

[Cleboost]

Heuu, je teste quelle PR vue que vous découpez tout ? Actuellement sur mon expo en dev la moitié de l'appli est cassé si je prends la PR

[Kgeek33]

Pour l'instant, j'ai pas encore fait cette pr, j'vais la faire tout à l'heure
Si tu testes sur Expo 52, c'est normal, toutes les nouvelles versions sont buggées

[Cleboost]

J'ai pris tout comme était dans ta PR. Expo est en v51. Redis moi ici pour que je teste quand tu aura fix :)

[Cleboost]

Oups... Je suis un peu con. J'ai clone la main, donc j'ai pas pris ta PR... Je reteste

[Kgeek33]

OK dis moi si ça fonctionne
Pendant ce temps, je prépare la pr

[Cleboost]

Perso je crois pas que ca a fix

[Kgeek33]

Ok je corrige ça, je viens de publier une PR
Je vais remplacer tout ça par "Plusieurs salles attribuées"

[Kgeek33]

pask ça peut être illisible si on fait un retour à la ligne sur une ligne minuscule, non ?

[Cleboost]

D'accord avec toi, le plusieurs salles est une bonne solution.