Dev#4
Merged
Merged
Conversation
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
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
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.
No description provided.