8000 Fix tooltips in Security defense tab in admin console by mabartos · Pull Request #53585 · keycloak/keycloak · GitHub
[go: up one dir, main page]

Skip to content

Fix tooltips in Security defense tab in admin console - #53585

Merged
edewit merged 1 commit into
keycloak:mainfrom
mabartos:KC-53142
Oct 6, 2026
Merged

edewit merged 1 commit into
keycloak:mainfrom
mabartos:KC-53142

Conversation

@mabartos
@mabartos mabartos commented Oct 6, 2026
Copy link
Copy Markdown
Member
image

@mabartos
mabartos requested review from a team as code owners October 6, 2026 09:50
Copilot AI balanced review requested due to automatic review settings October 6, 2026 09:50
@mabartos
mabartos requested a review from edewit October 6, 2026 09:51
Copilot AI 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.

Copilot review overview

🟡 Changes recommended

The tooltip regression needs automated coverage in the existing Security Defenses tests.

Review effort: Balanced
Findings: 1 Low severity

Open (1)
What changed in this PR

Restores tooltip behavior for Security Defenses help text.

Changes:

  • Wraps translated help content in HelpItem.
  • Preserves external documentation links inside tooltips.
File Description
HelpLinkTextInput.tsx Restores help icons and popovers for security-header fields.

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

Closes keycloak#53142

Signed-off-by: Martin Bartoš <mabartos@redhat.com>
Copilot AI balanced review requested due to automatic review settings October 6, 2026 10:03
Copilot AI 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.

Copilot review overview

🟢 Approval recommended

The focused fix correctly restores tooltip behavior and includes regression coverage.

Review effort: Balanced
Findings: None

Resolved since last review (1)

@edewit
edewit merged commit ff2c490 into keycloak:main Oct 6, 2026
73 checks passed
Awambeng pushed a commit to adorsys/keycloak-oid4vc that referenced this pull request Oct 6, 2026
Closes keycloak#53142

Signed-off-by: Martin Bartoš <mabartos@redhat.com>
edewit pushed a commit that referenced this pull request Oct 7, 2026
Closes #53142


(cherry picked from commit ff2c490)

Signed-off-by: Martin Bartoš <mabartos@redhat.com>
PadzerButch pushed a commit to PadzerButch/-50764_test_migration_keycloak that referenced this pull request Oct 22, 2026
Closes keycloak#53142

Signed-off-by: Martin Bartoš <mabartos@redhat.com>
PadzerButch pushed a commit to PadzerButch/-50764_test_migration_keycloak that referenced this pull request Oct 23, 2026
Closes keycloak#53142

Signed-off-by: Martin Bartoš <mabartos@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Realm settings - security defenses

3 participants

0