Skip to content

Conversation

@daltontf
Copy link
Contributor

I can navigate to "Logout" using the keyboard, but cannot select it.

I can navigate to "Logout" using the keyboard, but cannot select it.

Signed-off-by: Tim Dalton <daltont@objectcomputing.com>
<Avatar initials={$user?.username.charAt(0).toUpperCase()} />
</button>
<Dropdown.Items slot="items">
<Dropdown.Items.Item on:click={logout} label="Logout"></Dropdown.Items.Item>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

ChatGPT had suggestions involving role, as, tabindex attributes, but they didn't work. This did.

Signed-off-by: Tim Dalton <daltont@objectcomputing.com>
mvolkmann
mvolkmann previously approved these changes Dec 18, 2025
…ork.

Signed-off-by: Tim Dalton <daltont@objectcomputing.com>
goto('/');
}
function logout_keydown(e: CustomEvent) {

Choose a reason for hiding this comment

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

It looks like the convention in this project is for method names to be camelCase.

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