-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
chore: update code syntax highlighting in 3000 #19104
Conversation
Size Change: +153 kB (+8%) 🔍 Total Size: 1.99 MB
|
📸 UI snapshots have been updated2 snapshot changes in total. 0 added, 2 modified, 0 deleted:
Triggered by this commit. |
📸 UI snapshots have been updated1 snapshot changes in total. 0 added, 1 modified, 0 deleted:
Triggered by this commit. |
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.
@corywatilo I used the accent color as a background for the code snippet but as in this screenshot it can sometimes appear on top of an accent background. Any color we should use to avoid clashes?
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.
Can you point me to an example?
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.
This screenshot shows it, basically this portion:
The code snippet has the same background as the settings panel. https://app.posthog.com/notebooks#panel=settings if you want to see it in action
📸 UI snapshots have been updated1 snapshot changes in total. 0 added, 1 modified, 0 deleted:
Triggered by this commit. |
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.
Want to add a story with include3000
perhaps? 😃
Code looks good to me, but I'll also let @corywatilo check how this matches the website's style.
📸 UI snapshots have been updated1 snapshot changes in total. 0 added, 1 modified, 0 deleted:
Triggered by this commit. |
Problem
Our code blocks in the app do not match the marketing site
Changes
Sadly the
prism-react-renderer
package we use on the marketing site does not use the exact same theming systems asreact-syntax-highlighter
(we should possibly change that in future). Ported over the custom colors and everything looked decentHow did you test this code?
Visually