Skip to content

HoverCard: example accessibility fix.#5027

Merged
micahgodbolt merged 3 commits intomicrosoft:masterfrom
Vitalius1:v-vibr/HoverCard-keyboardFix
May 30, 2018
Merged

HoverCard: example accessibility fix.#5027
micahgodbolt merged 3 commits intomicrosoft:masterfrom
Vitalius1:v-vibr/HoverCard-keyboardFix

Conversation

@Vitalius1
Copy link
Copy Markdown
Contributor

@Vitalius1 Vitalius1 commented May 30, 2018

Pull request checklist

  • Addresses an existing issue: Fixes #0000
  • Include a change request file using $ npm run change

Description of changes

Fixing an example where user couldn't navigate with the keyboard to the HoverCard target.

Microsoft Reviewers: Open in CodeFlow

@Vitalius1 Vitalius1 requested review from Jahnp and atneik as code owners May 30, 2018 01:17
@Vitalius1
Copy link
Copy Markdown
Contributor Author

@Jahnp will need your click in here))) Thanks!

@micahgodbolt micahgodbolt merged commit c78b98f into microsoft:master May 30, 2018
Markionium added a commit to Markionium/office-ui-fabric-react that referenced this pull request Jun 1, 2018
* master: (95 commits)
  Applying package updates.
  Experiments/Nav component: display "show more" link only if there is atleast one hidden link (microsoft#5057)
  Add pointerup listener to exit keytip mode (microsoft#5051)
  Update PULL_REQUEST_TEMPLATE.md
  Update ISSUE_TEMPLATE.md
  Shimmer: resolve conflicts (microsoft#5034)
  Invalid ARIA attributes: Fix empty values (microsoft#5040)
  ComboBox: Correct invalid ARIA attributes. (microsoft#4873) (microsoft#5001)
  ComboBox: Fix submit pending value (microsoft#5048)
  FocusTrapZone - restore last focused descendant element (microsoft#4897)
  Applying package updates.
  Take 2 of the require.resolve change. This time using the "resolve" pkg (microsoft#5031)
  fixing webpack config to allow rush build to complete on a small VM (microsoft#5037)
  MessageBar: change color of X close button so that it is accessible (microsoft#5039)
  Theming: improve accessibility (microsoft#5038)
  Applying package updates.
  Added 'made with fabric' to readme (microsoft#5018)
  HoverCard: example accessibility fix. (microsoft#5027)
  Dropdown caret (microsoft#5016)
  Applying package updates.
  ...
@Vitalius1 Vitalius1 deleted the v-vibr/HoverCard-keyboardFix branch July 3, 2018 00:03
@Vitalius1 Vitalius1 restored the v-vibr/HoverCard-keyboardFix branch July 3, 2018 00:03
@Vitalius1 Vitalius1 deleted the v-vibr/HoverCard-keyboardFix branch July 3, 2018 00:04
@microsoft microsoft locked as resolved and limited conversation to collaborators Aug 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants