Skip to content

Modernize frontend: update WPF packages, Angular dependencies, and sy…#1202

Open
Symlink420 wants to merge 1 commit intoopenbullet:masterfrom
Symlink420:fix/frontend-modernization
Open

Modernize frontend: update WPF packages, Angular dependencies, and sy…#1202
Symlink420 wants to merge 1 commit intoopenbullet:masterfrom
Symlink420:fix/frontend-modernization

Conversation

@Symlink420
Copy link

Description
This PR modernizes the frontend components for both the WPF (Native) client and the Angular Web UI. It addresses technical debt, improves UI library performance, and resolves several dependency vulnerabilities.

Key Changes

  1. WPF (Native) Modernization
    MahApps.Metro.IconPacks: Upgraded from 4.8.0 to 5.1.0.
    Markdig: Upgraded from 0.37.0 to 0.40.0 for improved Markdown rendering.
    HtmlSanitizer: Upgraded to 9.0.835 (stable) to fix security vulnerabilities and modernization overhead.
    Standardized Extensions: Synced Microsoft.Extensions.* and EF Core libraries to 8.0.12/8.0.2 across all projects to ensure a consistent dependency tree.

  2. Angular Client (Web UI) Modernization
    Angular Framework: Re-platformed from 17.3.5 to a stable 17.3.10 baseline.
    PrimeNG: Upgraded from 17.14.1 to 17.18.0.
    Angular CDK: Integrated @angular/cdk (17.3.10) to provide low-level utilities required by modern PrimeNG components.
    CSS Frameworks: Updated Bootstrap to 4.6.2 and Moment.js to 2.30.1.
    Dependency Resolution: Used --legacy-peer-deps to stabilize the upgrade path for legacy components.
    Verification Results
    C# / .NET Build
    Status: SUCCESS ✅
    Verified that all projects (Native, Web, Core, RuriLib) compile without errors after the package upgrades.
    Standardized Microsoft.Extensions.DependencyInjection to 8.0.1 to resolve restore conflicts.
    Angular / Web Build
    Command: npm run build
    Status: SUCCESS ✅
    Verified that the production bundle generates correctly, resolving the previous DragDropModule symbol error by adding the missing CDK dependency.

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.

1 participant