feat: Enhance sidebar navigation with Link component and improve layout consistency - #399
Open
criptogus wants to merge 5 commits into
Open
feat: Enhance sidebar navigation with Link component and improve layout consistency#399criptogus wants to merge 5 commits into
criptogus wants to merge 5 commits into
Conversation
…ut consistency refactor: Clean up TranscriptionService test by removing unused base64 data refactor: Simplify agent model configuration logic in intelligent-router chore: Remove unused PDF creation helper function in assembler-tools fix: Update userId parameter to _userId in IntelligentToolSelector style: Adjust loading fallback component width in Dashboard for better responsiveness
- TodayCardShell: Add responsive height breakpoints (min-h/max-h) for sm and lg screens - TodayOverview: Add overflow-hidden, reduce padding/gaps on mobile, smaller font sizes - DailyPlanningScore: Smaller score circle on mobile, reduced gaps and font sizes, proper overflow handling https://claude.ai/code/session_01SBZBXwRoc7d5NLSpxC7C1D
- AppLayout: Add flex and h-full to main content for proper height distribution - Dashboard: Add h-full to container, reduce padding on mobile (p-1.5/p-2 instead of p-2/p-3) - Dashboard: Optimize messages and input areas padding for mobile (p-2 sm:p-3 lg:p-4) - CyberpunkChat: Replace 100vh with 100svh for proper mobile viewport handling - CyberpunkChat: Add iOS Safari fallback with -webkit-fill-available - CyberpunkChat: Reduce padding on mobile for better space utilization https://claude.ai/code/session_01SBZBXwRoc7d5NLSpxC7C1D
fix: Improve Today page cards responsiveness for mobile devices
- Create new ChatPage.tsx with clean component architecture - Add ChatMessages.tsx for isolated message rendering - Add ChatHeader.tsx for header with conversation management - Update routes to use new ChatPage instead of Dashboard - Fix lazy loading for named exports in modal components - Improve state isolation to prevent sidebar navigation issues https://claude.ai/code/session_01CuJcSef3wimqku6s5q4kHT
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
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
2 participants
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.
refactor: Clean up TranscriptionService test by removing unused base64 data
refactor: Simplify agent model configuration logic in intelligent-router
chore: Remove unused PDF creation helper function in assembler-tools
fix: Update userId parameter to _userId in IntelligentToolSelector
style: Adjust loading fallback component width in Dashboard for better responsiveness