Skip to content
This repository has been archived by the owner on Mar 4, 2020. It is now read-only.

feat(Accessibility): Initial accessibility attributes schema. #1911

Merged
merged 12 commits into from
Sep 25, 2019

Conversation

mshoho
Copy link
Member

@mshoho mshoho commented Sep 10, 2019

This is a work-in-progress. The Stardust part shouldn't change, the JSON schema is about to be updated, but we can already start talking about it.

The key idea is that there is a schema which declaratively defines allowed accessibility attributes and how the accessible components can be composed. And there is a validator (which is being added to the non-production builds only) which validates the integrity at runtime and shows errors in case something is not satisfying the schema.

@codecov
Copy link

codecov bot commented Sep 24, 2019

Codecov Report

Merging #1911 into master will increase coverage by 0.02%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1911      +/-   ##
==========================================
+ Coverage   70.26%   70.29%   +0.02%     
==========================================
  Files         894      894              
  Lines        7894     7901       +7     
  Branches     2286     2289       +3     
==========================================
+ Hits         5547     5554       +7     
  Misses       2334     2334              
  Partials       13       13
Impacted Files Coverage Δ
...ackages/react/src/components/Dropdown/Dropdown.tsx 87.78% <100%> (+0.03%) ⬆️
packages/react/src/lib/renderComponent.tsx 84.05% <100%> (+1.51%) ⬆️

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 2ad5c2c...445b9b4. Read the comment docs.

@DustyTheBot
Copy link
Collaborator

DustyTheBot commented Sep 24, 2019

Warnings
⚠️ New package.json added: packages/ability-attributes/package.json. Make sure you have approval before merging!

Generated by 🚫 dangerJS

codecov.yml Outdated Show resolved Hide resolved
@mshoho mshoho merged commit 88d17b1 into microsoft:master Sep 25, 2019
@mshoho mshoho deleted the accessibility-attributes-schema branch September 25, 2019 10:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants