"error" HTML class causes UX issues with vertical tabs

attaching a static patch for composer builds

Can we have this issue checked on 3.x and update the MR? I will merge in 2.1.x too if valid and works.

Bala.addweb → changed the visibility of the branch 3446306-error-html-class to hidden.

Bala.addweb → changed the visibility of the branch 3446306-error-html-class to active.

Can we have this issue checked on 3.x and update the MR? I will merge in 2.1.x too if valid and works.

Raised MR against 3.x
Kindly review.

@recrit When it is a "Hard limit" the source of the validation failure for the user should clear. We should make sure to do so and use soft-error when there is no "Hard limit" enabled.

@joevagyok it has been a minute since I created this patch, but I thought changing the class to "error" when it was enforced handled the "hard limit" OR am I missing something?

        if ($this.hasClass('maxlength_js_enforce')) {
          options['enforce'] = true;
          options.cssExceeded = 'error';
        }

I had a look and that is correct.

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