-
Notifications
You must be signed in to change notification settings - Fork 4
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
VL-refactor (CSS) #783
VL-refactor (CSS) #783
Conversation
…additions files and delete source code/files
Alle bijkomende aanpassingen zijn gecommit binnen de #705 branch. Niet verwijderd (buiten lijst bovenaan):
|
@ValenberghsSven: Dit zou het moeten zijn qua bijkomende aanpassingen. Ik heb in de tussentijd (heb er niet veel tijd in gestoken natuurlijk) ook al even de meest duidelijke errors (binnen gefaalde testen) er uit kunnen halen. Dit waren dan meestal CSS selectors zijn niet meer correct waren of reverts (van voorgaande aanpassingen) die ik niet doorgevoerd had binnen de testen. Waar mogelijk heb ik dan ook al vlug die CSS selectors vervangen door die data-attributes. Mocht je bij het verder fixen vragen hebben mbt mijn aanpassingen (binnen vl-refactor), laat zeker iets weten (de meeste weet ik vaak nog van buiten)! |
@brenner-company Ik ga de testen wat in orde brengen, er falen er nog 15 ofzo. Deze branch gaat wel een beetje clashen met andere grote refactors dat we aan het doen zijn. |
# Conflicts: # app/components/agenda/agendaitem/agendaitem-case/agendaitem-mandatees.hbs # app/components/cases/subcase-mandatees.hbs # app/components/subcase/subcase-case/subcase-mandatees.hbs # app/components/utils/minister-modal.hbs # app/components/web-components/vl-mandatee-link.hbs # cypress/support/commands/subcase-commands.js
…/kanselarij-vlaanderen/kaleidos-frontend into DES/enhancement/#705-vl-refactor
Via merge-commit in 852a254
# Conflicts: # app/components/subcases/subcase-detail-nav.hbs
# Conflicts: # app/components/agenda/agenda-detail/sidebar-item.hbs
Referentie issue: #705 'Algemene todo: Webcomponenten 3 .vl- eruit krijgen'
Ik heb geprobeerd goed te letten op tests die vertrouwen op specifieke CSS selectors binnen de refactor, maar zou verbazen mochten hier geen faulty tests uit komen.
Volgende componenten zijn gerefactored:
* code binnen bron/bestanden is nog niet 100% verwijderd
Volgende componenten zijn momenteel as-is gelaten:
Volgende componenten zijn toegevoegd/opgesplitst:
om VL-componenten te refactoren en niet direct een verandering in functionaliteit/look-and-feel te veroorzaken heb ik nog enkele tijdelijke styling rules toegevoegd. Voor au-kaleidos-css kunnen die gevonden worden onder
au-kaleidos-css/auk-additions
.