-
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
Serifs eruit #701
Comments
Aanpassingen op de volgende branch: DES/enhancement/#701-remove-serif-font |
Mixed met #705: Binnen het project zijn er nog redelijk veel titels aanwezig van het type 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). |
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. |
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. |
Lijkt me goed, ik zie wel dat dit een weerslag heeft op de look van “Kort bestek”, maar dat is niet zo erg. |
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 deauk
-equivalent.The text was updated successfully, but these errors were encountered: