[PR #160] [MERGED] feat: improve Dashboard reset logic, add save icon and update build props #153

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

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

Base: main ← Head: feat/ui-improvements

📝 Commits (1)

9c8b4ab feat: improve Dashboard reset logic, add save icon and update build props

📊 Changes

3 files changed (+8 additions, -3 deletions)

View changed files

📝 Directory.Build.props (+0 -1)
📝 src/Corsinvest.ProxmoxVE.Admin.Module.Dashboard/Components/Dashboard.razor.cs (+3 -0)
📝 src/Corsinvest.ProxmoxVE.Admin.Module.System/Components/SettingsAccordion.razor (+5 -2)

📄 Description

Summary

Dashboard: reset SelectedScreenResolution to first item on cleanup
SettingsAccordion: add save icon to "Save changes" button
Directory.Build.props: remove obsolete TestingExpirationMonths property

Test plan

Verify dashboard screen resolution resets correctly
Verify save icon appears on settings accordion button

_{🔄 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/160 **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-improvements` --- ### 📝 Commits (1) - [`9c8b4ab`](https://github.com/Corsinvest/cv4pve-admin/commit/9c8b4ab749c6a1f96aa049187b7f1f0dad9b2fa6) feat: improve Dashboard reset logic, add save icon and update build props ### 📊 Changes **3 files changed** (+8 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `Directory.Build.props` (+0 -1) 📝 `src/Corsinvest.ProxmoxVE.Admin.Module.Dashboard/Components/Dashboard.razor.cs` (+3 -0) 📝 `src/Corsinvest.ProxmoxVE.Admin.Module.System/Components/SettingsAccordion.razor` (+5 -2) </details> ### 📄 Description ## Summary - Dashboard: reset SelectedScreenResolution to first item on cleanup - SettingsAccordion: add `save` icon to "Save changes" button - Directory.Build.props: remove obsolete TestingExpirationMonths property ## Test plan - [ ] Verify dashboard screen resolution resets correctly - [ ] Verify save icon appears on settings accordion button --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>