-
Notifications
You must be signed in to change notification settings - Fork 33
Fix progress bar width calculation in FounderProjectItem component #425
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
* First brainstorming session= * Refactor encryption workflow and add Nip44Encryption support * Renamed to nip4 * Improve readability of `PostInvestmentRequestApproval2` and rename `G… * Replaced the sql style of writing * Moved the sign service implementation to a new interface * Align `IInvestmentAppService` method signatures with updated `Investm… * Update `EventSendResponse` usage, streamline service registrations, a… * Remove unused `TestSignService` class * Update primary URL in `NetworkConfiguration` and downgrade `System.Te… * Bump `System.Text.Json` to version `9.0.5` in project dependencies. * Update `NetworkStorage` namespace and streamline DI container registr…
* add a new test indexer * remove an indexer
There was a problem hiding this comment.
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 corrects the calculation of the progress bar width in the FounderProjectItem component.
- Adjusts the width style calculation in the progress bar element
- Introduces a string concatenation approach to apply the percentage value
<div class="progress rounded-pill mb-3" style="height: 16px; background-color: var(--btn-border);"> | ||
<div class="progress-bar-striped bg-success rounded-pill text-center" role="progressbar" | ||
style="width: @(projectStats.FundingProgressPercent + "%");"> | ||
</div> | ||
</div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider whether the additional nested progress container is necessary. If the extra wrapper is redundant, removing it could simplify the markup.
Copilot uses AI. Check for mistakes.
@miladsoft you will have to cherry pick this pr to current master sorry |
No description provided.