SVI/ProxmoxVE-Local
1
0
Fork 0
mirror of https://github.com/community-scripts/ProxmoxVE-Local.git synced 2026-03-31 06:23:54 -04:00
Code Issues 56 Packages Projects Releases 7 Wiki Activity

[PR #34] [MERGED] remove install method if pve-tool / addon & Improve dark mode initialization and modal UI #32 #124

New Issue
Closed
opened 2025-11-20 04:12:57 -05:00 by saavagebueno · 0 comments
saavagebueno commented 2025-11-20 04:12:57 -05:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/community-scripts/ProxmoxVE-Local/pull/34
Author: @MickLesk
Created: 10/6/2025
Status: ✅ Merged
Merged: 10/6/2025
Merged by: @MickLesk

Base: main ← Head: development

📝 Commits (2)

  • d59fd16 Add dark mode support across UI
  • 173aaa8 Improve dark mode initialization and modal UI (#32)

📊 Changes

13 files changed (+678 additions, -296 deletions)

View changed files

➕ src/app/_components/DarkModeProvider.tsx (+83 -0)
➕ src/app/_components/DarkModeToggle.tsx (+66 -0)
📝 src/app/_components/InstalledScriptsTab.tsx (+28 -31)
📝 src/app/_components/ResyncButton.tsx (+5 -5)
📝 src/app/_components/ScriptCard.tsx (+11 -11)
📝 src/app/_components/ScriptDetailModal.tsx (+404 -201)
📝 src/app/_components/ScriptsGrid.tsx (+3 -3)
📝 src/app/_components/ServerForm.tsx (+18 -18)
📝 src/app/_components/SettingsButton.tsx (+1 -1)
📝 src/app/_components/SettingsModal.tsx (+16 -16)
📝 src/app/layout.tsx (+33 -2)
📝 src/app/page.tsx (+8 -8)
📝 src/styles/globals.css (+2 -0)

📄 Description

✍️ Description

🔗 Related PR / Issue

Link: #

✅ Prerequisites (X in brackets)

  • Self-review completed – Code follows project standards.
  • Tested thoroughly – Changes work as expected.
  • No security risks – No hardcoded secrets, unnecessary privilege escalations, or permission issues.

Screenshot for frontend Change

🛠️ Type of Change (X in brackets)

  • 🐞 Bug fix – Resolves an issue without breaking functionality.
  • ✨ New feature – Adds new, non-breaking functionality.
  • 💥 Breaking change – Alters existing functionality in a way that may require updates.

🔄 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/community-scripts/ProxmoxVE-Local/pull/34 **Author:** [@MickLesk](https://github.com/MickLesk) **Created:** 10/6/2025 **Status:** ✅ Merged **Merged:** 10/6/2025 **Merged by:** [@MickLesk](https://github.com/MickLesk) **Base:** `main` ← **Head:** `development` --- ### 📝 Commits (2) - [`d59fd16`](https://github.com/community-scripts/ProxmoxVE-Local/commit/d59fd168894c92e4e2505be5557a118eb574a987) Add dark mode support across UI - [`173aaa8`](https://github.com/community-scripts/ProxmoxVE-Local/commit/173aaa81dcf7210cc760b154d56c9b8fcb5068ce) Improve dark mode initialization and modal UI (#32) ### 📊 Changes **13 files changed** (+678 additions, -296 deletions) <details> <summary>View changed files</summary> ➕ `src/app/_components/DarkModeProvider.tsx` (+83 -0) ➕ `src/app/_components/DarkModeToggle.tsx` (+66 -0) 📝 `src/app/_components/InstalledScriptsTab.tsx` (+28 -31) 📝 `src/app/_components/ResyncButton.tsx` (+5 -5) 📝 `src/app/_components/ScriptCard.tsx` (+11 -11) 📝 `src/app/_components/ScriptDetailModal.tsx` (+404 -201) 📝 `src/app/_components/ScriptsGrid.tsx` (+3 -3) 📝 `src/app/_components/ServerForm.tsx` (+18 -18) 📝 `src/app/_components/SettingsButton.tsx` (+1 -1) 📝 `src/app/_components/SettingsModal.tsx` (+16 -16) 📝 `src/app/layout.tsx` (+33 -2) 📝 `src/app/page.tsx` (+8 -8) 📝 `src/styles/globals.css` (+2 -0) </details> ### 📄 Description <!--🛑 All Pull Requests need to made against the development branch. PRs against main will get closed. --> ## ✍️ Description ## 🔗 Related PR / Issue Link: # ## ✅ Prerequisites (**X** in brackets) - [x] **Self-review completed** – Code follows project standards. - [x] **Tested thoroughly** – Changes work as expected. - [x] **No security risks** – No hardcoded secrets, unnecessary privilege escalations, or permission issues. ## Screenshot for frontend Change --- ## 🛠️ Type of Change (**X** in brackets) - [ ] 🐞 **Bug fix** – Resolves an issue without breaking functionality. - [x] ✨ **New feature** – Adds new, non-breaking functionality. - [ ] 💥 **Breaking change** – Alters existing functionality in a way that may require updates. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
saavagebueno added the pull-request label 2025-11-20 04:12:57 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
enhancement
enhancement
feature
in progress
pull-request
Mirrored from GitHub Pull Request
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
saavagebueno
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SVI/ProxmoxVE-Local#124
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?