Skip to content

feat: FIT-968: Improve annotation tab in Data Manager Quick View and Label/Review Stream (handle long project names, remove id, add items to the menu) #8840

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.

Sign up for GitHub

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

Jump to bottom
Merged
robot-ci-heartex merged 4 commits into from
Nov 19, 2025
Merged

feat: FIT-968: Improve annotation tab in Data Manager Quick View and Label/Review Stream (handle long project names, remove id, add items to the menu) #8840

robot-ci-heartex merged 4 commits into from
Nov 19, 2025

Conversation

@yyassi-heartex
Copy link
Contributor

@yyassi-heartex yyassi-heartex commented Nov 18, 2025

This pull request enhances the AnnotationButton component in the Annotations Carousel by adding new context menu actions, improving user interface elements, and introducing feature flag-based logic for dashboard navigation. The most significant changes are grouped below.

image

New Context Menu Actions

  • Added "Copy Annotation ID" action, allowing users to copy the annotation's ID to the clipboard. [1] [2]
  • Added "Open Performance Dashboard" action (Enterprise only), with routing logic that respects a feature flag to determine the correct dashboard URL. [1] [2]
  • Added "Show Other Annotations" action to toggle viewing all annotations. [1] [2]

UI and Usability Improvements

  • Truncated long usernames in the annotation button UI with ellipsis, max width, and tooltip for full display, improving readability. [1] [2] [3]
  • Updated the annotation button to use a tooltip for displaying the full username, ensuring that truncated names are accessible.

Technical Enhancements

  • Imported new icons (IconAnalytics, IconViewAll, IconClipboardCheck) and utility functions for feature flag checks.
  • Updated dependencies and memoization logic to include new handlers and state variables.

@yyassi-heartex
feat: FIT-968: Improve annotation tab in Data Manager Quick View and …
71b2590 
…Label/Review Stream (handle long project names, remove id, add items to the menu)
@netlify
Copy link

netlify bot commented Nov 18, 2025 •
edited
Loading

✅ Deploy Preview for label-studio-docs-new-theme canceled.

Name Link
🔨 Latest commit d362e2a
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-docs-new-theme/deploys/691da96390c8c400081484c1

@netlify
Copy link

netlify bot commented Nov 18, 2025 •
edited
Loading

✅ Deploy Preview for heartex-docs canceled.

Name Link
🔨 Latest commit d362e2a
🔍 Latest deploy log https://app.netlify.com/projects/heartex-docs/deploys/691da96315ab5d0008e8aa61

@netlify
Copy link

netlify bot commented Nov 18, 2025 •
edited
Loading

✅ Deploy Preview for label-studio-storybook ready!

Name Link
🔨 Latest commit d362e2a
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-storybook/deploys/691da9630cf962000852943f
😎 Deploy Preview https://deploy-preview-8840--label-studio-storybook.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@yyassi-heartex yyassi-heartex requested a review from a team November 18, 2025 16:51
@github-actions github-actions bot added the feat label Nov 18, 2025
@netlify
Copy link

netlify bot commented Nov 18, 2025 •
edited
Loading

✅ Deploy Preview for label-studio-playground canceled.

Name Link
🔨 Latest commit d362e2a
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-playground/deploys/691da9632053aa000894e361

@codecov
Copy link

codecov bot commented Nov 18, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 27.58621% with 21 lines in your changes missing coverage. Please review.
✅ Project coverage is 58.48%. Comparing base (ef83d11) to head (d362e2a).
⚠️ Report is 2 commits behind head on develop.

Files with missing lines Patch % Lines
...omponents/AnnotationsCarousel/AnnotationButton.tsx 27.58% 21 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (ef83d11) and HEAD (d362e2a). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (ef83d11) HEAD (d362e2a)
pytests 1 0
Additional details and impacted files 
@@             Coverage Diff             @@
##           develop    #8840      +/-   ##
===========================================
- Coverage    65.87%   58.48%   -7.40%     
===========================================
  Files          812      554     -258     
  Lines        63602    40231   -23371     
  Branches     10761    10765       +4     
===========================================
- Hits         41899    23528   -18371     
+ Misses       21699    16699    -5000     
  Partials         4        4
Flag Coverage Δ
lsf-e2e 52.62% <27.58%> (+2.27%) ⬆️
lsf-integration 49.09% <27.58%> (-0.02%) ⬇️
lsf-unit 8.39% <0.00%> (-0.01%) ⬇️
pytests ?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@yyassi-heartex yyassi-heartex marked this pull request as draft November 19, 2025 11:24
@yyassi-heartex yyassi-heartex marked this pull request as ready for review November 19, 2025 11:24
@robot-ci-heartex robot-ci-heartex merged commit c5286af into develop Nov 19, 2025
44 of 46 checks passed
@robot-ci-heartex robot-ci-heartex deleted the fb-fit-968/annotation-button branch November 19, 2025 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bmartel bmartel bmartel approved these changes

@hlomzik hlomzik Awaiting requested review from hlomzik hlomzik is a code owner

@nick-skriabin nick-skriabin Awaiting requested review from nick-skriabin nick-skriabin is a code owner

Assignees

No one assigned

Labels

feat

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@yyassi-heartex @bmartel @robot-ci-heartex