Skip to content

refactor(ui): QueryaIconButton and QueryaButton.icon are two icon-button systems #1440

Closed

Description

@ZhuchkaTriplesix

Problem

QueryaIconButton (26 call sites, Material ink and its own hover) and QueryaButton.icon (16 call sites, no ripple, the standard's states) look and react differently next to each other, for example in the SQL toolbar and in the results bar.

Scope

  • Make QueryaIconButton a thin wrapper over QueryaButton.icon (isActive, isDestructive, color / activeColor map to variant and tone), keeping density as the sm / md alias.
  • Move the call sites over, then remove the Material InkWell path.
  • Keep the 2 px keyboard focus ring and the selected semantics added in fix(a11y): focus ring and state semantics for QueryaIconButton #1395.

Found while driving a Linux debug build of dev after the button migration (UI consistency review). Part of epic #1331.

Activity

  1. added
    enhancementNew feature or request
    frontendTheme parser epic label: frontend
    uiUser interface components and widgets
    P2Medium priority / Parity & Refactoring
    on Oct 11, 2026
  2. ZhuchkaTriplesix commented on Oct 11, 2026

    @ZhuchkaTriplesix
    MemberAuthor

    Done in #1471: QueryaIconButton renders a ghost, icon-only QueryaButton; one icon-button system with the standard's states, focus ring and semantics.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    P2Medium priority / Parity & RefactoringenhancementNew feature or requestfrontendTheme parser epic label: frontenduiUser interface components and widgets

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions