[PR #151] [MERGED] feat: add icons to RadzenFormField Start slot across UI components #143

New issue

Closed

opened 2026-02-26 17:37:30 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 17:37:30 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Corsinvest/cv4pve-admin/pull/151
Author: @franklupo
Created: 2/17/2026
Status: ✅ Merged
Merged: 2/17/2026
Merged by: @franklupo

Base: main ← Head: feat/ui-form-icons

📝 Commits (2)

bfd1375 feat: add icons to RadzenFormField Start slot across UI components
28b2082 feat: add icons to ClusterSettingsDialog, NodesSettings and LoginLayout form fields

📊 Changes

39 files changed (+436 additions, -230 deletions)

View changed files

📝 src/Corsinvest.ProxmoxVE.Admin.Core/Components/ProxmoxVE/Cluster/ResourceUsageGauge.razor (+5 -6)
📝 src/Corsinvest.ProxmoxVE.Admin.Core/Components/ProxmoxVE/Cluster/ResourceUsageGaugeStacked.razor (+9 -39)
📝 src/Corsinvest.ProxmoxVE.Admin.Core/Components/ProxmoxVE/Nodes/Charts.razor (+6 -0)
📝 src/Corsinvest.ProxmoxVE.Admin.Core/Components/ProxmoxVE/Storage/Charts.razor (+6 -0)
📝 src/Corsinvest.ProxmoxVE.Admin.Core/Components/ProxmoxVE/Vm/Backup/EditDialog.razor (+12 -2)
📝 src/Corsinvest.ProxmoxVE.Admin.Core/Components/ProxmoxVE/Vm/Charts.razor (+6 -0)
📝 src/Corsinvest.ProxmoxVE.Admin.Core/Components/ProxmoxVE/Vm/Snapshot/EditDialog.razor (+11 -1)
📝 src/Corsinvest.ProxmoxVE.Admin.Core/Components/SendMessage/SendMessageDialog.razor (+12 -1)
📝 src/Corsinvest.ProxmoxVE.Admin.Core/Components/Settings/NotifierSettingsDialog.razor (+8 -1)
📝 src/Corsinvest.ProxmoxVE.Admin.Core/Components/SmtpEmailConfigEdit.razor (+15 -0)
📝 src/Corsinvest.ProxmoxVE.Admin.Core/Components/SwitchField.razor (+9 -1)
📝 src/Corsinvest.ProxmoxVE.Admin.Core/Components/WebHookEditor.razor (+2 -0)
📝 src/Corsinvest.ProxmoxVE.Admin.Core/Components/Widgets/WidgetThumbDetailsBase.razor.cs (+2 -2)
📝 src/Corsinvest.ProxmoxVE.Admin.Core/Extensions/RadzenUIExtensions.cs (+1 -1)
📝 src/Corsinvest.ProxmoxVE.Admin.Module.AIServer/Components/RenderSettings.razor (+5 -1)
📝 src/Corsinvest.ProxmoxVE.Admin.Module.AutoSnap/Components/JobDialog.razor (+29 -4)
📝 src/Corsinvest.ProxmoxVE.Admin.Module.AutoSnap/Components/RenderSettings.razor (+11 -2)
📝 src/Corsinvest.ProxmoxVE.Admin.Module.BackupAnalytics/Components/RenderSettings.razor (+5 -1)
📝 src/Corsinvest.ProxmoxVE.Admin.Module.BackupAnalytics/Components/Trends.razor (+57 -37)
📝 src/Corsinvest.ProxmoxVE.Admin.Module.Bots/Components/RenderSettings.razor (+10 -2)

...and 19 more files

📄 Description

Summary

Add semantic Material icons to all RadzenFormField components using the <Start> slot
Wrap existing field content in <ChildContent> where needed
Update ResourceUsageGaugeStacked to use Radzen series colors (--rz-series-1..8)
Add IconOn/IconOff to SwitchField components (e.g. play/pause, lock/unlock)

Test plan

Verify icons appear correctly in all modified form fields
Verify gauge stacked colors cycle through series colors

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/Corsinvest/cv4pve-admin/pull/151 **Author:** [@franklupo](https://github.com/franklupo) **Created:** 2/17/2026 **Status:** ✅ Merged **Merged:** 2/17/2026 **Merged by:** [@franklupo](https://github.com/franklupo) **Base:** `main` ← **Head:** `feat/ui-form-icons` --- ### 📝 Commits (2) - [`bfd1375`](https://github.com/Corsinvest/cv4pve-admin/commit/bfd1375e36f4487ec89f51bbb98f52f5460add91) feat: add icons to RadzenFormField Start slot across UI components - [`28b2082`](https://github.com/Corsinvest/cv4pve-admin/commit/28b2082ba1287da42eb68ab3fdf4fe80427c2d36) feat: add icons to ClusterSettingsDialog, NodesSettings and LoginLayout form fields ### 📊 Changes **39 files changed** (+436 additions, -230 deletions) <details> <summary>View changed files</summary> 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/Components/ProxmoxVE/Cluster/ResourceUsageGauge.razor` (+5 -6) 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/Components/ProxmoxVE/Cluster/ResourceUsageGaugeStacked.razor` (+9 -39) 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/Components/ProxmoxVE/Nodes/Charts.razor` (+6 -0) 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/Components/ProxmoxVE/Storage/Charts.razor` (+6 -0) 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/Components/ProxmoxVE/Vm/Backup/EditDialog.razor` (+12 -2) 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/Components/ProxmoxVE/Vm/Charts.razor` (+6 -0) 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/Components/ProxmoxVE/Vm/Snapshot/EditDialog.razor` (+11 -1) 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/Components/SendMessage/SendMessageDialog.razor` (+12 -1) 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/Components/Settings/NotifierSettingsDialog.razor` (+8 -1) 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/Components/SmtpEmailConfigEdit.razor` (+15 -0) 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/Components/SwitchField.razor` (+9 -1) 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/Components/WebHookEditor.razor` (+2 -0) 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/Components/Widgets/WidgetThumbDetailsBase.razor.cs` (+2 -2) 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/Extensions/RadzenUIExtensions.cs` (+1 -1) 📝 `src/Corsinvest.ProxmoxVE.Admin.Module.AIServer/Components/RenderSettings.razor` (+5 -1) 📝 `src/Corsinvest.ProxmoxVE.Admin.Module.AutoSnap/Components/JobDialog.razor` (+29 -4) 📝 `src/Corsinvest.ProxmoxVE.Admin.Module.AutoSnap/Components/RenderSettings.razor` (+11 -2) 📝 `src/Corsinvest.ProxmoxVE.Admin.Module.BackupAnalytics/Components/RenderSettings.razor` (+5 -1) 📝 `src/Corsinvest.ProxmoxVE.Admin.Module.BackupAnalytics/Components/Trends.razor` (+57 -37) 📝 `src/Corsinvest.ProxmoxVE.Admin.Module.Bots/Components/RenderSettings.razor` (+10 -2) _...and 19 more files_ </details> ### 📄 Description ## Summary - Add semantic Material icons to all `RadzenFormField` components using the `<Start>` slot - Wrap existing field content in `<ChildContent>` where needed - Update `ResourceUsageGaugeStacked` to use Radzen series colors (`--rz-series-1..8`) - Add `IconOn`/`IconOff` to `SwitchField` components (e.g. play/pause, lock/unlock) ## Test plan - [ ] Verify icons appear correctly in all modified form fields - [ ] Verify gauge stacked colors cycle through series colors --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>