fix: system tools UI spacing, Forgejo icon, and form organization #5
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "fix/system-tools-ui-spacing"
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?
Summary
Three UI fixes for the system tools admin panel:
1. CSS: Setting Item & Form Spacing (admin.css)
The
setting-item,settings-list,setting-card, andsetting-rowclasses had no CSS definitions despite being used extensively across all system tools tabs. This caused forms to render with zero spacing between fields, making them dense and hard to read.Added styles for:
.settings-list— vertical flex container with 16px gap between items.setting-item— horizontal row layout with glass background, border, padding, and hover state.setting-info— label area with proper title/description spacing.setting-card/.setting-card-inner/.setting-card-label— Settings tab card pattern.setting-row— side-by-side layout for paired settings.settings-form— form-level spacing for labels, hints, and groups.content-card— card wrapper with header/body styling (used in settings.php).setting-toggle-item— horizontal toggle with label and description.global-settings-content— flex column layout with gap.form-actions— proper action bar with border-top separator2. Forgejo Icon Fix (admin_system_tools.php)
Problem:
fas fa-git-altrenders as an X in a box in Font Awesome 6.5.1 becausefa-git-altis a brand icon that requires thefabprefix, notfas(solid). In FA6, the solid set does not include brand icons.Fix: Replaced
fa-git-altwithfa-code-branch(available in FA6 free solid set). This generic source-control icon is appropriate for the Forgejo update system and renders correctly.Changed 2 occurrences (System Updates card header and version status display).
3. Form Organization
All form sections now have consistent visual hierarchy:
View command line instructions
Manual merge helper
Use this merge commit message when completing the merge manually.
Checkout
From your project repository, check out a new branch and test the changes.Merge
Merge the changes and update on Forgejo.Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.