Skip to content

Conversation

@phil-snowplow
Copy link
Contributor

Introduces a more polished code block component. Also did some general css cleanup.

Changes

Code block improvements:

  • Added a separate light theme and updates the dark theme to make it less....jarringly purple
  • Refined the overall code block style to make it more polished and modern
  • Added a badge that shows the language used in the code block
  • Made sure to use only existing color variables for consistency

General cleanup:

  • Removed unused Tailwind configuration overrides that were no longer needed
  • Minor typography improvements, like better spacing between subsequent H2 and H3 headers

Screenshots

Light them
code-block-light
e

Dark theme
code-block-dark

@netlify
Copy link

netlify bot commented Nov 26, 2025

Deploy Preview for snowplow-docs ready!

Name Link
🔨 Latest commit 4410ca9
🔍 Latest deploy log https://app.netlify.com/projects/snowplow-docs/deploys/692da06edefd3a000823d6d7
😎 Deploy Preview https://deploy-preview-1510--snowplow-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 18 (🔴 down 1 from production)
Accessibility: 91 (no change from production)
Best Practices: 100 (🟢 up 8 from production)
SEO: 95 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Collaborator

@stanch stanch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I kind of glossed over the CSS changes, but the result looks great!

@phil-snowplow
Copy link
Contributor Author

I kind of glossed over the CSS changes, but the result looks great!

TBF the majority of the css diff is auto-linting that I was too lazy to figure out how to revert ¯_(ツ)_/¯

Copy link
Collaborator

@mscwilson mscwilson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't love this specific pair of themes (i can live with it), but the code block looks great. The language badge is a very good addition

@phil-snowplow
Copy link
Contributor Author

I don't love this specific pair of themes (i can live with it), but the code block looks great. The language badge is a very good addition

@mscwilson open to other themes if you have suggestions! I just tried to use one of the existing built-in ones which were somewhat limited.

@mscwilson
Copy link
Collaborator

I asked Claude to create themes based on our brand colours. How do we feel about these?
light
dark

@phil-snowplow
Copy link
Contributor Author

@mscwilson small nits, but I don't love how (1) it's not consistent across light and dark (e.g. teal vs purple comments, black vs teal variable names) and (2) the teal-on-grey comments is a bit hard to read in the light mode. Let me know if you want me to take a stab at updating that.

@mscwilson
Copy link
Collaborator

I've pushed my themes commit, feel free to do whatever with it!

I asked Claude to update it based on your feedback btw, it's more standardised across light/dark now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants