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

fix(tabs): improve tab button focus styling logic #9916

Merged
merged 1 commit into from
Nov 15, 2016

Conversation

clshortfuse
Copy link
Contributor

Tab buttons did not receive focus style unless the user performed
certain keypress events while navigating. This logic did not account
for tabbing into the component

  • Set variable when tab button is focused
  • Use mdInteraction to detect keyboard interaction

This new logic is similar to that of mdCheckbox

Fixes #9039

Tab buttons did not receive focus style unless the user performed
certain keypress events while navigating. This logic did not account
for tabbing into the component

* Set variable when tab button is focused
* Use mdInteraction to detect keyboard interaction

This new logic is similar to that of mdCheckbox

Fixes angular#9039
@clshortfuse clshortfuse added ui: focus needs: review This PR is waiting on review from the team P0: critical Critical issues that must be addressed immediately. labels Oct 25, 2016
@clshortfuse clshortfuse added this to the 1.1.2 milestone Oct 25, 2016
Copy link
Member

@devversion devversion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

parentheses could be removed around declaration but not a big deal!

@ThomasBurleson ThomasBurleson added needs: presubmit and removed needs: review This PR is waiting on review from the team labels Oct 27, 2016
@clshortfuse clshortfuse added the a11y This issue is related to accessibility label Oct 27, 2016
@kara kara added pr: merge ready This PR is ready for a caretaker to review and removed needs: presubmit labels Nov 15, 2016
@kara kara merged commit 166fb79 into angular:master Nov 15, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a11y This issue is related to accessibility P0: critical Critical issues that must be addressed immediately. pr: merge ready This PR is ready for a caretaker to review ui: focus
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants