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

md-button with tooltip displays focus style on window refocus #4245

Closed
mhchen opened this issue Aug 19, 2015 · 7 comments
Closed

md-button with tooltip displays focus style on window refocus #4245

mhchen opened this issue Aug 19, 2015 · 7 comments
Assignees
Milestone

Comments

@mhchen
Copy link

mhchen commented Aug 19, 2015

Similar to #3035

Steps to reproduce:

  1. Click on a button, observe that the focus style (darker color) is present.
  2. Mouse off the button, observe that the focus style fades.
  3. Tab to another window, and tab back.

Expected: button should not have focus style
Actual: button regains its focus style

@mhchen mhchen self-assigned this Aug 19, 2015
@mhchen mhchen changed the title md-button displays focus style on window refocus md-button with tooltip displays focus style on window refocus Aug 20, 2015
@mhchen
Copy link
Author

mhchen commented Aug 20, 2015

Sorry, to clarify, this only happens when used with the tooltip

@galenbrazell
Copy link

Also witnessing this, here's a gif:
focused-event-bug

@mhchen
Copy link
Author

mhchen commented Aug 20, 2015

Blocked by #4249 as the focus state is always shown

@galenbrazell
Copy link

Also seeing this with any buttons, no tooltip required:

focused-event-bug-button

@ThomasBurleson ThomasBurleson assigned topherfangio and unassigned mhchen Sep 2, 2015
@ThomasBurleson ThomasBurleson modified the milestone: 0.12.0 Sep 2, 2015
@topherfangio
Copy link
Contributor

So, is the underlying issue that the button does not lose focus once clicked?

I think the code might assume that some other element will gain the focus after whatever action the click triggers fires, but since we don't fire any real action on the demo site, the button never loses focus.

Can you describe how exactly this is affecting your live apps (or maybe provide a link/demo) so that we can make sure to prepare a good fix?

@topherfangio topherfangio added the needs: feedback The issue creator or community need to respond to questions in this issue label Sep 3, 2015
@topherfangio
Copy link
Contributor

I believe this is related to #4249.

@SanderElias
Copy link
Contributor

This seems to be a duplicate for #4249, I'm closing this one, so we can concentrate this on 1 place

@SanderElias SanderElias added resolution: duplicate and removed needs: feedback The issue creator or community need to respond to questions in this issue labels Nov 3, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants