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

[Sistent] Feat: JSX syntax highlighting on code pages of sistent #5535

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lakshz
Copy link
Contributor

@lakshz lakshz commented May 12, 2024

Description

This PR fixes #5534

Before

Screenshot 2024-05-12 at 15 20 51

After

  • Light theme
Screenshot 2024-05-12 at 15 21 12
  • Dark Theme
Screenshot 2024-05-12 at 15 21 49

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

Signed-off-by: lakshya <lakshyasatpal372@gmail.com>
@l5io
Copy link
Contributor

l5io commented May 12, 2024

🚀 Preview for commit 38b3d74 at: https://66409464683dc4aea8e762ba--layer5.netlify.app

@ritiksaxena124
Copy link
Contributor

ritiksaxena124 commented May 13, 2024

Hey @lakshz, let's discuss this on today's website meeting at 5:30 PM IST. I am adding this as an agenda item. Please Join https://meet.layer5.io/websites if you are available during the meet.

@Rexford74
Copy link

I'm dropping an approving review to this. I wonder if there are any contrary opinions @ritiksaxena124?.. 👀

@Rexford74 Rexford74 self-requested a review May 15, 2024 13:19
@sudhanshutech
Copy link
Member

@lakshz can you remove the Sistentthemeprovider wrapped from every component. It should be only mentioned at one place because it is wrapped globally and not on each comps.
It can be mentioned at about or its guidance // @Rexford74 @nebula-aac your thoughts?

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

Successfully merging this pull request may close these issues.

[Sistent] Code syntax highlighting for sistent components
5 participants