Skip to content

Conversation

@unique-Sachin
Copy link

@unique-Sachin unique-Sachin commented Jun 30, 2025

  • Refactored note title retrieval to use getDisplayTitle instead of getNoteTitle for better title management.
  • Added rename functionality in the context menu for notes, allowing users to edit note titles directly.
  • Enhanced note editing state management in the Redux slice to support temporary title changes.
  • Updated various components to reflect changes in note title handling and ensure consistent display across the application.

Description

Please read the Contribution Guidelines before opening a pull request. Include a summary of the change and which issue is fixed.

Closes #620

Browser checklist

This PR has been tested in the following browsers:

  • Chrome
  • Firefox
  • Safari

Testing checklist

  • End-to-end tests have been created if necessary

Screenshots

Screenshot 2025-07-01 at 12 33 31 AM Screenshot 2025-07-01 at 12 34 35 AM

- Upgraded package-lock.json to version 3 and replaced node-sass with sass.
- Refactored note title retrieval to use getDisplayTitle instead of getNoteTitle for better title management.
- Added rename functionality in the context menu for notes, allowing users to edit note titles directly.
- Enhanced note editing state management in the Redux slice to support temporary title changes.
- Updated various components to reflect changes in note title handling and ensure consistent display across the application.
…tracking

- Updated the regex pattern used in the setEditorOverlay function to be bounded and non-nested, enhancing performance and preventing potential issues with super-linear backtracking.
- Replaced anchor tags with buttons in NoteLink for better accessibility.
- Added keyboard event handling for Tab and Category components to support Enter and Space key interactions.
- Updated event handling in NoteList and CategoryList to manage keyboard events for context menus and note options.
- Improved context menu positioning for keyboard events to enhance user experience.
@unique-Sachin
Copy link
Author

@uplabhijit May you please approve with right access

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.

Add option to rename notes

2 participants