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

VL-refactor (CSS) #783

Merged
merged 204 commits into from
Apr 8, 2021
Merged

VL-refactor (CSS) #783

merged 204 commits into from
Apr 8, 2021

Conversation

brenner-company
Copy link
Contributor

@brenner-company brenner-company commented Mar 23, 2021

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:

  • action-group
  • badge
  • button
  • checkbox
  • data-table
  • description-data
  • form-message
  • form-structure
  • icon
  • input-field*
  • link
  • loader
  • modal
  • pager
  • pill
  • radio
  • tabs
  • textarea
  • titles
  • typography

* code binnen bron/bestanden is nog niet 100% verwijderd

Volgende componenten zijn momenteel as-is gelaten:

  • accordion
  • alert
  • datepicker (refactor met auk-component)
  • document
  • input-addon (hangt vast aan datepicker refactor)
  • input-group (hangt vast aan datepicker refactor)
  • upload

Volgende componenten zijn toegevoegd/opgesplitst:

  • toggle (opgesplitst na verwijdering checkbox component)

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.

@brenner-company
Copy link
Contributor Author

Alle bijkomende aanpassingen zijn gecommit binnen de #705 branch.


Niet verwijderd (buiten lijst bovenaan):

  • .vl-editor [editor.scss]: nog in gebruik binnen vl-rdfa-editor
  • .vl-upload__element & .vl-uploaded-document [file-uploader.scss]: nog in gebruik binnen file-uploader & vl-uploaded-document

@brenner-company
Copy link
Contributor Author

brenner-company commented Mar 25, 2021

@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)!

@ValenberghsSven
Copy link
Contributor

ValenberghsSven commented Mar 30, 2021

@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.
Ongeacht welke we eerst mergen, we moeten een controle doen of er geen vl-classes teruggekomen zijn enzo

@MikiDi
Copy link
Contributor

MikiDi commented Apr 1, 2021

Nu ook alle vl-'s die terug binnengeslopen waren door merge 852a254 en c4b6f7f recht gezet.

@brenner-company brenner-company marked this pull request as ready for review April 8, 2021 15:48
@erikap erikap merged commit fac423d into development Apr 8, 2021
@ValenberghsSven ValenberghsSven deleted the DES/enhancement/#705-vl-refactor branch April 15, 2021 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants