Skip to content

Fix dark-mode interactive states + match-master wide-table scroll - #63

Merged
Fivell merged 1 commit into
masterfrom
fix/dark-mode-interactive-states
Oct 12, 2026
Merged

Fivell merged 1 commit into
masterfrom
fix/dark-mode-interactive-states

Conversation

@Fivell

@Fivell Fivell commented Oct 12, 2026

Copy link
Copy Markdown
Member

Dark-mode QA pass on the admin chrome (found while bumping consumers to theme 3.x). All changes in wigu/active_admin_theme.scss, driven by the --aa-* tokens.

Fixed

  • select2 clear '×' — transparent/muted glyph, no oversized blue hover square (defensive for select2 4.0 span + 4.1 button).
  • pagination bottom — restored below the table (#index_footer { clear: both }); the float:left top controls no longer pull the footer above the table.
  • disabled inputs / select2 — muted surface + opacity + not-allowed so they read as disabled in dark.
  • confirm dialog — .ui-dialog-content { min-height: 0 } removes jQuery-UI's reserved empty space.
  • pagination page-number hover — white text on the accent fill (contrast was ~1.16:1, now ~2.74:1).
  • dropdown caret (Actions etc.) — visible colour on the dark themed button (was dark-on-dark).
  • dropdown / flyout menus — dropped the per-item border so it reads as a menu, not a table.
  • wide index tables — scroll the whole page like master (reverts the earlier contain approach) and pin #utility_nav position: sticky so the clock / theme toggle / logout stay reachable while scrolling sideways.

Verified live on a consumer admin (pbx) in dark + light, measured before/after (contrast ratios, element positions). Does not touch the clock refactor (separate PR).

Dark-mode QA pass on the admin chrome:
- select2 clear 'x': transparent/muted glyph, no oversized blue hover square
- pagination: restore the bottom control below the table (clear:both on #index_footer)
- disabled inputs / select2: muted surface + opacity so they read as disabled in dark
- confirm dialog: stop reserving empty space (ui-dialog-content min-height:0)
- pagination page-number hover: white text on the accent fill (contrast was ~1:1)
- dropdown caret (Actions etc.): visible colour on the dark themed button
- dropdown/flyout: drop the per-item border so it reads as a menu, not a table
- wide index tables: scroll the whole page like master (revert the contain
  approach) and pin #utility_nav sticky so the clock/toggle/logout stay reachable
@Fivell
Fivell merged commit a311508 into master Oct 12, 2026
2 checks passed
@Fivell
Fivell deleted the fix/dark-mode-interactive-states branch October 12, 2026 11:46
@Fivell Fivell mentioned this pull request Oct 20, 2026
senid231 pushed a commit to senid231/active_admin_theme that referenced this pull request Nov 4, 2026
Minor, not major: the variable contract only grows - $skinTitleBarHeight
joins it - and nothing that worked against 3.1.1 stops working. Several
defaults do look different, and the Upgrading section now spells them out.

What moved since 3.1.1:

  activeadmin-plugins#74  a title-bar dropdown opens under its own button again
  activeadmin-plugins#73  button_to action items stay on the title bar's row
  activeadmin-plugins#72  a wigu/all umbrella require; title-bar dropdowns right-align
  activeadmin-plugins#71  the header, title bar and flash bar are pinned to the viewport and
       index-table headers stick, with per-table opt-outs
  activeadmin-plugins#70  date-range filters laid out with flexbox
  activeadmin-plugins#69  documentation for crisscross highlighting and the Trix editor
  activeadmin-plugins#68  crisscross cell highlighting for index tables
  activeadmin-plugins#67  Trix / ActionText editor themed in light and dark
  activeadmin-plugins#65  --aa-datetimepicker-* bridge variables
  activeadmin-plugins#64  utility nav with the server clock, theme switch and glyph logout
  activeadmin-plugins#63  dark-mode interactive states
  activeadmin-plugins#62  dark styling for select2 and tom-select

The npm package moves with it: src/ is regenerated from app/assets by the
prepare script, so the Sprockets and bundler consumers ship the same files.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant