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

New issue

Closed

opened 2026-02-26 12:40:26 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 12:40:26 +03:00

Owner

📋 Pull Request Information

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

Base: development ← Head: Remove-Install-Method-in-Dialogue-if-Addon-PVE-Tool

📝 Commits (1)

33e0aea Improve dark mode initialization and modal UI

📊 Changes

3 files changed (+437 additions, -204 deletions)

View changed files

📝 src/app/_components/DarkModeProvider.tsx (+11 -3)
📝 src/app/_components/ScriptDetailModal.tsx (+404 -201)
📝 src/app/layout.tsx (+22 -0)

📄 Description

✍️ Description

Remove "Install Method" if pve-tool or addon

Link: #31

✅ 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

LXC:

Tool:

🛠️ 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/32 **Author:** [@MickLesk](https://github.com/MickLesk) **Created:** 10/6/2025 **Status:** ✅ Merged **Merged:** 10/6/2025 **Merged by:** [@michelroegl-brunner](https://github.com/michelroegl-brunner) **Base:** `development` ← **Head:** `Remove-Install-Method-in-Dialogue-if-Addon-PVE-Tool` --- ### 📝 Commits (1) - [`33e0aea`](https://github.com/community-scripts/ProxmoxVE-Local/commit/33e0aeafbf0cc9ef7ead2a76b9ac21bfeb741329) Improve dark mode initialization and modal UI ### 📊 Changes **3 files changed** (+437 additions, -204 deletions) <details> <summary>View changed files</summary> 📝 `src/app/_components/DarkModeProvider.tsx` (+11 -3) 📝 `src/app/_components/ScriptDetailModal.tsx` (+404 -201) 📝 `src/app/layout.tsx` (+22 -0) </details> ### 📄 Description  ## ✍️ Description Remove "Install Method" if pve-tool or addon ## 🔗 Related PR / Issue Link: #31 ## ✅ 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 LXC: <img width="872" height="712" alt="image" src="https://github.com/user-attachments/assets/1c2297b8-8569-4cf6-8470-dd1ef40dc374" /> Tool: <img width="872" height="416" alt="image" src="https://github.com/user-attachments/assets/bc13054f-290b-42c7-b125-dde507031faa" /> --- ## 🛠️ 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>