-
Notifications
You must be signed in to change notification settings - Fork 0
Refactor UI components, enhance markdown support, and improve accessibility #59
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
aditya-2k23
wants to merge
29
commits into
main
Choose a base branch
from
refactor/design
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
deaa36a
refactor: replace Toaster with CustomToaster component and update sty…
aditya-2k23 b763c74
Add react-markdown dependency to enhance markdown rendering capabilities
aditya-2k23 551e036
feat: add StyleTools component for markdown formatting and enhance Jo…
aditya-2k23 f3e6ac7
fix: change console.log to console.error for better error handling in…
aditya-2k23 36a326e
refactor: optimize deleteCollectionTree function to use batching and …
aditya-2k23 6c52d03
refactor: restructure Firebase initialization to handle missing envir…
aditya-2k23 f52c83e
chore: update version to 3.1.0 in package.json and remove redundant r…
aditya-2k23 47f8804
refactor: enhance styling and transitions for UI components in global…
aditya-2k23 ded8d03
refactor: extract design tokens for consistent theming and update glo…
aditya-2k23 d866ae3
Add richtext markdown style formatting for text in journal textarea a…
aditya-2k23 907bfae
journal textbox autosize and page-scroll
aditya-2k23 14fe3bb
fine I'll work with this!
aditya-2k23 8de362d
Fixed some color styling
aditya-2k23 566620a
Done with the animations
aditya-2k23 df2a5f3
style(ui): improve background styling and dark mode opacity
aditya-2k23 e480a45
chore(security): allow loom.com in CSP headers
aditya-2k23 2fab408
style(ui): refine visual consistency and interaction feedback
aditya-2k23 60d6b87
fix(ui): improve accessibility, save logic, and component stability
aditya-2k23 79aabd2
feat(ui): enhance footer with logo and creator attribution
aditya-2k23 9a9332e
style(ui): enhance button spacing and text visibility in AIInsightsSe…
aditya-2k23 f3d9c71
style(css): update transition properties and improve gradient definit…
aditya-2k23 1cbb618
fix(ui): enhance toast animations, improve input focus styles, and up…
aditya-2k23 6ac6ab8
feat(ui): enhance journal prompt instructions with rich text formatti…
aditya-2k23 90d74aa
feat(ui): integrate Loader component for loading states in Logout, Fi…
aditya-2k23 ab07edd
feat(ui): implement smooth splash screen animation and enforce minimu…
aditya-2k23 a931c48
feat(ui): enhance chat experience with Tiptap editor integration, mar…
aditya-2k23 964819b
fix(api): sanitize chat content and improve message previews in chat …
aditya-2k23 7c80d49
feat(ui): enhance chat components with markdown support and style imp…
aditya-2k23 3f16432
feat(ui): add underline support to RichTextEditor and StyleTools; enh…
aditya-2k23 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -43,3 +43,5 @@ yarn-error.log* | |
| next-env.d.ts | ||
|
|
||
| .vscode | ||
| # Ignore Git Commit Genie data | ||
| .gitgenie/** | ||
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Inconsistent quote-stripping regex between assistant bubbles and other paths.
Line 70 and Line 93 use
/^["']+|["']+$/g(greedy — strips all leading/trailing quote chars), while Line 86 uses/^["']|["']$/g(strips only a single char on each end). For the same source content, the preview, assistant bubble content, and non-assistant content can therefore disagree on how much quoting is removed. Pick one regex and apply consistently — preferably a "single matched pair" pattern so legitimate dialogue quotes within messages aren't clobbered:♻️ Suggested helper
…then call
stripWrappingQuotes(...)at all three sites.This issue is paired with the same concern raised in
app/api/chat/route.jsLines 595-597.Also applies to: 86-86, 91-94
🤖 Prompt for AI Agents