yahoo/jafar

Add classes to the default FieldView

Open

#36 opened on Feb 23, 2020

 (0 comments) (0 reactions) (0 assignees)JavaScript (9 forks)auto 404
enhancementgood first issuehelp wantedquestion

Repository metrics

Stars
 (109 stars)
PR merge metrics
 (No merged PRs in 30d)

Description

We should add classes to the default FieldView (in react-form) to indicate fields statues.

Question to figure before implementing: should we use aria- instead? And is all the alternative aria-* relevant for non input elements such a container div? is there a better way to describe this on the DOM?

To the root div of the field

  • jafar-field (always exists)
  • jafar-required (only if required=true)
  • jafar-empty (only if empty=true)
  • jafar-disabled (only if disabled=true)
  • jafar-invalid (only if invalid=true)
  • jafar-dirty (only if dirty=true)

To the div that wraps the component

  • jafar-component

To the div that wraps the error

  • jafar-error

To the header

  • jafar-header

To the div that wraps the label

  • jafar-label

To the div that wraps the description

  • jafar-description

To the div that wraps the asterisk of the required

  • jafar-required-indicator

Contributor guide