Skip to content

feat 215: Comments section#18

Closed
andrxpie wants to merge 78 commits into
mainfrom
feat/215/comments-section
Closed

feat 215: Comments section#18
andrxpie wants to merge 78 commits into
mainfrom
feat/215/comments-section

Conversation

@andrxpie

Copy link
Copy Markdown
Contributor

Overview

This PR implements the initial routing configuration and the sidebar (Drawer) shell for the commenting system on the Streetcode page. This foundation ensures that comments can be accessed via a specific sub-route (/comments) while keeping the main content visible in the background.

Changes Made

  • Routing Configuration: Added a new route structure in React Router to intercept URL navigation and trigger the comment sidebar.
  • Drawer Shell: Created the CommentsDrawerContainer component using Ant Design's Drawer.
  • Navigation Behavior: Implemented logic to ensure:
    • The drawer opens automatically when the URL contains /comments.
    • Navigating back closes the drawer and redirects the user to the base Streetcode URL.
    • Page refreshes on the comments sub-path maintain the correct background state.

Technical Notes

  • The drawer is positioned exclusively on the right side of the screen.
  • Used relative: 'path' configuration in React Router to prevent unintended redirects to the root page.

Checklist

  • My code follows the style guidelines of this project.
  • I have performed a self-review of my code.
  • The routing logic handles page refreshes correctly.

andrxpie and others added 30 commits May 19, 2026 20:52
…m-dependencies-to-latest-stable-versions

Chore 83: update npm dependencies to latest stable versions
feat/33: Add fact crud
Removed specific Visual Studio files and directories from .gitignore.
@andrxpie andrxpie self-assigned this Jun 10, 2026
@andrxpie andrxpie added enhancement New feature or request frontend labels Jun 10, 2026
@sonarqubecloud

Copy link
Copy Markdown

@andrxpie andrxpie closed this Jun 10, 2026
@andrxpie

Copy link
Copy Markdown
Contributor Author

Create into wrong branch.
New PR: #19

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

Labels

enhancement New feature or request frontend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants