Skip to content

161 - Finished Displaying Regen as a Percentage of Total Capacity#163

Open
devmchechi wants to merge 11 commits intodevelopfrom
161-task--display-regen-as-a-percentage-of-total-capacity
Open

161 - Finished Displaying Regen as a Percentage of Total Capacity#163
devmchechi wants to merge 11 commits intodevelopfrom
161-task--display-regen-as-a-percentage-of-total-capacity

Conversation

@devmchechi
Copy link
Contributor

@devmchechi devmchechi commented Nov 8, 2025

Changes

In the EfficiencyScreen and SpeedScreen, I added booleans to go back and forth between amps and percentage. I added variables and functions to initialize the maxRegenCapacity and calculate the percentage of regen.

Screenshots

Performance Screen:
image
Efficiency Screen:
image

Checklist

It can be helpful to check the Checks and Files changed tabs.
Please review the contributor guide and reach out to your Tech Lead if anything is unclear.
Please request reviewers and ping on slack only after you've gone through this whole checklist.

  • All commits are tagged with the ticket number
  • No linting errors / newline at end of file warnings
  • All code follows repository-configured prettier formatting
  • No merge conflicts
  • All checks passing
  • Screenshots of UI changes (see Screenshots section)
  • Remove any non-applicable sections of this template
  • Assign the PR to yourself
  • Request reviewers & ping on Slack
  • PR is linked to the ticket (fill in the closes line below)

Closes 161

@devmchechi devmchechi requested a review from bracyw November 8, 2025 00:06
@devmchechi devmchechi self-assigned this Nov 8, 2025
@devmchechi devmchechi added the medium A more complex task label Nov 8, 2025
@devmchechi devmchechi marked this pull request as ready for review November 9, 2025 22:59
bracyw
bracyw previously requested changes Nov 17, 2025
Copy link
Contributor

@bracyw bracyw left a comment

Choose a reason for hiding this comment

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

Two questions, otherwise looks good.

Copy link
Contributor Author

@devmchechi devmchechi left a comment

Choose a reason for hiding this comment

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

This needed to be changed because you wanted to make the percentage on the Efficiency screen look like that Torque Percentage. Before the percentage was on the below the number, and smaller. I changed it so the percentage UI looked like the Torque percentage.

@devmchechi devmchechi dismissed bracyw’s stale review November 19, 2025 01:39

This needed to be changed because you wanted to make the percentage on the Efficiency screen look like that Torque Percentage. Before the percentage was on the below the number, and smaller. I changed it so the percentage UI looked like the Torque percentage.

@devmchechi devmchechi requested a review from bracyw November 27, 2025 04:19
@devmchechi devmchechi marked this pull request as draft January 18, 2026 20:56
@devmchechi devmchechi marked this pull request as ready for review January 18, 2026 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

medium A more complex task

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants