Refine DarkShield admin UI with responsive design and branding - #3
Open
hoseinrza wants to merge 4 commits into
Open
Refine DarkShield admin UI with responsive design and branding#3hoseinrza wants to merge 4 commits into
hoseinrza wants to merge 4 commits into
Conversation
- Rewrite admin-style.css with CSS variables, card shadows, reusable layout helpers (stats row, actions row, filter row) and RTL support - Wire every admin view to use the existing .card/.darkshield-stat-card/ .darkshield-notice classes instead of duplicated inline styles - Fix .darkshield .nav-tab-wrapper selector never matching by adding the darkshield class to each page wrapper - Wrap all data tables (log, whitelist, scanner, performance) in a scrollable container so wide tables no longer break the layout on narrow screens - Restyle mode switch controls as proper option cards - Group service quick-add buttons with shared markup/classes
- New design-token system (brand indigo/cyan palette, spacing, radius, and shadow scales) instead of borrowing raw WP admin colors - Branded gradient header component (shared partial) with a live mode-status chip, replacing the plain <h1> on every screen - Nav tabs and the Performance sub-tabs rebuilt as a custom pill segmented control instead of WP's default nav-tab-wrapper look - Mode switching (dashboard quick-switch + settings radio group) redesigned as a shared icon-card grid with a clear active state - Stat cards, badges, and notices restyled around the new palette with colored accent bars for quicker scanning - Page structure, AJAX behavior, and settings logic are unchanged — visual layer only
The emoji character used across the redesigned UI (🛡️ 🟢 🟡 🔴 …) can be rewritten by WordPress core into <img> tags fetched from s.w.org unless its emoji loader is disabled — meaning our own screens could depend on an outside request even while National/Offline mode is supposedly cutting off external traffic. - Fix a wrong hook name in the "Block Emoji" blocker: WP core registers its admin-side emoji script on 'admin_print_scripts', not 'admin_head' (priority 7), so remove_action() was silently a no-op on wp-admin pages even with the setting enabled. - Unconditionally strip the same WP core emoji hooks while rendering any DarkShield admin screen, regardless of the "Block Emoji" toggle, so our own pages never trigger that external fetch.
…d4oxq5 Redesign admin UI with a custom DarkShield visual identity
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Refine DarkShield admin UI with responsive design and branding