Fixed Dark Mode and Added Delete modal#52
Conversation
Signed-off-by: Spandan Hota <spandanhota2005@outlook.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Signed-off-by: Spandan Hota <spandanhota2005@outlook.com>
There was a problem hiding this comment.
Code Review
This pull request implements a comprehensive theming system by replacing hardcoded colors with CSS variables across the landing page and forum components, supporting multiple color themes and dark mode. It introduces a new DeleteConfirmationModal component for safer deletions of posts and threads and adds URL auto-linking to the markdown renderer. Additionally, various responsive design improvements and CSS transitions were added. A suggestion was made to refine the URL regex in the markdown component to avoid capturing trailing punctuation.
| } | ||
|
|
||
| const mentionRegex = /(^|[^a-zA-Z0-9_])@([a-zA-Z0-9_]{3,32})/g; | ||
| const urlRegex = /(https?:\/\/[^\s<>"\)]+)/g; |
There was a problem hiding this comment.
The current urlRegex captures trailing punctuation like periods or commas at the end of a sentence (e.g., in Check this: https://google.com.). Consider using a regex that ensures the URL does not end with common punctuation marks unless they are followed by more URL characters.
| const urlRegex = /(https?:\/\/[^\s<>"\)]+)/g; | |
| const urlRegex = /(https?:\/\/[^\s<>"\)]+[^.,!:;?\s])/g; |
|
Sorry for the delay! Buzz deserves better maintainers 😢 |
No description provided.