-
Notifications
You must be signed in to change notification settings - Fork 5
Feature/annotation list #236
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds an AnnotationList toolbox item that provides a comprehensive interface for managing and navigating annotations. The feature includes a scrollable list displaying annotations with spatial type icons and class names, bidirectional hover highlighting between the list and canvas, filtering options (show/hide deprecated, group by class), and a collapsible interface. Additionally, a toast notification was added to show annotation position during navigation (e.g., "3 / 10").
Key Changes:
- New
AnnotationListToolboxItemclass with filtering, grouping, and navigation capabilities - Toast notification system for annotation navigation feedback
- Enhanced edit candidate detection logic to prioritize containing annotations
- Integration with existing fly-to functionality
Reviewed Changes
Copilot reviewed 9 out of 11 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| src/version.js | Version bump to 0.21.0 |
| src/toolbox_items/annotation_list.ts | New toolbox item implementation with list UI, filtering, and navigation |
| src/index.js | Added toast notification display and improved edit candidate selection logic |
| src/configuration.ts | Registered AnnotationList toolbox item and added to default order |
| package.json | Version update to match src/version.js |
| demo/multi-class.html | Added AnnotationList to demo toolbox configuration |
| api_spec.md | Documentation for the new AnnotationList toolbox item |
| CHANGELOG.md | Documented new features in version 0.21.0 |
| .github/tasks.md | Updated task tracking with completed implementation steps |
joshua-dean
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, just a few small tweaks.
Co-authored-by: Joshua Dean <[email protected]>
…l into feature/annotation-list
Annotation List
Description
fly_tocalls and shows annotation position in the ordering (e.g., "3 / 10")AnnotationListtoolbox item for managing and navigating annotationsPR Checklist
package.jsonhas been bumped since last releasepackage.jsonandsrc/version.jsnpm installandnpm run buildAFTER bumping the version numberapi_spec.md)changelog.mdBreaking API Changes