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

Links block #418

Merged
merged 4 commits into from
May 17, 2024
Merged

Links block #418

merged 4 commits into from
May 17, 2024

Conversation

Olivia206
Copy link
Contributor

@Olivia206 Olivia206 commented May 17, 2024

Type

  • Nouvelle fonctionnalité
  • Bug
  • Ajustement
  • Rangement

Description

Ajout du bloc "liens" au thème.

Reste à faire :

  • Les crédits des images
  • Valider l'alignement des blocs lorsqu'il n'y a pas tout le temps des images (cf. screens plus bas)

Je me suis fondée sur plusieurs autres blocs pour établir le schéma de données, n'hésitez pas à me dire s'il peut être amélioré :

- kind: block
    template: links
    title: >-
      Le bloc de liens
      
    ranks:
      self: false
    data:
      description: >-
        <p>Quelques liens pour tester</p>

      links:
        
      - title: >-
          Espace Numérique de Travail

        description: >-
          Espace Numérique de Travail des Personnels (ENTP) Université Bordeaux Montaigne

        url: >-
          https://support.osuny.org/

        target_blank: true

        alt: >-
          
        credit: >-
          
        
      - title: >-
          Messagerie

        description: >-
          Webmail de l’Université

        url: >-
          https://www.noesya.coop/

        target_blank: true

        image:
          id: "99fb2adb-d07e-435f-870e-0723cdebe899"
          file: "99fb2adb-d07e-435f-870e-0723cdebe899"

        alt: >-
          
        credit: >-
          <p>Daniel Faro<br>Death to stock</p>

      - title: >-
          Mon compte IUT

        description: >-
          
        url: >-
          https://support.osuny.org/

        target_blank: true

        alt: >-
          
        credit: >-

Niveau d'incidence

  • Incidence faible 😌
  • Incidence moyenne 😲
  • Incidence forte 😱

Référence (ticket et/ou figma)

FIGMA | Nécessité pour EPV qui nous a relancé sur le sujet

URL de test sur example.osuny.org

N/A il faut pour le moment copier le schéma ci-dessus et le mettre dans une page d'example.

URL de test du site (optionnel)

N/A

Screenshots

À discuter

@arnaudlevy je veux bien ton avis pour l'alignement des blocs, s'il te plaît, j'ai du mal à arbitrer entre deux situations (ou +, je n'ai pas fait d'autres tests) concernant le cas où on a un lien sans image suivi d'un lien avec image :

Une première possibilité :

Capture d’écran 2024-05-17 à 11 07 39

Une autre :

Capture d’écran 2024-05-17 à 11 07 30

Je rajouterai des captures d'écran quand le crédit sera bien intégré.

@Olivia206 Olivia206 added question Further information is requested feature Incidence faible 😌 Risque faible de régression sur les sites labels May 17, 2024
@Olivia206 Olivia206 linked an issue May 17, 2024 that may be closed by this pull request
2 tasks
@arnaudlevy
Copy link
Member

osunyorg/admin#1889

@Olivia206 Olivia206 marked this pull request as ready for review May 17, 2024 16:34
@Olivia206 Olivia206 merged commit be0240b into main May 17, 2024
2 checks passed
@Olivia206 Olivia206 deleted the links-block branch May 17, 2024 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Incidence faible 😌 Risque faible de régression sur les sites question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bloc lien
2 participants