Improve Portfolio section (Avalonia) #447
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Introduced new fields: `FundedProjects`, `TotalInvested`, `ProjectsInRecovery`, and `RecoveredToPenalty` in `IPortfolioSectionViewModel` and implemented them in `PortfolioSectionViewModel` and `PortfolioSectionViewModelDesign`. - Added new SVG assets (`btc.svg` and `recovery.svg`) for visual elements in the portfolio view. - Enhanced `PortfolioSectionView.axaml` to include investment summary cards and recovery details with updated bindings and icons. - Updated design-time data to reflect new investment and recovery metrics for improved development UX.
- Created a new `Badge` control with `Color` property and default styling in `Badge.axaml` and `Badge.axaml.cs`. - Added `Badge.axaml` to the application styles in `App.axaml`. - Updated `Light.axaml` to remove unused namespaces.
- Added `Invested` field to `IPortfolioProject` and `PortfolioProject` for tracking invested amount. - Enhanced `PortfolioProjectView` with structured investment and recovery sections using `CardGrid` and `HeaderedContainer`. - Replaced old inline styles with theme-based styles for better consistency and maintainability. - Updated `PortfolioProjectDesign` with default test data for `Target`, `Raised`, `InRecovery`, and `Invested` fields. - Modified `CardGrid` control to support themed item containers.
…lated references - Adjusted `PortfolioProjectView.axaml` to use `OfferAcceptedView` for improved context clarity. - Changed design width of `PortfolioProjectView` from 900 to 600 for better alignment with UI requirements. - Simplified card structure by removing redundant `Target`, `Raised`, and `Founder Status` sections in investment details. - Updated investment status badge to display "Pending" with an orange color for accurate representation.
…logic and improve UI responsiveness - Transitioned from `IsInvestmentCompleted` boolean to `InvestmentStatus` enum in related models, view models, and data transfer objects for more granular investment tracking. - Updated all relevant bindings and UI components to reflect the new `InvestmentStatus`, including investment status badge and related behaviors. - Introduced a new control, `MinMaxUniformGrid`, for dynamic column sizing in grids, integrated it into `CardGrid` for better layout adaptability. - Renamed and revamped `NotCompletedProjectView` to `OfferAcceptedView` to align with the updated investment flow. - Adjusted package dependencies with minor version updates for `Zafiro.Avalonia` and `Zafiro.Avalonia.Dialogs`. - Improved error messaging in `SensitiveWalletDataProvider` for clearer feedback during decryption failures.
…mpty state in FounderSectionView - Added default values for MinWidth, MinHeight, MaxWidth, and MaxHeight in Light theme for non-Desktop platforms. - Changed `RequestedThemeVariant` in `App.axaml` from "Dark" to "Default" to follow system theme preference. - Enhanced `FounderSectionView` with empty state messaging for projects list when no items are available. - Commented out unused SlimWizardControl style in Light theme for cleanup.
- Changed `RequestedThemeVariant` in `App.axaml` from "Default" to "Dark". - Updated `Badge` control styles to use more refined colors (`DarkOrange`, `Goldenrod`, `LimeGreen`) for better visual distinction of investment statuses. - Improved `Badge` template structure for enhanced layering and styling flexibility. - Cleaned up redundant spacing and unused properties in theme resources.
| @@ -12,7 +13,7 @@ public class InvestedProjectDto | |||
| public Amount Raised { get; set; } | |||
| public string Description { get; set; } | |||
| public Amount InRecovery { get; set; } | |||
| public bool IsInvesmentCompleted { get; set; } | |||
| public InvestmentStatus InvestmentStatus { get; set; } | |||
There was a problem hiding this comment.
Do we still need the Founder status in this case?
There was a problem hiding this comment.
I don't know if it's relevant for the investor. Shall we remove it from the invested projects?
DavidGershony
approved these changes
Jun 27, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.