8000 Add aria attributes on error fields by bjorvack · Pull Request #3485 · forkcms/forkcms · GitHub
[go: up one dir, main page]

Skip to content

Add aria attributes on error fields#3485

Merged
Merged

Add aria attributes on error fields#3485
carakas merged 4 commits intoforkcms:masterfrom
bjorvack:aria-stuff-on-forms

Conversation

@bjorvack
Copy link
Contributor

Type

  • Enhancement

Pull request description

Forms with errors should have the necessary aria attributes

@codecov
Copy link
codecov bot commented Dec 16, 2021

Codecov Report

Merging #3485 (5d09269) into master (3a0eb13) will not change coverage.
The diff coverage is n/a.

❗ Current head 5d09269 differs from pull request most recent head 8df6fd4. Consider uploading reports for the commit 8df6fd4 to get more accurate results
Impacted file tree graph

@@            Coverage Diff            @@
##             master    #3485   +/-   ##
=========================================
  Coverage     27.88%   27.88%           
  Complexity     8145     8145           
=========================================
  Files           575      575           
  Lines         30691    30691           
=========================================
  Hits           8557     8557           
  Misses        22134    22134           
Flag Coverage Δ
functional 23.78% <ø> (ø)
installer 3.84% <ø> (ø)
unit 7.69% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


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 3a0eb13...8df6fd4. Read the comment docs.

@bjorvack bjorvack requested a review from a team as a code owner December 17, 2021 09:41
@carakas carakas added this to the 5.11.1 milestone Jan 1, 2022
@carakas carakas merged commit 7a9e5eb into forkcms:master Jan 1, 2022
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.

2 participants

0