Skip to content

Rewrite Nginx Proxy Manager theme for the current UI #734

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
GilbN merged 3 commits into from
Sep 14, 2026
Merged

Rewrite Nginx Proxy Manager theme for the current UI #734

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
c50c30e
Rewrite Nginx Proxy Manager theme for the React UI
GilbN Sep 14, 2026
7a76426
Add NPM development setup and theme variable checks
GilbN Sep 14, 2026
e59bd53
Require PR screenshots and add NPM before and after evidence
GilbN Sep 14, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion .github/CONTRIBUTING.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,9 @@

## Bug fixes

- When submitting bugfixes please show a before and after screenshot of the fix, and a description of what the fix does.
- PR descriptions for visual changes must include before/after screenshots and a short description of the fix. This applies to bug fixes, theme rewrites, and documentation gallery updates.
- Use comparable app states, theme options, and viewport sizes. Embed the images in the description; local-only screenshots do not meet this requirement.
- Changes with no visible result, such as workflow instructions, do not need screenshots.

## New theme option

Expand Down
5 changes: 4 additions & 1 deletion AGENTS.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,8 @@ affected app and inspect the result in a browser.

## Maintainer instructions

- Use `admin` / `admin` for disposable development app logins. If an app enforces an email address or a longer password, use `admin@example.com` and `adminadmin` as needed, and document the exception and login beside its startup URL.

- Do not push, publish, deploy, or trigger deployment workflows without an
explicit instruction from the maintainer. Describing their usual release
workflow does not authorize a push. Keep trial-run work local and reviewable.
Expand All @@ -27,7 +29,8 @@ affected app and inspect the result in a browser.
untested ideas. Keep run-specific evidence under ignored `dev/artifacts/`.
- Always apply the `unslop` skill to writing. Keep PR titles and descriptions
short, stating the user-visible fixes and relevant validation. Before/after
screenshot attachments are optional for bug-fix PRs, per the maintainer.
screenshots are required in PR descriptions for visual changes, including
bug fixes and theme rewrites. Show the affected UI in comparable states.
The skill source is
[cursor/plugins unslop](https://raw.githubusercontent.com/cursor/plugins/refs/heads/main/pstack/skills/unslop/SKILL.md).

Expand Down
Loading