-
Notifications
You must be signed in to change notification settings - Fork 13
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
Trek syntax highlighting gelijk tussen Markdown en <Canvas> #397
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
a068760
to
2fbdd71
Compare
2fbdd71
to
d3793ef
Compare
@hidde Het zit er goed uit, qua frontend (als je al die inline CSS meuk niet meetelt, want echt... bah). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hidde LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm goed dat de pre er nu hetzelfde uit ziet, maar jammer dat daarvoor alles nu in custom.css moet staan. Ik hoop dat we binnenkort tijd hebben om de boel eens even goed in elkaar te draaien, maar voor nu ok
Had de overweging gemaakt tussen iets viezere |
* docs: add descriptions * docs: lint-fix:prettier * Add footer info * typo * Add FormFooterInfo * fix: remove comment * Update _description-target-size.md Visual toegevoegd met begeleidende tekst. * fix: Make syntax highlighting same for MD code blocks and code in a Canvas (#397) * Maak richtlijnen meer zoals Jeffrey's ontwerp (#398) * fix: Align dos and donts better with intended visual design * docs: Make dos and donts uppercase to work better in fixed design * fix: button style on focus --------- Co-authored-by: Yolijn <[email protected]> * fix: Capitalise title to work in new layout * fix: Tweak vertical spacing * fix: Lint * docs: remove double ids --------- Co-authored-by: Hidde de Vries <[email protected]> Co-authored-by: Jeffrey Lauwers <[email protected]> Co-authored-by: Yolijn <[email protected]>
fixes #394
Er zat verschil tussen syntax highlighting in code die:
```
) was toegevoegd<Canvas>
Met deze PR worden die gelijk getrokken, namelijk beide in de kleuren die al in de
Canvas
zaten (want voldoet aan WCAG). Dit introduceert wel wat ongelukkige herhalingen.Wat uitleg:
theme.styles
had de styles in een ander formaat nodig dan we reeds in PrismStyle.ts hadden leven.code
/pre
+ de syntax highlighting related custom properties verhozen vanCanvas
naarcustom.css
, zodat ze werken voor alle codeblokken, inclusief die die niet in eenCanvas
zitten; deze bestaan ook nog inPrismStyle.ts
, omdat die anders worden overschreven door andere inline stylesPrismStyle.ts
is fix: we misten daar--
in de syntaxTer illustratie,
voor:
na: