Skip to content

Test and Index fix#682

Open
NeptuneHub wants to merge 3 commits into
mainfrom
devel
Open

Test and Index fix#682
NeptuneHub wants to merge 3 commits into
mainfrom
devel

Conversation

@NeptuneHub

@NeptuneHub NeptuneHub commented Jun 22, 2026

Copy link
Copy Markdown
Owner

Test and Index fix

PR test builds:

NeptuneHub and others added 2 commits June 22, 2026 18:14
Accessibility:
- Global :focus-visible keyboard focus ring (style.css; white variant in sidebar via menu.css)
- Nav toggles: hamburger and "Administration" submenu get aria-label/aria-expanded/aria-controls;
  Administration is now a <button>; arrow marked aria-hidden
- Sidebar focus management: focus moves in on open, Esc closes and returns focus, closed
  off-screen sidebar marked inert + aria-hidden (no longer tabbable)
- aria-live status regions + role=progressbar(aria-valuenow) for long-task/search feedback
  (index, similarity, chat)
- Accessible combobox keyboard nav for the song autocomplete (similarity, map): Up/Down/Enter/Esc,
  combobox/listbox/option roles, aria-activedescendant

Button system convergence:
- Make .btn a drop-in (0.75rem 1.5rem); add .btn-success and .btn-block utilities
- Migrate every primary-action button across 13 templates to .btn .btn-{primary,danger,success} .btn-block
- Remove duplicated per-page .danger-button/.active-button blocks, the per-ID color selector lists,
  the !important green, and the brittle button[style*="#16A34A"] attribute selector
- Note: artist-similarity "Create Playlist" button is now green (success) for consistency with all
  other "Create Playlist on Media Server" buttons

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a shared rate limiter using flask-limiter to protect authentication and user administration endpoints, and implements a versioned, ordered database migration framework. It also enhances frontend accessibility with improved keyboard navigation, ARIA attributes, and focus styles, while refactoring button styles into shared utility classes. Feedback on these changes highlights a potential startup crash if the rate limit configuration strings are empty, suggesting a check to ensure they are truthy before applying the decorator.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread app_auth.py
@github-advanced-security

Copy link
Copy Markdown
Contributor

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

@sonarqubecloud

Copy link
Copy Markdown

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.

2 participants