Skip to content

Modern UI Rework - #9

Merged
fabienengels merged 17 commits into
fabienengels:mainfrom
loannlagarde:main
Nov 11, 2025
Merged

fabienengels merged 17 commits into
fabienengels:mainfrom
loannlagarde:main

Conversation

@loannlagarde

Copy link
Copy Markdown
Contributor

Bonjour,

Arrivé il y a un mois dans la communauté, je suis plutôt discret. Les projets Meshtastic et Alsamesh ont su piquer ma curiosité, et j’aimerais contribuer à ma manière.

Je vous propose ici une petite amélioration graphique du site. L’objectif est d’obtenir un visuel plus moderne et attrayant.

Le contenu n’a pas été modifié. Cependant, j’ai dû ajouter et adapter certains éléments de la page d’accueil afin de la rendre plus accrocheuse. Certains points pourront bien sûr être ajustés selon vos retours.

Je suggère de ne pas fusionner directement cette proposition dans la branche principale, mais plutôt de créer une branche de travail dédiée pour affiner les modifications ensemble.

À mon goût, il reste encore un peu de travail à faire sur la page d’accueil, notamment concernant les images d’illustration et le composant AlsaceShape. Cependant, je préfère vous soumettre cette proposition à ce stade afin d’éviter un développement inutile.

En espérant que vous apprécierez mon travail.

- Removed unused `toggleMenu.js`.
- Enhanced dark mode support with updated classes and colors.
- Redesigned sections for better alignment and responsiveness.
- Updated assets, including new icons and SVG elements.
- Simplified header/menu structure with improved links and interaction logic.
- Added animation for visual enhancements in the "AlsaceShape" component.
- Updated layout structure for improved readability and alignment.
- Enhanced typography styles using `@tailwindcss/typography` plugin.
- Improved visual hierarchy and dark mode support.
- Extend maximum width of content container for better readability.
- Adjust typography breakpoints for improved responsiveness.
- Enhance em tag with better inline-block support.

@Silhm Silhm left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work!
see my comments in some files

Comment thread bun.lock
Comment thread package-lock.json Outdated
Comment thread rework-screenshoot.png Outdated

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe a simple webp would be enough here, no need for super high-quality png screenshot imho

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

La modification a été effectuée, toutefois son impact sur le dépôt me semble limité.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Les images peuvent être optimisées automatiquement lors du build : https://docs.astro.build/en/guides/images/ si jamais

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est noté. Dans le cas présent l'image ne sera pas compilée.

NB: c'est la première fois que j'utilise Astro 😉. N’hésitez donc pas à me conseiller.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je n'ai pas non plus une grande expérience avec Atro mais j'utilise Alsamesh pour explorer de nouvelles solutions :)

@fabienengels
fabienengels marked this pull request as draft November 2, 2025 15:56
- Add `@fortawesome/fontawesome-free` at version `^7.1.0`.
- Upgrade `tailwindcss` to version `^4.1.16`.
- Created a `node-points.json` file to store GPS coordinates.
- Implemented `gpsToSvg` function to convert GPS coordinates to SVG space.
- Automated SVG point rendering using mapped coordinates with animation effects.
- Introduced a QR code dialog with toggle functionality for node configuration.
- Added new styling and layout updates for the configuration section.
- Incorporated a button to display the QR code and a clickable link for alternative access.
@loannlagarde

Copy link
Copy Markdown
Contributor Author

Concernant les photos d’illustration, j’ai quelques idées, mais il serait préférable de faire une sélection d’environ six photos (huit maximum).
Je verrais bien un carrousel comportant, pour chaque image, un titre, une courte description et un crédit.
Il serait préférable d’éviter les captures d’écran, qui sont généralement moins attractives et s’accordent moins bien avec le reste du thème du site.
Nous pourrions également inviter l’ensemble de la communauté, par exemple via un canal Discord dédié pour le dépôt des photos, en utilisant les réactions pour le vote.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

N’hésitez pas à me signaler s’il y a des nœuds à ajouter. J’ai réalisé une extraction de ceux que je peux actuellement détecter depuis chez moi.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pour les noeuds, je suis en train de bosser sur une solution pour les stocker dynamiquement et faire une carte dynamique. La carte sur le site actuelle est plus la pour l'illustration. Te casses pas trop la tête avec ça :)

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Le carrousel peut être sympa tant que ça reste responsive design, je n'ai pas encore mis en place d'analytics sur le site mais je pense qu'on a pas mal de visite via mobile.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pour les photos, on peut organiser un vote mais j'avoue que pour ce carrousel, peut être garder les choses simples et te faire confiance pour la sélection. Ceci dit, tu peux toujours faire un appel pour que les gens envoient leurs meilleures photos sur le Discord.

(Et on peut toujours peut être prévoir une galerie si les photos sont assez nombreuses)

- Implemented a new image carousel using a marquee effect.
- Introduced `shuffle` utility in `ArrayUtils.ts` for randomizing image order.
- Replaced static image grid with dynamically loaded and shuffled images.
- Added new images to the project under `/carrousel` directory.
Comment thread src/components/Home.astro Outdated
},
{
file: image6,
title: "@FAB1"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

j'ai un doute sur celle-là

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est Nicolas posant à côté de ESPL :)

@loannlagarde

Copy link
Copy Markdown
Contributor Author

Je pense que la branche est assez mature pour être fusionnée. Je vous laisse me faire votre retour.

@fabienengels fabienengels left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deux petits trucs à corriger mais après, pour moi c'est ok pour le merge :)

Comment thread README.md
Comment thread src/components/Home.astro Outdated
},
{
file: image6,
title: "@FAB1"

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est Nicolas posant à côté de ESPL :)

@loannlagarde

Copy link
Copy Markdown
Contributor Author

Voila

@fabienengels
fabienengels marked this pull request as ready for review November 11, 2025 13:29
@fabienengels
fabienengels merged commit e22aff4 into fabienengels:main Nov 11, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants