Repository navigation
add webid hover on account - #927
SharonStrats wants to merge 2 commits into
Conversation
There was a problem hiding this comment.
🟢 Approval recommended
There are no active review issues blocking the account-menu WebID tooltip and accessible-label change.
No open issues 🎉
What changed in this PR
Adds WebID information to the account menu trigger so users can identify the active account on hover and through its accessible name.
Changes:
- Displays the account’s WebID in a native hover tooltip.
- Adds an accessible label containing the WebID.
- Fixes indentation in an existing code editor accessibility test without changing its behavior.
| File | Description |
|---|---|
test/unit/codeEditor.test.js |
Corrects indentation in the accessible-description test. |
src/components/account/Account.ts |
Adds the WebID tooltip and accessible label to the account button. |
Review effort: Balanced
…istry@5.0.2-3) (latest: rdflib@2.4.1)
|
I believe we should not implement hover on solidOS because it is not mobile nor accessibility friendly. We need to find other Ui solutions for their need. |
Uh oh!
There was an error while loading. Please reload this page.