[website]add theme-aware images and fix broken links#3497
Open
Prajwal-banakar wants to merge 3 commits into
Open
[website]add theme-aware images and fix broken links#3497Prajwal-banakar wants to merge 3 commits into
Prajwal-banakar wants to merge 3 commits into
Conversation
Contributor
Author
binary-signal
approved these changes
Jun 20, 2026
Contributor
There was a problem hiding this comment.
LGTM. One minor thing I noticed: in the light theme, the search bar seems to use the same opacity as the dark theme, which makes it look a bit too transparent. The hover state looks good, so it might be worth disabling or reducing the transparency for the default state in the light theme.
On hover
Not on hover
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
Linked issue: close #3496
The purpose of this change is to improve the readability and visual consistency of the documentation website by implementing theme-aware diagrams that adapt to the user's selected light or dark mode. This also resolves multiple broken link warnings that occurred after renaming files.
Brief change log
ThemedImage.jscomponent: A new reusable React component was added inwebsite/src/components/to handle the logic for displaying different images based on the current theme.*-light.pngversions of existing diagrams were added towebsite/docs/assets/..mdto.mdx: Several documentation files containing diagrams were renamed from Markdown to MDX to allow the use of React components.<ThemedImage>component in the updated.mdxfiles.client-support-matrix.md,getting-started.md,intro.mdx, etc.) to point to the new.mdxfile paths, resolving all build warnings.Tests
npm run start) and verified that all "couldn't be resolved" warnings related to broken links have been eliminated from the build output.API and Format
No, this change does not affect any public APIs or storage formats. It is a documentation-only enhancement.
Documentation
Yes, this change enhances the existing documentation by improving its visual presentation and fixing navigational issues caused by broken links.