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

Fix: homepage styles #655

Merged
merged 18 commits into from
Feb 4, 2025
Merged

Fix: homepage styles #655

merged 18 commits into from
Feb 4, 2025

Conversation

narduin
Copy link
Contributor

@narduin narduin commented Jan 21, 2025

Fix layout quirks.
Fix horizontal scrolling.
Proposal for enhanced responsive.

fix ecolabdata/ecospheres#509

Copy link

netlify bot commented Jan 21, 2025

Deploy Preview for ecospheres ready!

Name Link
🔨 Latest commit 25aa02c
🔍 Latest deploy log https://app.netlify.com/sites/ecospheres/deploys/67a22792c055f800089906e2
😎 Deploy Preview https://deploy-preview-655--ecospheres.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Jan 21, 2025

Deploy Preview for meteo-france ready!

Name Link
🔨 Latest commit 25aa02c
🔍 Latest deploy log https://app.netlify.com/sites/meteo-france/deploys/67a22792de63290008a32001
😎 Deploy Preview https://deploy-preview-655--meteo-france.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@narduin narduin force-pushed the fix/homepage-styles branch from 25cf943 to 461555f Compare January 23, 2025 11:55
@narduin
Copy link
Contributor Author

narduin commented Jan 24, 2025

@Thesauruv une piste pour mettre les liens plus en avant.

localhost_5173_
localhost_5173_ (1)
localhost_5173_ (2)
localhost_5173_ (3)

@Thesauruv
Copy link

Thesauruv commented Jan 24, 2025

@narduin merci pour cette proposition pour les liens des blocs explorer et contribuer ! Je trouve ces boutons plus visibles, cette version a également le mérite de s'éloigner de l'effet lien hypertexte "brut".

@thomas-duport qu'en penses-tu ?

@Thesauruv
Copy link

@narduin au point si ce n'est pas de trop dans ce ticket. Pourrais-tu enlever la majuscule aux mentions "Ministère en charge de l'environnement" lorsque celles-ci sont en milieu de phrase ?

image

image

image

image

Merci à toi.

@ghost
Copy link

ghost commented Jan 25, 2025

Ok pour cette nouvelle proposition de lien mais sans les borders autour de chaque lien. Nous ne sommes pas sur un bouton mais un lien conservons la cohérence

remove uppercase of Ministère
@narduin
Copy link
Contributor Author

narduin commented Feb 3, 2025

Ok pour cette nouvelle proposition de lien mais sans les borders autour de chaque lien. Nous ne sommes pas sur un bouton mais un lien conservons la cohérence

@Thesauruv tu veux rester sur cette mise en page (plutôt à droite) ou revenir sur l'ancienne ?

localhost_5173_ (1)

@narduin narduin marked this pull request as ready for review February 3, 2025 09:39
@narduin narduin requested a review from abulte February 3, 2025 09:39
@Thesauruv
Copy link

Thesauruv commented Feb 3, 2025

@narduin Comme discuté il y a deux semaines, voici la dernière proposition de @thomas-duport concernant les deux sections bouquets : il propose de les réunir sur un même fond gris comme ci-dessous. La FAQ devra elle apparaître sur fond blanc. À ce stade, je propose d'ignorer les boutons "bouquets mis en avant" et "derniers bouquets". Je te propose de rediscuter ces points dans l'après-midi.

Capture d’écran 2025-02-03 à 12 20 51

src/components/bouquets/BouquetCard.vue Outdated Show resolved Hide resolved
src/custom/ecospheres/components/HomeFaq.vue Show resolved Hide resolved
src/custom/ecospheres/views/HomeView.vue Show resolved Hide resolved
src/custom/ecospheres/views/HomeView.vue Outdated Show resolved Hide resolved
@Thesauruv
Copy link

@narduin Suite à notre discussion, pas de changement majeur du texte dans cette PR hormis ce qui vient d'être discuté : dans la section "bouquet", remplacer "Bouquets de données : articuler politiques publiques et données territoriales" par "Cataloguer les usages grâce aux bouquets". Un grand merci.

@narduin narduin requested a review from abulte February 3, 2025 16:01
@abulte
Copy link
Contributor

abulte commented Feb 4, 2025

La définition de --blue-france-sun-113 a au moins un effet de bord : carte des organisations vire au bleu.

.datagouv-components :where(.fr-link, a:not(.fr-btn)) {
    color: var(--blue-france-sun-113);
}
Capture d’écran 2025-02-04 à 09 51 05

@narduin narduin requested a review from abulte February 4, 2025 11:16
Copy link
Contributor

@abulte abulte left a comment

Choose a reason for hiding this comment

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

Désolé j'ai approuvé trop vite. Petit problème d'espacement dans le composant nom de l'organisation.

Capture d’écran 2025-02-04 à 14 26 24 Capture d’écran 2025-02-04 à 14 26 05

@narduin
Copy link
Contributor Author

narduin commented Feb 4, 2025

Désolé j'ai approuvé trop vite. Petit problème d'espacement dans le composant nom de l'organisation.

Ah oui bien vu c'est corrigé !

@narduin narduin force-pushed the fix/homepage-styles branch from 2952455 to ac0eda5 Compare February 4, 2025 13:38
abulte
abulte previously approved these changes Feb 4, 2025
@narduin
Copy link
Contributor Author

narduin commented Feb 4, 2025

@Thesauruv c'est bon de ton côté pour la page d'accueil ?

@Thesauruv
Copy link

Thesauruv commented Feb 4, 2025

@narduin Concernant la section bouquets, comme présenté dans le document de Thomas (voir screenshot ci-dessus), possible de réduire l'espace et d'insérer une ligne de séparation entre le bloc "Documenter un besoin, lister les informations, etc." et celui des bouquets à découvrir ?

image

versus

image

@narduin
Copy link
Contributor Author

narduin commented Feb 4, 2025

@narduin Concernant la section bouquets, comme présenté dans le document de Thomas (voir screenshot ci-dessus), possible de réduire l'espace et d'insérer une ligne de séparation entre le bloc "Documenter un besoin, lister les informations, etc." et celui des bouquets à découvrir ?

Je n'avais pas vu la bordure de séparation !

@narduin narduin merged commit 42cb8ec into main Feb 4, 2025
8 checks passed
@narduin narduin deleted the fix/homepage-styles branch February 4, 2025 15:17
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.

Ajustements CSS homepage mobile et desktop
3 participants