Skip to content

HoverToolbar: Requested Changes#464

Merged
csAyushDubey merged 1 commit intodevelop_v3from
VE-6600-Hover-Toolbar-Support
Jul 18, 2025
Merged

HoverToolbar: Requested Changes#464
csAyushDubey merged 1 commit intodevelop_v3from
VE-6600-Hover-Toolbar-Support

Conversation

@csAyushDubey
Copy link
Copy Markdown
Contributor

@csAyushDubey csAyushDubey commented Jul 18, 2025

  1. Changed order in tooltip
  2. Hide custom cursor while interacting with hover toolbar

SS

Screen.Recording.2025-07-18.at.2.04.38.PM.mov

@csAyushDubey csAyushDubey requested a review from a team as a code owner July 18, 2025 08:25
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!!

@csAyushDubey csAyushDubey merged commit 565d56d into develop_v3 Jul 18, 2025
9 of 10 checks passed
@csAyushDubey csAyushDubey deleted the VE-6600-Hover-Toolbar-Support branch July 18, 2025 08:41
@github-actions
Copy link
Copy Markdown

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 71.77% 9026 / 12575
🔵 Statements 71.77% 9026 / 12575
🔵 Functions 71.61% 323 / 451
🔵 Branches 84.21% 1131 / 1343
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/visualBuilder/components/Tooltip.tsx 6.03% 50% 33.33% 6.03% 29-170, 176-182
src/visualBuilder/listeners/mouseHover.ts 61.37% 50% 62.5% 61.37% 52-59, 76-77, 94-97, 127-137, 143-144, 151-154, 157-159, 165, 179-180, 189-201, 208-210, 212-253, 261-263, 265-267, 275-283, 286-290, 292, 294-296, 311-315, 337-340
Generated in workflow #471 for commit cc71759 by the Vitest Coverage Report Action

csAdityaPachauri pushed a commit that referenced this pull request Jul 24, 2025
csAdityaPachauri pushed a commit that referenced this pull request Jul 24, 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