Skip to content
This repository has been archived by the owner on Aug 25, 2024. It is now read-only.

Swipe #275

Open
wants to merge 48 commits into
base: development
Choose a base branch
from
Open

Swipe #275

wants to merge 48 commits into from

Conversation

Sacha338
Copy link
Contributor

Checklist d'avant pull request

  • Vous avez testé de build le projet avec vos modifications et ce build a réussi
  • Vous respectez les conventions de codage et de nommage du projet
  • Vous utilisez la tabulation pour l'indentation afin de maintenir un code lisible
  • Cette pull request n'est pas un duplicata d'une autre
  • Cette pull request est prête à être revue (review) et fusionnée (merge)
  • Cette pull request doit être fusionnée dans la branche development (le cas contraire préciser laquelle)
  • Il n'y a pas de TODO (aka des annotations pour du code manquant) dans vos modifications
  • Il n'y a pas d'erreurs de langue dans votre code (grammaire, vocabulaire, conjugaison, orthographe)
  • Les détails des changements ont été décrits ci-dessous
  • Cette pull-request n'est pas une "breaking-change" (par exemple des modifications qui vont entraîner la modification du fonctionnement de certaines fonctionnalités déjà existantes)

Explications des changements

Maintenant on peut swiper pour marquer une actualité comme lue ou non lue.

Changelogs proposés

Maintenant on peut swiper pour marquer une actualité comme lue ou non lue.

Informations supplémentaires

Dites moi si les couleurs et les icons sont bien

@Sacha338
refactor: streamline ListItem component and clean up types and styles
c44636d
@Sacha338
refactor: update NativeItem component types and streamline styles
091700a
@Sacha338
refactor: improve typings and add missing props in NativeItem component
52ae70d
@Sacha338
refactor: update NativeList.ios.tsx for improved prop typing and memo…
941d1e2 
…ization
@Sacha338
feat: Enhanced NativeList component with sectionProps and sideBar option
1c8b53a
@Sacha338
refactor: update PapillonButton component styles and props
b0324c8
@Sacha338
feat: add size prop and update styles in PapillonIcon component
77d9fd8
@Sacha338
fix: add missing plain prop in PapillonList component
37684cb
@Sacha338
feat: make subtitle optional and update icon prop in PapillonLoading …
63e243c 
…component
@Sacha338
refactor: refactor fetch functions to use async/await in LoginFlow.js
f0d1f42
@Sacha338
fix(bagReminder): correct type casting and null checks for timetable …
89edb47 
…fetching and notification body
@Sacha338
fix(grades): correct type casting and handle null values in grade pro…
a74360e 
…cessing
@Sacha338
Refactor: Add explicit types and improve readability in Homework noti…
35f2bfd 
…fication logic
@Sacha338
Enhancement: Add null check for dataInstance in Lessons.ts and improv…
ad776b9 
…e logging
@Sacha338
Refactor: Add typings and update date comparison in SelfReminder.ts
3b96f92
@Sacha338
Refactor: Add classroom property to PapillonLesson and sort timetable…
1acf59a 
… by start date
@Sacha338
Refactor: Add observations property to PapillonVieScolaire and handle…
348eb60 
… absence intervals in EDvieScolaireHandler
@Sacha338
style: add semicolons for consistent code formatting
246c83f
@Sacha338
fix: handle undefined grade values with defaults
0816b96
@Sacha338
Update index.ts
1906cb0
@Sacha338
feat(timetable): add classroom and num properties to PapillonLesson i…
83cd740 
…nterface
@Sacha338
fix(grades): handle undefined grade values in getGradeState
7e7eb98
@Sacha338
feat(timetable): add classroom property to PapillonLesson
e08d856
@Sacha338
Refactor: Update type definitions for PapillonVieScolaire interfaces
74d2360
@Sacha338
Refactor: Simplify AlertBottomSheet component props handling and anim…
f56f570 
…ation logic
@Sacha338
Refactor TimeSeparator to TypeScript with Props Interface
7f82441
@Sacha338
Refactor NextCours component to improve performance and UI details ha…
0d6f65f 
…ndling

