Skip to content

Vinay K Tooltip Hover Preview#4753

Open
Vinay944924 wants to merge 4 commits intodevelopmentfrom
vinay_k_tools_section
Open

Vinay K Tooltip Hover Preview#4753
Vinay944924 wants to merge 4 commits intodevelopmentfrom
vinay_k_tools_section

Conversation

@Vinay944924
Copy link
Copy Markdown
Contributor

Description

Screenshot 2026-01-22 at 10 35 03 PM

Related PRS (if any):

This frontend PR is related to the development backend PR.

Main changes explained:

  • Updated the Tools Horizontal Bar Chart JSX and even changed the CSS file.

How to test:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. Go to Dashboard → Reports → Total Construction Summary → Tools and Equipment
  6. Verify that the Tools and Equipments show the Chart Preview as shown in the Screenshot.

Screenshots or videos of changes:

Screenshot 2026-01-22 at 10 10 28 PM

Note:

Include the information the reviewers need to know.

@netlify
Copy link
Copy Markdown

netlify bot commented Jan 23, 2026

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 0c21aee
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/69e6979a1094f2000834441a
😎 Deploy Preview https://deploy-preview-4753--highestgoodnetwork-dev.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.

@sonarqubecloud
Copy link
Copy Markdown

@one-community one-community added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Apr 9, 2026
@Nirali164
Copy link
Copy Markdown

Hello Vinay,

I have checked the feature, but the UI looks off on my screen. the Tool tip is working
Screenshot 2026-04-09 at 3 24 55 PM
Screenshot 2026-04-09 at 3 25 06 PM

Copy link
Copy Markdown

@HemanthNidamanuru HemanthNidamanuru left a comment

Choose a reason for hiding this comment

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

Hi Vinay,

I tested this PR locally. The placeholder text "Hover to preview, apply filters to load data" is appearing as static text on the page rather than as a tooltip triggered on hover. Also, after the chart loads, the tooltip is misaligned and overlapping the chart area, and the filter reset button is not working.

Image Image

Copy link
Copy Markdown

@Anusha-Gali Anusha-Gali left a comment

Choose a reason for hiding this comment

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

Hi Vinay,

I have reviewed your PR locally, however the functionality does not seem to work. There is no interactive Hover tooltip. Also in case the chart tooltip also comes under this PR, the UI is not that great.

Image Image Image Image

@Vinay944924 Vinay944924 changed the title Tooltip_Hover_Preview Vinay K Tooltip Hover Preview Apr 17, 2026
@sonarqubecloud
Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants