Skip to content

Conversation

@thomasdarimont
Copy link
Contributor

  • Add missing icons with help labels
  • Use correct help text for webAuthnPolicyCreateTimeout

Fixes #34465

@thomasdarimont thomasdarimont requested a review from a team as a code owner October 29, 2024 22:01
@thomasdarimont
Copy link
Contributor Author

thomasdarimont commented Oct 29, 2024

FYI @tnorimat I added missing help label icons of the Webauthn Policy and Webauthn Passwordless Policy settings and revised some help texts.

Before:
Screenshot 2024-10-29 at 22 10 51

After:
Screenshot 2024-10-29 at 23 02 54

Copy link
Contributor

@tnorimat tnorimat left a comment

Choose a reason for hiding this comment

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

Thank you for the PR. It looks good to me.

- Add missing icons with help labels
- Use correct help text for webAuthnPolicyCreateTimeout

Fixes keycloak#34465

Signed-off-by: Thomas Darimont <[email protected]>
@thomasdarimont thomasdarimont force-pushed the issue/gh-34465-fix-missing-helpicons-in-webauthn-policy branch from 27d5f82 to 8b9d7b3 Compare October 30, 2024 08:26
@edewit edewit merged commit e41ca1f into keycloak:main Oct 30, 2024
edewit pushed a commit to edewit/keycloak that referenced this pull request Nov 6, 2024
…oak#34466)

- Add missing icons with help labels
- Use correct help text for webAuthnPolicyCreateTimeout

Fixes keycloak#34465

Signed-off-by: Thomas Darimont <[email protected]>
(cherry picked from commit e41ca1f)
edewit pushed a commit to edewit/keycloak that referenced this pull request Nov 11, 2024
…oak#34466)

- Add missing icons with help labels
- Use correct help text for webAuthnPolicyCreateTimeout

Fixes keycloak#34465

Signed-off-by: Thomas Darimont <[email protected]>
(cherry picked from commit e41ca1f)
jonkoops pushed a commit that referenced this pull request Nov 11, 2024
- Add missing icons with help labels
- Use correct help text for webAuthnPolicyCreateTimeout

Fixes #34465

Signed-off-by: Thomas Darimont <[email protected]>
(cherry picked from commit e41ca1f)

Co-authored-by: Thomas Darimont <[email protected]>
@edewit edewit mentioned this pull request Nov 19, 2024
@edewit edewit mentioned this pull request Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Missing help icons in Webauthn Policy and Webauthn Passwordless Policy missing in admin ui

3 participants