Skip to content
This repository was archived by the owner on Sep 5, 2024. It is now read-only.

Button tooltip comes back when dialog is canceled #8263

Closed
lelanophilus-zz opened this issue Apr 28, 2016 · 8 comments
Closed

Button tooltip comes back when dialog is canceled #8263

lelanophilus-zz opened this issue Apr 28, 2016 · 8 comments
Assignees
Labels
needs: investigation The cause of this issue is not well understood and needs to be investigated by the team or community P4: minor Minor issues. May not be fixed without community contributions.
Milestone

Comments

@lelanophilus-zz
Copy link

  1. Go to http://codepen.io/anon/pen/MyPOad (md-tooltips added to the basic usage example)
  2. Click prompt dialog
  3. tooltip goes away
  4. Select "I'm a cat person"
  5. tooltip is back though mouse is not hovering

The problem, I think, is that after the dialog closes it sets the focus back to the button, which causes the tooltip to activate. I expect the button to have the focus after the dialog closes only if the user was using the keyboard to navigate, not if they clicked on the button with a mouse.

@lelanophilus-zz
Copy link
Author

I think this might be an additional wrinkle on top of the problems discussed in this article: http://marcysutton.com/button-focus-hell/

@marcysutton
Copy link
Contributor

I'm not actually seeing that problem in Chrome. What browser?

@lelanophilus-zz
Copy link
Author

Chrome 50. Trying to reproduce the error it looks like it doesn't happen every time. It looks like repeating and sometimes choosing the other button or hovering over another button to view the custom dialog helps. I'll work on a more deterministic example.

I was able to get it to manifest after a few attempts though:

http://i.imgur.com/RqYIS3B.gif

@ThomasBurleson
Copy link
Contributor

@crisbeto - is this related to "tooltips get disconnected" issues?

@ThomasBurleson ThomasBurleson added type: bug needs: investigation The cause of this issue is not well understood and needs to be investigated by the team or community and removed type: bug labels Apr 29, 2016
@ThomasBurleson ThomasBurleson added this to the - Backlog milestone Apr 29, 2016
@crisbeto
Copy link
Member

My bet is that it is, but I'll double check.

@crisbeto
Copy link
Member

Tried it out, it's a different issue.

@crisbeto crisbeto added P4: minor Minor issues. May not be fixed without community contributions. needs: investigation The cause of this issue is not well understood and needs to be investigated by the team or community and removed needs: investigation The cause of this issue is not well understood and needs to be investigated by the team or community labels Apr 29, 2016
@ThomasBurleson ThomasBurleson modified the milestones: - Backlog, 1.1.1 May 17, 2016
@crisbeto
Copy link
Member

Re-opening and putting in the backlog since @devversion has a pending PR that should help with this (see #7965).

@crisbeto crisbeto reopened this May 24, 2016
@crisbeto crisbeto modified the milestones: - Backlog, 1.1.1 May 24, 2016
@ThomasBurleson ThomasBurleson modified the milestones: - Backlog, Deprecated May 26, 2016
@ThomasBurleson
Copy link
Contributor

This issue is closed as part of our ‘Surge Focus on Material 2' efforts.
For details, see our forum posting @ http://bit.ly/1UhZyWs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
needs: investigation The cause of this issue is not well understood and needs to be investigated by the team or community P4: minor Minor issues. May not be fixed without community contributions.
Projects
None yet
Development

No branches or pull requests

4 participants