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

Display sponsor description only for platinium level sponsors #81

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

JulienPradet
Copy link
Contributor

@JulienPradet JulienPradet commented Mar 23, 2019

https://www.paris-web.fr/design-system/feature-sponsors/pages/sponsors/

Voici l'implémentation de la nouvelle page de sponsor. J'ai essayé de faire le minimum de changement en terme de HTML pour que ce soit facile à mettre à jour.

Pour ajouter un sponsor, il faudra faire :

<div class="sponsor sponsor--<niveau>">
  <div class="sponsor__logo">
    <img src="lien_vers_l_image.svg"  alt="Nom du sponsor" />
  </div>
</div>

Sachant que <niveau> correspond à :

  • or
  • argent
  • bronze
  • autre (= sponsors orateurs)

Ceci est documenté sur cette page : https://www.paris-web.fr/design-system/feature-sponsors/components/sponsor/
Au survol d'un sponsor, un petit </> va apparaître qui vous permettra de voir le code.

Par rapport à la dernière remarque de François qui dit que c'est bizarre de mettre Delicious Insight tout seul en haut, j'ai simplement laissé le titre pour l'instant pour montrer que ce n'est pas juste un traitement de faveur. A voir ce que vous en pensez.

@JulienPradet
Copy link
Contributor Author

Il y a un souci sur le logo de Delicious Insight qui est trop petit sur firefox.

@joachimesque
Copy link
Collaborator

Super, l'intégration :) Est-ce que tu veux la photo à utiliser pour le bloc en bas de page ?

Pour le CSS, voilà les hauteurs de chaque contenant :

  • or : height: 160px;
  • argent : height: 140px;
  • bronze : height: 120px;
  • autre : height: 100px;

Pour les logos : tu as bien fait de mettre un max-width de 70%, je mettrais un max-height: calc(100% - 10px).

Pour la taille du logo, essaye d'enlever le width et le height dans la balise <svg /> du fichier logo svg.

@JulienPradet
Copy link
Contributor Author

Ah oui, je veux bien l'image :) Je ne sais pas pourquoi, dans ma tête ce bloc n'avait pas bougé mais je n'avais pas vu que même dans l'ancienne intégration j'avais zappé l'image.

Pour les hauteurs elles sont maintenant appliquées sur chaque logo. Par contre, ce n'est qu'une hauteur max sur mobile afin d'éviter des espaces bizarres entre les logos très horizontaux. A voir si c'est une bonne chose.

Pour le max-height, je ne suis pas sûr de comprendre ce que tu veux dire par là, ni où tu veux le mettre.

C'est réglé pour la taille du logo sur firefox :)

@kloh-fr
Copy link

kloh-fr commented Mar 27, 2019

C'est top comme ça, et simple à mettre à jour du coup. Peut-être qu'on centrerait juste les logos aussi lorsqu'ils sont les uns sous les autres non ? Je pense que ce serait un peu plus lisible, en permettant notamment de distinguer les logos des titres des niveaux de partenariat.

Copy link

🛺 Branch preview has been deployed!

👉 https://www.paris-web.fr/design-system/feature-sponsors/index.html

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