-
Notifications
You must be signed in to change notification settings - Fork 3.3k
fix: BROS-641: Select annotation from username in Compare #8898
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
We have a regression: username/userpic still have pointer cursor in Compare table in View All, but click on them does nothing. The reason is the change in `selectAnnotation()` method — now it doesn't close View All by default, we need to pass an option `exitViewAll: true` to it.
✅ Deploy Preview for label-studio-storybook ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for heartex-docs canceled.
|
✅ Deploy Preview for label-studio-playground ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for label-studio-docs-new-theme canceled.
|
Codecov Report✅ All modified and coverable lines are covered by tests.
Additional details and impacted files@@ Coverage Diff @@
## develop #8898 +/- ##
===========================================
- Coverage 65.09% 57.74% -7.36%
===========================================
Files 828 561 -267
Lines 64773 40877 -23896
Branches 11041 11041
===========================================
- Hits 42164 23603 -18561
+ Misses 22605 17270 -5335
Partials 4 4
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
They were not actually selecting new annotation
|
/git merge
|
We have a regression: username/userpic still have pointer cursor in Compare table in View All, but click on them does nothing.
The reason is the change in
selectAnnotation()method — now it doesn't close View All by default, we need to pass an optionexitViewAll: trueto it.Also fixed "Create New Annotation" and "Duplicate Annotation" actions.