Skip to content

Dev#4

Merged
dev-Ninjaa merged 10 commits into
mainfrom
dev
Jan 30, 2026
Merged

Dev#4
dev-Ninjaa merged 10 commits into
mainfrom
dev

Conversation

@dev-Ninjaa

Copy link
Copy Markdown
Member

No description provided.

dev-Ninjaa and others added 10 commits January 30, 2026 00:34
merge: Integrate main branch features into dev branch
Fix editor cursor jump and @mentions
…ollection

- Create new FeedbackPanel component with modal interface for collecting user feedback
- Implement feedback type selection (bug, feature request, general feedback)
- Add textarea input for detailed feedback messages
- Include keyboard escape handler to close panel
- Display app version information in feedback submission
- Add success state with confirmation message after submission
- Implement panel state reset on close with 200ms delay
- Log feedback data to console with timestamp and app version metadata
- Style panel with stone color scheme matching existing design system
- Add backdrop blur effect and proper z-index layering for modal overlay
- Import and integrate FeedbackPanel component for user feedback collection
- Add isFeedbackPanelOpen state to manage feedback panel visibility
- Add onOpenFeedback callback to sidebar for opening feedback panel
- Improve handleSelectNote to prevent opening deleted notes
- Enhance handleDeleteForever to clean up all mentions of deleted notes in other notes
- Update affected notes' lastUpdatedAt timestamp when mentions are removed
- Refactor note selection logic with early return for deleted notes
…andling

- Add try-catch error handling to formatDate function to gracefully handle invalid dates
- Return original date string if parsing fails instead of crashing
- Add validation checks for NaN dates before processing
- Replace ISO string date parsing with local date formatting to avoid timezone issues
- Handle both Date objects and string date inputs in getDaysWithActivity
- Add validation for each date field (lastOpenedAt, updatedAt, createdAt) before adding to set
- Improve getNotesForDate helper to use local date formatting and handle invalid dates
- Add error logging for invalid dates to aid debugging
- Prevent sidebar from breaking when notes contain malformed date values
- Import MessageSquare icon from lucide-react for feedback button
- Add onOpenFeedback callback prop to SidebarProps interface
- Add onOpenFeedback parameter to Sidebar component destructuring
- Add feedback button to sidebar navigation menu with MessageSquare icon
- Position feedback button in the navigation section alongside search and other actions
- Enable users to open feedback panel directly from sidebar for improved accessibility
…t handling

- Add useRef hook to track menu element for accurate click-outside detection
- Replace 'click' event listener with 'mousedown' for more reliable event capture
- Implement proper click-outside logic that checks if click target is within menu
- Remove stopPropagation from menu div as it's no longer needed
- Reorder className properties for consistency (flex items before padding)
- Improve code formatting and readability in event handlers
- Prevent menu from closing immediately on open by maintaining setTimeout delay
- Replace target element reference with editor container query selector for more reliable positioning
- Update menu position calculation to use editor container bounds instead of event target bounds
- Adjust vertical position to use rect.bottom instead of rect.top for better menu placement
- Add 10px offset below selection for improved visual spacing
- Add null check for editor container to prevent positioning errors when container is unavailable
- Fixes context menu appearing at incorrect coordinates when triggered from nested elements
- Remove stopPropagation() call from keydown handler to allow proper event bubbling
- Remove duplicate overflow-y-auto class from menu container
- Simplify positioning by removing unnecessary transform translateY values
- Improve event handling to prevent unintended side effects on parent elements
@dev-Ninjaa dev-Ninjaa merged commit 96f7375 into main Jan 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants