Skip to content

Conversation

@ricardoantoniocm
Copy link
Contributor

This pull request introduces several style adjustments to improve the appearance and accessibility of disabled multi-select dropdowns in the taxonomy control. The main focus is on ensuring consistent visual cues for disabled states.

Disabled multi-select dropdown styling:

  • Updated the background color of disabled multi-select dropdown selectors to use var(--color-neutral-surface) for clearer differentiation.
  • Changed the border color to transparent and text color to var(--color-neutral-content-subtle) for selection items in disabled multi-selects, making the disabled state more visually apparent.
  • Modified the color of the dropdown arrow in disabled selects to use var(--color-neutral-content-subtlest), further reinforcing the disabled state.

Screenshots

Before

image

After

image

@ricardoantoniocm ricardoantoniocm self-assigned this Sep 3, 2025
@netlify
Copy link

netlify bot commented Sep 3, 2025

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

Name Link
🔨 Latest commit 2b9a86f
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-docs-new-theme/deploys/68b856acf0f76300082fd864

@netlify
Copy link

netlify bot commented Sep 3, 2025

Deploy Preview for heartex-docs canceled.

Name Link
🔨 Latest commit 2b9a86f
🔍 Latest deploy log https://app.netlify.com/projects/heartex-docs/deploys/68b856ac1e8a5e00087794a9

@netlify
Copy link

netlify bot commented Sep 3, 2025

Deploy Preview for label-studio-playground ready!

Name Link
🔨 Latest commit 2b9a86f
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-playground/deploys/68b856acf0f76300082fd869
😎 Deploy Preview https://deploy-preview-8327--label-studio-playground.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.

@netlify
Copy link

netlify bot commented Sep 3, 2025

Deploy Preview for label-studio-storybook ready!

Name Link
🔨 Latest commit 2b9a86f
🔍 Latest deploy log https://app.netlify.com/projects/label-studio-storybook/deploys/68b856acd25dfb0008a1ec18
😎 Deploy Preview https://deploy-preview-8327--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.

@codecov
Copy link

codecov bot commented Sep 3, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.49%. Comparing base (911cc40) to head (2b9a86f).
⚠️ Report is 4 commits behind head on develop.

❗ There is a different number of reports uploaded between BASE (911cc40) and HEAD (2b9a86f). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (911cc40) HEAD (2b9a86f)
pytests 1 0
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #8327      +/-   ##
===========================================
- Coverage    66.59%   61.49%   -5.10%     
===========================================
  Files          755      538     -217     
  Lines        55746    37625   -18121     
  Branches      9924     9924              
===========================================
- Hits         37123    23139   -13984     
+ Misses       18620    14483    -4137     
  Partials         3        3              
Flag Coverage Δ
lsf-e2e 55.33% <ø> (+1.28%) ⬆️
lsf-integration 51.92% <ø> (-0.02%) ⬇️
lsf-unit 8.78% <ø> (ø)
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.

@nass600
Copy link
Contributor

nass600 commented Sep 3, 2025

@ricardoantoniocm worth a look at whitelabel css as well as there might be a chance they were broken there as well. cc: @yyassi-heartex

@robot-ci-heartex robot-ci-heartex merged commit 528843c into develop Sep 3, 2025
45 of 46 checks passed
@robot-ci-heartex robot-ci-heartex deleted the fb-fit-139/dropdown-color-taxonomy branch September 3, 2025 18:18
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.

5 participants