8000 Fix progress bar width calculation in FounderProjectItem component by miladsoft · Pull Request #425 · block-core/angor · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

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

Closed
wants to merge 7 commits into from

Conversation

miladsoft
Copy link
Member

No description provided.

SuperJMN and others added 7 commits June 11, 2025 16:33
* 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
@Copilot Copilot AI review requested due to automatic review settings June 12, 2025 10:21
Copy link
@Copilot Copilot AI left a 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

Comment on lines +88 to 92
<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>
Copy link
Preview
Copilot AI Jun 12, 2025

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.

@dangershony
Copy link
Member

@miladsoft you will have to cherry pick this pr to current master sorry

@miladsoft miladsoft closed this Jun 15, 2025
@miladsoft miladsoft deleted the fix/html-bug branch June 15, 2025 16:42
@miladsoft miladsoft restored the fix/html-bug branch June 15, 2025 16:42
@miladsoft miladsoft deleted the fix/html-bug branch June 15, 2025 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0