Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add tooltip to the license explanation popover disclosure #612

Open
1 task
sarayourfriend opened this issue Feb 22, 2022 · 3 comments
Open
1 task

Add tooltip to the license explanation popover disclosure #612

sarayourfriend opened this issue Feb 22, 2022 · 3 comments
Assignees
Labels
♿️ aspect: a11y Concerns related to the project's accessibility ✨ goal: improvement Improvement to an existing user-facing feature 🟨 priority: medium Not blocking but should be addressed soon 🧱 stack: frontend Related to the Nuxt frontend 🎨 tech: css Involves CSS 🟨 tech: javascript Involves JavaScript 🔧 tech: vue Involves Vue.js

Comments

@sarayourfriend
Copy link
Collaborator

Problem

In WordPress/openverse-frontend#850 we implemented a new license explanation popover; however, we left out the tooltip that is meant to show on the disclosure due to the reasons outlined in this comment WordPress/openverse-frontend#850 (comment)

Description

We need to add a VTooltip component that implements the accessible behaviors for this kind of component. Once that is added, wrap the license explanation popover disclosure with it to show the tooltip on hover/focus. Please refer to the WAI-ARIA spec for the tooltip role.

Implementation

  • 🙋 I would be interested in implementing this feature.
@sarayourfriend sarayourfriend added 🟨 priority: medium Not blocking but should be addressed soon ✨ goal: improvement Improvement to an existing user-facing feature ♿️ aspect: a11y Concerns related to the project's accessibility 🟨 tech: javascript Involves JavaScript 🔧 tech: vue Involves Vue.js 🎨 tech: css Involves CSS labels Feb 22, 2022
@obulat obulat transferred this issue from WordPress/openverse-frontend Feb 22, 2023
@github-project-automation github-project-automation bot moved this to 📋 Backlog in Openverse Backlog Feb 23, 2023
@obulat obulat added the 🧱 stack: frontend Related to the Nuxt frontend label Feb 24, 2023
dhruvkb pushed a commit that referenced this issue Apr 14, 2023
@obulat obulat mentioned this issue May 24, 2023
8 tasks
@fcoveram
Copy link
Contributor

Here are the Figma links to:

@sarayourfriend
Copy link
Collaborator Author

@obulat Do you have a sense of whether this would be a good first issue or help wanted? I'm not familiar enough with the frontend anymore to make a good judgement of that.

@dhruvkb dhruvkb moved this from 📋 Backlog to 📅 To Do in Openverse Backlog May 13, 2024
@obulat obulat self-assigned this Jun 3, 2024
@obulat obulat added the ⛔ status: blocked Blocked & therefore, not ready for work label Jul 22, 2024
@obulat
Copy link
Contributor

obulat commented Jul 22, 2024

Blocked by the code freeze during Nuxt 3 migration.

@openverse-bot openverse-bot moved this from 📅 To Do to ⛔ Blocked in Openverse Backlog Jul 22, 2024
@obulat obulat removed the ⛔ status: blocked Blocked & therefore, not ready for work label Jul 31, 2024
@openverse-bot openverse-bot moved this from ⛔ Blocked to 📋 Backlog in Openverse Backlog Jul 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♿️ aspect: a11y Concerns related to the project's accessibility ✨ goal: improvement Improvement to an existing user-facing feature 🟨 priority: medium Not blocking but should be addressed soon 🧱 stack: frontend Related to the Nuxt frontend 🎨 tech: css Involves CSS 🟨 tech: javascript Involves JavaScript 🔧 tech: vue Involves Vue.js
Projects
Status: 📋 Backlog
Development

No branches or pull requests

3 participants