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

Serifs eruit #701

Closed
Wolfr opened this issue Feb 16, 2021 · 5 comments
Closed

Serifs eruit #701

Wolfr opened this issue Feb 16, 2021 · 5 comments
Assignees
Labels
css-refactor This code exists on the css-refactor branch good first issue Good for newcomers validated

Comments

@Wolfr
Copy link
Contributor

Wolfr commented Feb 16, 2021

Algemeen: serif fonts eruit halen en typografie van design volgen.

Overal waar we een serif font gebruiken, code genre <h3 class="vl-title vl-title--h3">, dit vervangen door de auk-equivalent.

@Wolfr Wolfr added the good first issue Good for newcomers label Feb 16, 2021
@brenner-company brenner-company self-assigned this Feb 16, 2021
@brenner-company
Copy link
Contributor

brenner-company commented Feb 16, 2021

Aanpassingen op de volgende branch: DES/enhancement/#701-remove-serif-font

@brenner-company
Copy link
Contributor

brenner-company commented Feb 16, 2021

Mixed met #705:

Binnen het project zijn er nog redelijk veel titels aanwezig van het type vl-title vl-title--h[#]. Deze kunnen eventueel ook vervangen worden, maar de pagina's waar ze binnen staan hebben meestal geen design referentie binnen de Figma mockups of het design is aangepast (om te verbeteren/vernieuwen).

Ofwel laten we deze nog staan tot het correcte design geïmplementeerd kan worden ofwel kunnen ze pagina per pagina aangepast worden (daar is enkel wat visuele controle ivm font-size en margins tegenover elementen nodig).

@Wolfr
Copy link
Contributor Author

Wolfr commented Feb 16, 2021

Van bijna elk scherm bestaat een design hoor; maar zoals je misschien gezien hebt zitten er meestal nog wel meer verbeteringen in dan enkel de titels veranderen.

Maar wachten op een volledige implementatie is niet de bedoeling. Dan gaat het nooit gebeuren. We moeten stapje voor stapje werken. Dus je laatste suggestie is het beste.

@Wolfr
Copy link
Contributor Author

Wolfr commented Feb 16, 2021

Zoals aangegeven in #705, lijken unit testen af te hangen van CSS selectors.

Nochtans was er ooit in het project afgesproken om met data attributes te werken. Hier is precies niet echt aan gehouden.

Ik heb de vraag gesteld (of eerder opmerking gegeven) in het algemene dev kanaal.

Ik denk dat je best eens vraagt hoe je die testen kan runnen lokaal aan iemand van het dev team, ik heb hier zelf geen ervaring mee.

@brenner-company brenner-company added the css-refactor This code exists on the css-refactor branch label Feb 17, 2021
@Wolfr
Copy link
Contributor Author

Wolfr commented Feb 17, 2021

Lijkt me goed, ik zie wel dat dit een weerslag heeft op de look van “Kort bestek”, maar dat is niet zo erg.

@brenner-company brenner-company added validated and removed css-refactor This code exists on the css-refactor branch labels Feb 18, 2021
@Wolfr Wolfr added the css-refactor This code exists on the css-refactor branch label Feb 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
css-refactor This code exists on the css-refactor branch good first issue Good for newcomers validated
Development

No branches or pull requests

2 participants