Inline form errors will be stable in release 8.4. Let's add support for it in the social themes. This will make the overal experience a lot better.
https://www.drupal.org/node/2504847

CommentFileSizeAuthor
#2 2912755-inline-form-errors-2.patch76.55 KBmaikelkoopman
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

maikelkoopman created an issue. See original summary.

maikelkoopman’s picture

Status: Active » Needs review
FileSize
76.55 KB

Pull request can be found here: https://github.com/goalgorilla/open_social/pull/578
The module will not be enabled by default until we update to drupal core version 8.4 in the distro.

maikelkoopman’s picture

Status: Needs review » Fixed

Will be part of release 1.6. The module is not enabled by default as long as Open Social is on D8.3.

  • maikelkoopman committed 05988be on 8.x-1.x
    #2912755 add errors to form template
    
  • maikelkoopman authored 1991c61 on 8.x-1.x
    Merge pull request #578 from goalgorilla/feature/2912755-inline-form-...
  • maikelkoopman committed 2388af0 on 8.x-1.x
    #2912755 by maikelkoopman: fix coding standards
    
  • maikelkoopman committed 437bb2f on 8.x-1.x
    #2912755 by maikelkoopman: correct locatiof errors on fieldset (radios)
    
  • maikelkoopman committed 4913779 on 8.x-1.x
    #2912755 by maikelkoopman: update runsequence of build styleguide task
    
  • maikelkoopman committed 5583ea9 on 8.x-1.x
    #2912755 by maikelkoopman: update outdated info in styleguide
    
  • maikelkoopman committed 72ffab9 on 8.x-1.x
    #2912755 by maikelkoopman: merge with develop
    
  • maikelkoopman committed 8678f7e on 8.x-1.x
    #2912755 by maikelkoopman: theme inline form errors, remove unused...
  • maikelkoopman committed 8d75ca7 on 8.x-1.x
    #2912755 by maikelkoopman: fix inline errors for date field and border...
  • maikelkoopman committed 8f51c11 on 8.x-1.x
    #2912755 by maikelkoopman: document inline form errors in style guide
    
  • maikelkoopman committed bca003b on 8.x-1.x
    #2912755 by maikelkoopman: remove brand-border-radius class, which is...
  • maikelkoopman committed bd565d0 on 8.x-1.x
    #2912755 by maikelkoopman: decrease font of feedback
    
  • maikelkoopman committed d06ac06 on 8.x-1.x
    #2912755 by maikelkoopman: differentiate radius of alerts as messages...
  • maikelkoopman committed d16c16b on 8.x-1.x
    #2912755 by maikelkoopman: include full search from using border radius...
  • maikelkoopman committed e3b3cf9 on 8.x-1.x
    #2912755 by maikelkoopman: split up border radius settings into cards,...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.