Skip to content

fix(tab): typo aria-lablelledby #954

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

Merged
merged 1 commit into from
Aug 30, 2017
Merged

fix(tab): typo aria-lablelledby #954

merged 1 commit into from
Aug 30, 2017

Conversation

indr
Copy link

@indr indr commented Aug 30, 2017

There is a small type for the attribute :aria-labelledby.

@codecov-io
Copy link

Codecov Report

Merging #954 into master will increase coverage by 0.09%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #954      +/-   ##
==========================================
+ Coverage      40%   40.09%   +0.09%     
==========================================
  Files          68       68              
  Lines        2217     2217              
  Branches      633      633              
==========================================
+ Hits          887      889       +2     
+ Misses       1171     1169       -2     
  Partials      159      159
Impacted Files Coverage Δ
lib/components/tab.vue 46.66% <ø> (ø) ⬆️
lib/components/form-fieldset.vue 67.34% <0%> (+4.08%) ⬆️

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 136960d...de0d16a. Read the comment docs.

@indr indr changed the title Fix typo aria-lablelledby fix(tab): Fix typo aria-lablelledby Aug 30, 2017
@indr indr changed the title fix(tab): Fix typo aria-lablelledby fix(tab): typo aria-lablelledby Aug 30, 2017
@tmorehouse tmorehouse merged commit 72a33c7 into bootstrap-vue:master Aug 30, 2017
costeirs added a commit to costeirs/bootstrap-vue that referenced this pull request Jul 23, 2018
This was fixed in bootstrap-vue#954 , but apparently somewhere along the way `lib/` was changed to `src/` and the fix was lost.
Found the issue because it was affecting my Lighthouse accessibility scores.
mosinve pushed a commit that referenced this pull request Jul 24, 2018
This was fixed in #954 , but apparently somewhere along the way `lib/` was changed to `src/` and the fix was lost.
Found the issue because it was affecting my Lighthouse accessibility scores.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants