Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Minor] 200 - Add "topMenu" Layout #205

Merged
merged 1 commit into from
Nov 10, 2024
Merged

[Minor] 200 - Add "topMenu" Layout #205

merged 1 commit into from
Nov 10, 2024

Conversation

mguihal
Copy link
Collaborator

@mguihal mguihal commented Nov 4, 2024

Hello,

Ajout du second layout, nommé "TopMenu" dans l'architecture décrite dans le ticket #200.

Il permet d'avoir une interface comme la capture ci-dessous.
Capture d’écran 2024-11-04 à 01 07 53

Quelques petites remarques en plus de l'ajout :

  • Le LayoutContext et le LayoutProvider ont dû être séparé dans deux fichiers séparés sinon le hot-reloading Vite ne fonctionne pas correctement.
  • Un loader a été rajouté dans le composant Layout de base, nécessaire vu que les fichiers de layout se chargent de manière lazy désormais.
  • Le titre de l'app a été déplacé dans la configuration de façon à pouvoir s'en servir dans les fichiers de Layout également.
  • Le composant CssBaseline de Material-ui a été rajouté au lieu d'avoir un fichier index.css à la racine, ça permet d'utiliser le thème de l'application y compris pour color le html et body.

@mguihal mguihal added this to the Archipelago customization milestone Nov 4, 2024
@mguihal mguihal self-assigned this Nov 4, 2024
@mguihal mguihal changed the title [Minor] 200 - Top Menu Layout [Minor] 200 - Add "topMenu" Layout Nov 4, 2024
Base automatically changed from 200-AdaptFirstLayoutToNewArchitecture to master November 6, 2024 18:59
@mguihal mguihal merged commit dc082bc into master Nov 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant