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

[Table] Row action tooltips should not be clickable #3771

Open
1 task done
gillibrand opened this issue Apr 11, 2023 · 0 comments
Open
1 task done

[Table] Row action tooltips should not be clickable #3771

gillibrand opened this issue Apr 11, 2023 · 0 comments
Labels
status: needs priority 📥 Enhancements without a `val #` label status: needs triage 🔍 type: enhancement 💡 New feature or request

Comments

@gillibrand
Copy link
Collaborator

What package is this for?

  • React

Summary

Row action buttons use a Carbon tooltip (black bubble). The problem is that the bubble it clickable, which clicks the button. It's too easy to inadvertently trigger and click the tooltip, performing the action unintentionally. Additionally, the Overflow button uses a native tootlip ("title"), so the buttons on a table row are inconsistent. Ideally, they will all use a native tooltip to be less intrusive.

There is similar work going on in Graphite to prefer native tooltips in some places: https://jsw.ibm.com/browse/GRAPHITE-60523.

Specific timeline issues / requests

Do you want this work within a specific time period? Is it related to an
upcoming release?

This has come up as a defect in ELM: 561411: Manage reports, row action > Clicking Edit hover tip opens report in Edit mode

Available extra resources

What resources do you have to assist this effort?

I, Jay, did start investigating this. The problem is, there doesn't seem to be a way to disable the Carbon tooltip on the button components used for row actions. Even passing null or empty string will show an empty black bubble still. Passing a "title" attribute does work as expected though. Possibly the empty tooltip can be hidden with CSS. I'm not sure how feasible it is to change this in Carbon since PAL is not on the latest Carbon release anyway. Due to other obligations I can't spend more time on this for a while.

@gillibrand gillibrand added type: enhancement 💡 New feature or request status: needs triage 🔍 status: needs priority 📥 Enhancements without a `val #` label labels Apr 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: needs priority 📥 Enhancements without a `val #` label status: needs triage 🔍 type: enhancement 💡 New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant