Restruct UI components "Panel" #939
Conversation
MikiDi
commented
Jun 8, 2021
MikiDi
commented
- rename components
- document arguments
- use boolean values for boolean arguments
| @@ -1,7 +1,7 @@ | |||
| {{!-- TODO every use of @activity. will change --}} | |||
| <div class="auk-u-mb-3"> | |||
| <WebComponents::AuPanel @scrollable="true"> | |||
| <WebComponents::AuPanel::Header @borderless={{this.isCollapsed}}> | |||
| <Auk::Panel> | |||
There was a problem hiding this comment.
Deze scrollable deed precies niks
There was a problem hiding this comment.
Zie commit message ea0a7bc
| @@ -3,8 +3,8 @@ | |||
| {{!-- DOSSIER PANEL --}} | |||
| <div class="auk-u-mb-3"> | |||
| {{#if this.isInscriptionInEditMode}} | |||
| <WebComponents::AuPanel data-test-publication-case-panel-case-editing class="auk-panel--editing"> | |||
| <WebComponents::AuPanel::Header> | |||
| <Auk::Panel data-test-publication-case-panel-case-editing class="auk-panel--editing"> | |||
There was a problem hiding this comment.
deze auk-panel--editing wordt 2x gebruikt, 1 keer op de auk::Panel, 1 keer op de auk::Panel::Body (documents-filter).
waar hoort deze thuis en gaan we dit een optie maken in de auk component ?
There was a problem hiding this comment.
There was a problem hiding this comment.
Die class zou alleszins al op auk::Panel moeten staan en niet op auk::Panel::Body. De mogelijkheid om dit als optie te kunnen doorgeven lijkt me wel handig 👍
There was a problem hiding this comment.
@MikiDi Ik kijk daar direct even voor.
There was a problem hiding this comment.
@MikiDi Zie comment binnen kanselarij-vlaanderen/au-kaleidos-css#9
Maak ik daarvoor dan nog een issue aan? (ik neem die dan wel op)
