epfl-si / elements

Charte graphique Web EPFL
https://epfl-si.github.io/elements
22 stars 12 forks source link

Give item preview full height space #717

Closed alinekeller closed 4 months ago

alinekeller commented 5 months ago

Une petite proposition d'amélioration pour l'affichage "full render". Actuellement, la hauteur de

, qui contient l'exemple de composant, s'adapte à la hauteur du composant. S'il contient un élément qui déborde, comme un dropdown, celui-ci n'est pas visible. Grâce à flexbox, on peut forcer le container à occuper tout l'espace disponible dans la fenêtre.

github-actions[bot] commented 5 months ago

Unit Test Results

    1 files      1 suites   0s :stopwatch: 268 tests 262 :heavy_check_mark: 0 :zzz: 0 :x: 6 :fire: 268 runs  256 :heavy_check_mark: 0 :zzz: 6 :x: 6 :fire:

For more details on these errors, see this check.

Results for commit c83debbd.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 5 months ago

🔎 Download the Backstop report for this pull request (link valid for 90 days):

williambelle commented 4 months ago

Est-il possible d'avoir la même fonctionnalité mais sans le "débordement" de la couleur ? Par exemple, pour le footer, le background en noir dépasse/s'étire jusqu'au bas et on dirait que le composant ne se comporte pas correctement.

Avant Après
epfl_elements_organisms_footer-light-dark_0_document_0_lg epfl_elements_organisms_footer-light-dark_0_document_0_lg
alinekeller commented 4 months ago

@williambelle Non comme la couleur de fond se trouve sur l'élément qui est agrandi, ce n'est pas possible. Une autre possibilité serait de supprimer la propriété overflow:hidden qui se trouve sur .tlbx-item-preview, ainsi par exemple les dropdowns seraient quand même visibles même s'ils débordent.

williambelle commented 4 months ago

Je vois maintenant dans le code qu'il y avait une exception pour le dropdown :

https://github.com/epfl-si/elements/blob/83bce322b78bc569183894dc406fca449bfce5f3/reader/reader.scss#L44-L48

mais il faudrait remplacer tlbx-component-dropdown par tlbx-element-dropdown. On pourrait ajouter tlbx-element-select-multiple pour le select multiple qui est aussi caché derrière le code.

alinekeller commented 4 months ago

Je vois maintenant dans le code qu'il y avait une exception pour le dropdown :

https://github.com/epfl-si/elements/blob/83bce322b78bc569183894dc406fca449bfce5f3/reader/reader.scss#L44-L48

mais il faudrait remplacer tlbx-component-dropdown par tlbx-element-dropdown. On pourrait ajouter tlbx-element-select-multiple pour le select multiple qui est aussi caché derrière le code.

Parfait, j'ai modifié ça et ajouté directement la classe pour le breadcrumb avec dropdown.