Skip to content

Conversation

@justFadel19
Copy link
Collaborator

This pull request introduces a modal-based interface for editing annotation labels in the SAT Annotator, along with related UI enhancements and functionality updates. Key changes include the addition of a new modal, updated event handling for label editing, and improved styling for responsiveness and usability.

New Feature: Edit Label Modal

  • Added a modal (editLabelModal) in web/index.html for editing annotation labels, including dropdown and custom input options for labels.
  • Introduced new methods in AnnotationManager (showEditLabelModal, closeEditLabelModal, saveEditedLabel, etc.) in web/js/annotations.js to handle modal interactions and label updates. [1] [2]

Event Handling Enhancements

  • Added event listeners for modal actions (e.g., close, save, dropdown changes) in AnnotationManager to improve user interaction.
  • Updated button actions to use window.annotationManager to ensure proper method scoping. [1] [2]

UI and Styling Improvements

  • Updated web/styles.css to style the modal, including animations, responsive design for mobile, and improved aesthetics for buttons and inputs. [1] [2]
  • Added a .modal.show class to toggle modal visibility and adjusted modal dimensions for better usability.

Bug Fixes and Error Handling

  • Enhanced error handling in editAnnotation and modal-related methods to log issues when elements or annotations are not found. [1] [2]

These changes collectively improve the user experience for editing annotation labels, making the interface more intuitive and visually appealing.

- Redesigned modal with clean, modern styling
- Made modal compact yet spacious (380px x 320px)
- Added gradient backgrounds and smooth hover effects
- Removed suggestions section for simplified UI
- Fixed JavaScript errors with null element checks
- Improved button styling with subtle animations
- Enhanced mobile responsiveness
- Removed focus effects as requested
- Streamlined user experience for label editing
- Added logo.png as favicon in HTML head tag
- Copied logo.png from docs to web directory for proper access
- Favicon will now display in browser tabs and bookmarks
@AhmedFatthy1040 AhmedFatthy1040 merged commit 9b43283 into AhmedFatthy1040:main Jun 15, 2025
3 checks passed
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