KAS-2689 (DES-802): Elementen/structuur omzetten naar Octane componenten - toolbars #987
Conversation
…ctane-component-refactor/KAS-2689-toolbars # Conflicts: # app/components/access-level-pill.hbs # app/components/agenda/agenda-header.hbs # app/components/agenda/side-nav.hbs # app/components/cases/cases-header.hbs # app/components/cases/new-case.hbs # app/components/cases/new-subcase.hbs # app/components/newsletter/newsletter-header-overview.hbs # app/components/subcases/subcase-header.hbs
|
|
…ctane-component-refactor/KAS-2689-toolbars
…ctane-component-refactor/KAS-2689-toolbars
| @@ -41,23 +41,21 @@ | |||
| /> | |||
| {{/if}} | |||
| {{/if}} | |||
| <div class="auk-toolbar-complex__item"> | |||
There was a problem hiding this comment.
Het wegvallen van de auk-toolbar-complex__item-klasse breekt een test ... na te kijken http://kal-kastaar.s.redpencil.io:8080/job/kaleidos/job/FeatureBranches/view/change-requests/job/PR-987/2/
There was a problem hiding this comment.
zal daar eens naar kijken
There was a problem hiding this comment.
Dit lag blijkbaar aan een .vlc-document-card-toolbar__right die er door mij uitgehaald was. Ik heb die nu vervangen door een meer gebruiksvriendelijke data-attribute.
@ValenberghsSven Kan je zelf nog eens vlug kijken of dit ok is?
There was a problem hiding this comment.
ik heb de fix iets anders gedaan, gewoon document.documentCard.card gebruikt.
In cypress fix deel 4 gaan we deze test toch herwerken en dan kunnen we document.accessLevelPill.pill gebruiken etc
…factor/KAS-2687-navbars' of github.com:kanselarij-vlaanderen/kaleidos-frontend into DES/enhancement/802-html-element-to-octane-component-refactor/KAS-2689-toolbars
|
heb de navbar PR hierin gemerged, er waren een paar moeilijke conflicten in newsletter-header-overview.hbs maar zou moeten ok zijn |
|
@ValenberghsSven met de aanpassingen die je binnen DES-802 don't mix (on click) with (action), complete octane refactor gedaan hebt zie ik nu wel volgende error verschijnen als die knoppen gebruikt binnen
|
| @@ -41,23 +41,21 @@ | |||
| /> | |||
| {{/if}} | |||
| {{/if}} | |||
| <div class="auk-toolbar-complex__item"> | |||
There was a problem hiding this comment.
ik heb de fix iets anders gedaan, gewoon document.documentCard.card gebruikt.
In cypress fix deel 4 gaan we deze test toch herwerken en dan kunnen we document.accessLevelPill.pill gebruiken etc
…os-frontend into DES/enhancement/802-html-element-to-octane-component-refactor/KAS-2689-toolbars
ValenberghsSven
deleted the
DES/enhancement/802-html-element-to-octane-component-refactor/KAS-2689-toolbars
branch
Aanpassingen binnen deze PR:
auk-toolbar(-...)classes bevatten omzetten (met enkele uitzonderingen waar dit niet mogelijk is of waar dit met een latere refactor aan bod komt).Ticket binnen Jira: https://kanselarij.atlassian.net/browse/KAS-2689
Issue (algemeen) binnen repo: #802