Update Matbay à Angular 18 + fix smartphone UI#177
Open
Aezurly wants to merge 25 commits into
Open
Conversation
and update styling
…Yatga into refactor/update-angular
…grade.component and song-list, it absolutely needs cleaning)
Contributor
Author
|
En local mon |
Contributor
Author
|
en fait le fix était ultra simple il fallait juste une bonne nuit de sommeil |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
📝 TL:DR
npm run start:prodpour lancer le client en mode production plus facilement🚀 Nouveauté du UI
Avant
Après :


🎉 Quality of life : Mobile
Déplacement des cards, dialog et navbar pour qu'elles soient toujours accessible même sur mobile.
Warning
Il manque le bouton pour ramener la sidebar de gauche (avec la liste des utilisateurs) sur l'écran, donc actuellement il n'est pas possible de shuffle, gérer ses musiques, ou quitter la constitution. Ça ne me semble pas être une priorité parce que je ne pense pas que ces fonctionnalités soient agréables à utiliser sur mobile anyways
Avant
🛠️ Refactoring
<br>inutiles, ainsi que des classes css répétées placées dansstyles.scssRemarques
Note
C'est les modifications que j'avais fait la semaine dernière que j'ai rapidement terminé. Il reste encore énormément à faire pour avoir un frontend ne serait-ce qu'un peu propre, j'ai juste fait le strict minimum pour avoir une version viable avec les modifications que j'avais commencé.