- Add  for better handling of insets on iOS
- Implement dynamic detail animation based on scroll offset
- Enhance course filtering to exclude cancelled courses and sort by start time
- Refactor time and progress bar calculation for accuracy
- Update interval-based course checking and state updates
- Improve layout and styling of course details and progress bar
- Add  prop handling for conditional rendering
- Use fallback color for progress bar to ensure visibility
- Optimize useEffect hooks for better state management and performance
@Sacha338
refactor: clean up and type view options in AppStack
9f2afc9
@Sacha338
refactor: add type assertion and platform import in AuthStack
72326fe
@Sacha338
refactor: optimize screenOptions for InsetSettings and add Platform i…
1ba11dd 
…mport
@Sacha338
refactor(AppContext): add TypeScript type annotations and fix context…
4a57a0f 
… type definitions
@Sacha338
feat(useUIColors): add textLight and statusBar properties to enhance …
f5b7b04 
…theme customization
@Sacha338
refactor: remove unused imports and redundant prop in MessageScreen
5937e87
@Sacha338
style: remove redundant style properties in ChangelogScreen.tsx
d919c5b
@Sacha338
refactor: update EDExtendedMenu.tsx for improved module rendering and…
128ef26 
… type safety
@Sacha338
refactor: update GradesSettings.tsx for improved type safety and styl…
66af5dd 
…e handling
@Sacha338
refactor: optimize GradesSimulatorMenu.tsx for better performance and…
76e2f3d 
… styling consistency
@Sacha338
Typing and improving views
e96a90e
@Sacha338
Tentative de résolution du conflicts
812e4a5
@Sacha338
Ajout de la fonction de swipe
435935c
@Sacha338
Ajout de la lib
65a1a57
@Sacha338
Petits fixs
9d74657
@Sacha338
Copy link
Contributor Author

Je sais pas pourquoi ça a mélangé tous les commits

@Gabriel29306
Copy link
Contributor

Gabriel29306 commented Jun 28, 2024 •
edited
Loading

J'ai trouvé plusieurs problèmes pour le moments:

  • Toutes les nouvelles sont affichées quand on va sur l'écran des actualités. Si c'est volontaire, il faudrait retirer le Non lues / Toutes en haut.
  • Le message indiquant le nombre de pièces jointes prennent maintenant toute la largeur.
  • Les messages Lu et Non Lue n'ont pas la même taille.
  • On dit "Lu" et "Non lu", Le participe passé "lu" est invariable et s'accorde donc avec le mot "actualité" de manière générique. (Pour le filtre Non lues / Toutes c'est bon)

Autres différences n'ayant que un aspect esthétique:

  • La couleur du texte pour le Non lues / Toutes en haut est maintenant en blanc au lieu du rouge d'avant.
  • Les marges sur le côté ont été réduite + La marge entre le header et les actus a été augmentée. (Il suffit de remettre styles comme avant)

Gabriel29306
Gabriel29306 reviewed Jun 28, 2024
View reviewed changes
views/NewsScreen.tsx Outdated Show resolved Hide resolved
Gabriel29306
Gabriel29306 reviewed Jun 28, 2024
View reviewed changes
views/NewsScreen.tsx Outdated Show resolved Hide resolved
Gabriel29306
Gabriel29306 reviewed Jun 28, 2024
View reviewed changes
views/NewsScreen.tsx Outdated Show resolved Hide resolved
Gabriel29306
Gabriel29306 reviewed Jun 28, 2024
View reviewed changes
views/NewsScreen.tsx Outdated Show resolved Hide resolved
@Gabriel29306
Copy link
Contributor

Désolé si ça fait beaucoup mais j'aime bien comprendre chaque changements.

@Sacha338
Copy link
Contributor Author

J'ai tout remis en ordre.

J'ai juste pas compris où est ce que tu as vu que j'avais touché aux marges. Si tu peux m'en dire plus.

@Gabriel29306
Copy link
Contributor

Après Avant
Screenshot_20240628_194116_Papillon DEV Screenshot_20240628_194145_Papillon DEV

@Gabriel29306
Copy link
Contributor

Gabriel29306 commented Jun 29, 2024 •
edited
Loading

PS: C'est avant tes fix
Faut que je teste après tes fixes

@LeGeek01
Copy link
Contributor

faut que je teste et que je merge c'est ça

@Sacha338
Copy link
Contributor Author

@LeGeek01, tu as pu tester ?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Gabriel29306 Gabriel29306 Gabriel29306 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Sacha338 @Gabriel29306 @LeGeek01