Skip to content
This repository has been archived by the owner on Jul 6, 2021. It is now read-only.

Implement RovingTabIndexMixin and 2 other mixins #7

Merged
merged 4 commits into from
Dec 2, 2019

Conversation

web-padawan
Copy link
Member

@web-padawan web-padawan commented Nov 18, 2019

Fixes #4

This PR contains the 3 mixins prototyped at vaadin/vaadin-accordion#30

  1. SlottedItemsMixin
  2. KeyboardDirectionMixin
  3. RovingTabIndexMixin

They are needed for the following components:

  • vaadin-accordion: 1 and 2 only
  • vaadin-tabs: 1, 2 3
  • vaadin-list-box: 1, 2, 3

@web-padawan web-padawan force-pushed the roving-tabindex branch 2 times, most recently from 0ec9429 to 06c018a Compare November 29, 2019 11:47
@web-padawan web-padawan changed the title feat: implement roving-tabindex-mixin Implement RovingTabIndexMixin and 2 other mixins Nov 29, 2019
Copy link

@Haprog Haprog left a comment

Choose a reason for hiding this comment

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

Looks good. Found just one small typo and one sentence that is missing something at the end. Please fix those, but I'm approving this already.

packages/keyboard-direction-mixin/README.md Outdated Show resolved Hide resolved
packages/slotted-items-mixin/README.md Outdated Show resolved Hide resolved
@web-padawan web-padawan merged commit 44635e2 into master Dec 2, 2019
@web-padawan web-padawan deleted the roving-tabindex branch December 2, 2019 09:52
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.

Implement roving tabindex mixin
2 participants