fix: remove unused dark mode tour step and dead code#709
Merged
mttrbrts merged 3 commits intoaccordproject:mainfrom Feb 25, 2026
Merged
fix: remove unused dark mode tour step and dead code#709mttrbrts merged 3 commits intoaccordproject:mainfrom
mttrbrts merged 3 commits intoaccordproject:mainfrom
Conversation
Signed-off-by: Rahul-R79 <rahul.devworks@gmail.com>
✅ Deploy Preview for ap-template-playground ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR performs a cleanup task by removing the unused ToggleDarkMode component and its associated tour step, as the dark mode functionality has been consolidated into the SettingsModal component. The changes streamline the codebase by eliminating dead code while preserving the dark mode feature in its new location.
Changes:
- Removed the
dark-mode-toggletour step from the guided tour - Deleted the unused
ToggleDarkMode.tsxcomponent file - Deleted the unused
ToggleDarkMode.tsstyled-components file
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/components/Tour.ts | Removed the dark-mode-toggle tour step (lines 300-321) that referenced the now-deleted standalone toggle component |
| src/components/ToggleDarkMode.tsx | Deleted entire component file that provided standalone dark mode toggle functionality now available in SettingsModal |
| src/styles/components/ToggleDarkMode.ts | Deleted styled-components file that defined the styling for the removed ToggleDarkMode component |
mttrbrts
approved these changes
Feb 25, 2026
Shubh-Raj
pushed a commit
to Shubh-Raj/playground
that referenced
this pull request
Mar 4, 2026
Signed-off-by: Rahul-R79 <rahul.devworks@gmail.com> Co-authored-by: Matt Roberts <7544022+mttrbrts@users.noreply.github.com>
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.
Changes
dark-mode-togglestep from src/components/Tour.ts as the toggle is no longer visible on the main screen.editor-settingstour step description in src/components/Tour.ts to inform users that Dark Mode is now available within the Settings modal.Screenshots or Video
N/A (Cleanup task)
Related Issues
Author Checklist
--signoffoption of git commit.mainfromfork:fix/tour-dark-mode-cleanup