Skip to content
This repository was archived by the owner on Jul 6, 2026. It is now read-only.

i button is now clickable#2546

Merged
anth-volk merged 2 commits into
PolicyEngine:masterfrom
SEEDART007:explain-ai-clickable
Jun 19, 2025
Merged

i button is now clickable#2546
anth-volk merged 2 commits into
PolicyEngine:masterfrom
SEEDART007:explain-ai-clickable

Conversation

@SEEDART007

Copy link
Copy Markdown

Fixes #2528

Screen.Recording.2025-05-19.162519.mp4

Copilot AI review requested due to automatic review settings May 19, 2025 10:59
@vercel

vercel Bot commented May 19, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
policyengine-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 6, 2025 5:56am

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes issue #2528 by making the info icon clickable.

  • Added an onClick event handler to the InfoCircleOutlined component.

Comment thread src/pages/household/output/NetIncomeBreakdown.jsx
@anth-volk

Copy link
Copy Markdown
Collaborator

@SEEDART007 Thanks for this! If you could just run make format and push the changes, we can get this in.

@anth-volk anth-volk self-requested a review June 5, 2025 23:12
@SEEDART007

Copy link
Copy Markdown
Author

@anth-volk done

@anth-volk anth-volk merged commit 5c6f427 into PolicyEngine:master Jun 19, 2025
4 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in policyengine-app Jun 19, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants