Fix Settings, Scrollbar and Docs#26
Merged
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR addresses issues #15, #13, and #12 by implementing theming improvements for the documentation pages, customizing scrollbar styling to match the dark theme, and adding flexibility to the Card component through a new style prop.
Key Changes:
- Added a
.docs-themeCSS class to override default documentation styles and align them with the site's card-based UI - Implemented custom scrollbar styling with fixed colors and thinner width for better dark theme integration
- Enhanced the Card component to accept inline styles via a new
styleprop
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
app/src/components/ui/Card/Card.tsx |
Added style prop to CardProps interface and merged it with default styles |
app/src/components/Settings/Settings.tsx |
Applied inline style to Card component for responsive height |
app/src/app/globals.css |
Added .docs-theme overrides and customized scrollbar colors/width |
app/src/app/docs/layout.tsx |
Applied .docs-theme class, centered content, and adjusted sidebar positioning |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
This was
linked to
issues
Oct 22, 2025
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
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.
Fixes for #15 #13 #12
This pull request updates the documentation layout and theme to better align with the overall site UI, improves scrollbar styling, and introduces more flexible styling for the
Cardcomponent. The most notable changes are grouped below.Documentation Theme and Layout Enhancements:
.docs-themeCSS class inglobals.cssto override default background, border, and text color utilities within the docs, ensuring consistent appearance with the site’s card-based UI. This includes custom styles for code blocks, prose, and links.DocsLayoutcomponent to wrap children in a.docs-themecontainer and center content with a maximum width, improving readability and design consistency. The sidebar layout was also adjusted to account for the top navigation bar.UI Component Improvements:
Cardcomponent to accept astyleprop, allowing inline styles to be merged with default styles. TheSettingspage now uses this to set the card height responsively. [1] [2] [3] [4]Scrollbar Styling:
globals.cssto better match the app’s dark theme, using fixed background and thumb colors and making the scrollbar thinner.