Skip to content

Header menu colors - #47

Closed
dmitry-sinina wants to merge 2 commits into
activeadmin-plugins:masterfrom
yeti-switch:header_menu_colors
Closed

dmitry-sinina wants to merge 2 commits into
activeadmin-plugins:masterfrom
yeti-switch:header_menu_colors

Conversation

@dmitry-sinina

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

Hovered dropdown items override the newly configurable text color with hard-coded white.

Review effort: Balanced
Findings: 1 Medium severity

Open (1)
What changed in this PR

Adds configurable header menu and title-bar styling while improving dropdown sizing and nesting behavior.

Changes:

  • Adds menu, header, logo, and title-bar variables.
  • Improves dropdown sizing and nested-menu behavior.
  • Applies configurable spacing and colors.
File Description
app/​assets/​stylesheets/​wigu/​active_admin_theme.scss Adds and applies theme customization variables.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

}
li:hover > a, li.current > a {
background-color: $skinMenuItemHoverColor;
color: #ffffff;
@Fivell

Fivell commented Oct 2, 2026

Copy link
Copy Markdown
Member

merged in #51 thanks

@Fivell Fivell closed this Oct 2, 2026
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.

3 participants