Skip to content

Fix: Issue with hover toolbar click-ability#467

Merged
csAyushDubey merged 1 commit intodevelop_v3from
VE-7005-hover-toolbar-clickability
Jul 22, 2025
Merged

Fix: Issue with hover toolbar click-ability#467
csAyushDubey merged 1 commit intodevelop_v3from
VE-7005-hover-toolbar-clickability

Conversation

@csAyushDubey
Copy link
Copy Markdown
Contributor

This is to fix the issue where user was not able to click on field parents through the hover toolbar.

Problem:

Screen.Recording.2025-07-21.at.8.20.20.PM.mov

Solved:

Screen.Recording.2025-07-22.at.11.54.06.AM.mov

@csAyushDubey csAyushDubey requested a review from a team as a code owner July 22, 2025 06:28
Copy link
Copy Markdown
Contributor

@hiteshshetty-dev hiteshshetty-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!!

@github-actions
Copy link
Copy Markdown

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 71.76% 9028 / 12580
🔵 Statements 71.76% 9028 / 12580
🔵 Functions 71.52% 324 / 453
🔵 Branches 84.21% 1131 / 1343
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/visualBuilder/listeners/mouseHover.ts 60.8% 50% 58.82% 60.8% 52-59, 76-77, 94-97, 127-137, 143-144, 151-154, 157-159, 165, 179-180, 189-205, 212-214, 216-257, 265-267, 269-271, 279-287, 290-294, 296, 298-300, 315-319, 341-344
Generated in workflow #478 for commit ac4fdb3 by the Vitest Coverage Report Action

@csAyushDubey csAyushDubey merged commit 3b897a4 into develop_v3 Jul 22, 2025
14 of 16 checks passed
@csAyushDubey csAyushDubey deleted the VE-7005-hover-toolbar-clickability branch July 22, 2025 07:40
csAdityaPachauri pushed a commit that referenced this pull request Jul 24, 2025
…ability

Fix: Issue with hover toolbar click-ability
csAyushDubey added a commit that referenced this pull request Aug 6, 2025
…ability

Fix: Issue with hover toolbar click-ability
@faraazb faraazb mentioned this pull request Aug 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants