Skip to content

Conversation

Swapnil-Dhamal
Copy link
Contributor

@Swapnil-Dhamal Swapnil-Dhamal commented Feb 19, 2025

Closes: #692

Description
This PR introduces a summary card that displays the breakdown of rent calculations, including base rent, power bill, water bill, and other fees. Additionally, a Reset button has been implemented, which:

Appears after the total rent is calculated.
Clears all input fields and hides the summary card when clicked.

Key Features:
✅ Displays a detailed rent summary after calculation.

rent_calculate.1.mp4

✅ Formats amounts in currency (USD) using Intl.NumberFormat for better readability.
✅ Adds a Reset button to clear inputs and hide the summary.
✅ Ensures a smooth user experience by resetting the state properly.

Testing & Verification
Enter values in the form fields and click Calculate Total Rent → The summary card appears with formatted values.
Click Reset → Inputs clear and the summary card disappears.

Copy link

netlify bot commented Feb 19, 2025

Deploy Preview for visionary-fenglisu-db1b1d ready!

Name Link
🔨 Latest commit aa27270
🔍 Latest deploy log https://app.netlify.com/sites/visionary-fenglisu-db1b1d/deploys/67b5d4e14c909c00085d86e0
😎 Deploy Preview https://deploy-preview-693--visionary-fenglisu-db1b1d.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 site configuration.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Great job, @Swapnil-Dhamal! 🎉 Thank you for submitting your pull request for Rentalog. Our dedicated team will review it diligently.Your contribution is valuable and we appreciate your efforts to improve our project.

@harmeetsingh11
Copy link
Contributor

harmeetsingh11 commented Feb 21, 2025

@Swapnil-Dhamal good work. Next time, please don't forget to mention issue number in the PR.

@harmeetsingh11 harmeetsingh11 merged commit 7604cee into gauravsingh1281:main Feb 21, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Display Calculated Total with Formatting
3 participants