Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(dropdown, button, link): various bug fixes and aria fixes (Fixes #1814,#1817) #2159

Merged
merged 21 commits into from Nov 12, 2018

Conversation

Projects
None yet
1 participant
@tmorehouse
Copy link
Member

commented Nov 10, 2018

Description of Pull Request:

  • Various bug fixes for dropdown
  • Fixes for ARIA compliance on button/link when a non button/link tag is requested (assistive technology users would find dropdowns, buttons, and links were inaccessible via keyboard navigation and missing proper role attributes). Fixes ARIA issues introduced by PR #1929

Fixes #1814
Fixes #1817

Eliminates the need for the clickout.js mixin

PR checklist:

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • Feature
  • Enhancement to an existing feature
  • ARIA accessibility
  • Documentation update
  • Other, please describe:

Does this PR introduce a breaking change? (check one)

  • Yes
  • No

The PR fulfills these requirements:

  • It's submitted to the dev branch, not the master branch
  • When resolving a specific issue, it's referenced in the PR's title (i.e. fixes #xxxx[,#xxxx], where "xxxx" is the issue number)
  • PR titles should following the Conventional Commits naming convention (i.e. "fix(alert): not alerting during SSR render", "docs(badge): Updated pill examples, fix typos", "chore: fix typo in docs", etc)

If new features/enhancement/fixes are added or changed:

  • Includes documentation updates (including updating the component's package.json for slot and event changes)
  • New/updated tests are included and passing (if required)
  • Existing test suites are passing
  • The changes have not impacted the functionality of other components or directives
  • ARIA Accessibility has been taken into consideration (does it affect screen reader users or keyboard only users? clickable items should be in the tab index, etc)

tmorehouse added some commits Nov 7, 2018

fix(button): Add aria compliance when tag prop is neither 'button' or…
… 'a'

Ensure that the nonstandard tag is available in tab index and has an appropriate role, and that it is in tab order when not disabled
fix(button): ensure the appropriate role and tab index set on non lin…
…ks or buttons

Ensure that the nonstandard tag is available in tab index and has an appropriate role, and that it is in tab order when not disabled.

Fixes for ARIA compliance
@codecov

This comment has been minimized.

Copy link

commented Nov 10, 2018

Codecov Report

Merging #2159 into dev will increase coverage by 0.81%.
The diff coverage is 64.1%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev    #2159      +/-   ##
==========================================
+ Coverage   65.04%   65.85%   +0.81%     
==========================================
  Files         159      158       -1     
  Lines        2958     2932      -26     
  Branches      812      803       -9     
==========================================
+ Hits         1924     1931       +7     
+ Misses        748      727      -21     
+ Partials      286      274      -12
Impacted Files Coverage Δ
src/components/button/button.js 86.36% <100%> (+9.44%) ⬆️
src/mixins/dropdown.js 3.96% <6.66%> (-3.29%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3165e14...3bad47d. Read the comment docs.

tmorehouse added some commits Nov 12, 2018

dropdown mixin: listen for focusout on this.$el rather than this.$ref…
…s.menu

Allows for clicking the toggle button to close the menu.

Currently it was closing an immediately re-opening the menu

@tmorehouse tmorehouse changed the title [WIP] fix(dropdown, button, link): various bug fixes and aria fixes [WIP] fix(dropdown, button, link): various bug fixes and aria fixes (Fixes #1814,#1817) Nov 12, 2018

tmorehouse added some commits Nov 12, 2018

dropdown mixin: only add listeners when opene
Eliminates need for listenOnRoot mixin and clickout mixin
Delete clickout.js
No longer needed mixin

@tmorehouse tmorehouse removed the Status: WIP label Nov 12, 2018

@tmorehouse tmorehouse changed the title [WIP] fix(dropdown, button, link): various bug fixes and aria fixes (Fixes #1814,#1817) fix(dropdown, button, link): various bug fixes and aria fixes (Fixes #1814,#1817) Nov 12, 2018

@tmorehouse tmorehouse merged commit e79270d into dev Nov 12, 2018

7 of 9 checks passed

codecov/patch 64.1% of diff hit (target 65.04%)
Details
License Compliance FOSSA is analyzing this commit
Details
ci/circleci: build Your tests passed on CircleCI!
Details
ci/circleci: lint Your tests passed on CircleCI!
Details
ci/circleci: setup Your tests passed on CircleCI!
Details
ci/circleci: test-beta Your tests passed on CircleCI!
Details
ci/circleci: test-latest Your tests passed on CircleCI!
Details
codecov/project 65.85% (+0.81%) compared to 3165e14
Details
deploy/netlify Deploy preview ready!
Details

@tmorehouse tmorehouse deleted the tmorehouse/dropdown branch Nov 12, 2018

@tmorehouse tmorehouse referenced this pull request Nov 14, 2018

Closed

Bootstrap-Vue V2.0.0-rc.12 Tracker #2173

81 of 89 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.