Skip to content

Conversation

@Gondragos
Copy link
Contributor

Tooltip did not work with disabled buttons as they did not emit any events. This is a workaround that listens to the parent element to catch hover state changes by mouseover / mouseleave events on it.

It provides a solution that will not change element layout, so it will keep the visual part in the expected way.

Before:

image

@netlify
Copy link

netlify bot commented Jul 26, 2025

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

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

@netlify
Copy link

netlify bot commented Jul 26, 2025

Deploy Preview for label-studio-storybook ready!

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

@netlify
Copy link

netlify bot commented Jul 26, 2025

Deploy Preview for heartex-docs canceled.

Name Link
🔨 Latest commit 2f096a3
🔍 Latest deploy log https://app.netlify.com/projects/heartex-docs/deploys/68908609d920d300083bcff4

@github-actions github-actions bot added the fix label Jul 26, 2025
@netlify
Copy link

netlify bot commented Jul 26, 2025

Deploy Preview for label-studio-playground ready!

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

@codecov
Copy link

codecov bot commented Jul 26, 2025

Codecov Report

❌ Patch coverage is 81.81818% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 66.16%. Comparing base (4e5e32a) to head (2f096a3).
⚠️ Report is 1 commits behind head on develop.

Files with missing lines Patch % Lines
web/libs/ui/src/lib/Tooltip/Tooltip.tsx 78.94% 4 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #8049      +/-   ##
===========================================
- Coverage    70.49%   66.16%   -4.33%     
===========================================
  Files          719      505     -214     
  Lines        51356    33841   -17515     
  Branches      8729     8733       +4     
===========================================
- Hits         36205    22392   -13813     
+ Misses       15148    11446    -3702     
  Partials         3        3              
Flag Coverage Δ
lsf-e2e 59.76% <81.81%> (+0.01%) ⬆️
lsf-integration 55.42% <81.81%> (+0.05%) ⬆️
lsf-unit 9.52% <ø> (ø)
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.

@Gondragos
Copy link
Contributor Author

Gondragos commented Jul 28, 2025

/git merge

Workflow run
Successfully merged: 78 files changed, 4076 insertions(+), 168 deletions(-)

@Gondragos Gondragos force-pushed the fb-FIT-418/tooltip-with-disabled-button branch from fab8998 to d259f41 Compare August 1, 2025 09:57
# Conflicts:
#	web/libs/core/src/lib/utils/feature-flags/flags.ts
@Gondragos Gondragos force-pushed the fb-FIT-418/tooltip-with-disabled-button branch from d259f41 to 5a28638 Compare August 1, 2025 10:00
@niklub niklub enabled auto-merge (squash) August 1, 2025 10:01
@niklub
Copy link
Collaborator

niklub commented Aug 4, 2025

/git merge

Workflow run
Successfully merged: create mode 100644 web/libs/editor/tests/e2e/tests/paragraphs-enhanced.test.js

@niklub niklub merged commit 60a1bd3 into develop Aug 4, 2025
62 of 66 checks passed
@robot-ci-heartex robot-ci-heartex deleted the fb-FIT-418/tooltip-with-disabled-button branch August 4, 2025 11:11
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.

3 participants