Skip to content

fix: LEAP-2064: Choices on imported annotations are not displayed in View All on initial load #7488

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

Merged
merged 4 commits into from
May 8, 2025

Conversation

nass600
Copy link
Contributor

@nass600 nass600 commented May 7, 2025

There are two flags at play related with this bug:

  • fflag_fix_front_dev_3391_interactive_view_all
  • fflag_fix_front_leap_443_select_annotation_once

The issue comes from selecting "Compare all annotations" view and having the flag fflag_fix_front_leap_443_select_annotation_once enabled given that introduces performance improvements so we can keep the interactivity of the annotations while we do not select all of them just the current one.

Now, during the select process, control tags (Choices, Taxonomy, Rating, DateTime, Number) values are computed so they can be rendered with the right checked/unchecked state and having this flag enabled skips this process.

The fix introduces the computation of these selections based on the results when the user enters in the "Compare all annotation" mode without "selecting" them

Additionally, the PR will fix some cosmetic issues with antd checkboxes

Copy link

netlify bot commented May 7, 2025

Deploy Preview for label-studio-docs-new-theme ready!

Name Link
🔨 Latest commit 06f21dc
🔍 Latest deploy log https://app.netlify.com/sites/label-studio-docs-new-theme/deploys/681cb39da2e9150008158738
😎 Deploy Preview https://deploy-preview-7488--label-studio-docs-new-theme.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 site configuration.

Copy link

netlify bot commented May 7, 2025

Deploy Preview for label-studio-storybook ready!

Name Link
🔨 Latest commit 06f21dc
🔍 Latest deploy log https://app.netlify.com/sites/label-studio-storybook/deploys/681cb39d34cfe00008a95720
😎 Deploy Preview https://deploy-preview-7488--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 site configuration.

@github-actions github-actions bot added the fix label May 7, 2025
Copy link

netlify bot commented May 7, 2025

Deploy Preview for heartex-docs ready!

Name Link
🔨 Latest commit 06f21dc
🔍 Latest deploy log https://app.netlify.com/sites/heartex-docs/deploys/681cb39df3333b00080af9ce
😎 Deploy Preview https://deploy-preview-7488--heartex-docs.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 site configuration.

@nass600
Copy link
Contributor Author

nass600 commented May 8, 2025

/git merge

Workflow run
Successfully merged: create mode 100644 docs/source/guide/install_prompts.md

@nass600 nass600 merged commit 41038bd into develop May 8, 2025
38 checks passed
@robot-ci-heartex robot-ci-heartex deleted the fb-leap-2064/choices-view-all branch May 8, 2025 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants