From 88e8ca2f775c0878328523ab3a0df206ee6f3286 Mon Sep 17 00:00:00 2001 From: Cameron Cundiff Date: Thu, 11 Apr 2013 08:20:32 -0400 Subject: [PATCH] Move aria-required option to html5 component. [Finishes #780] --- lib/simple_form/components/html5.rb | 5 ++++- lib/simple_form/inputs/base.rb | 1 - 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/lib/simple_form/components/html5.rb b/lib/simple_form/components/html5.rb index cb513703..545b8f16 100644 --- a/lib/simple_form/components/html5.rb +++ b/lib/simple_form/components/html5.rb @@ -7,7 +7,10 @@ module SimpleForm def html5 @html5 = true - input_html_options[:required] = true if has_required? + if has_required? + input_html_options[:required] = true + input_html_options['aria-required'] = true + end nil end diff --git a/lib/simple_form/inputs/base.rb b/lib/simple_form/inputs/base.rb index 79fbd2ce..a6d6f550 100644 --- a/lib/simple_form/inputs/base.rb +++ b/lib/simple_form/inputs/base.rb @@ -66,7 +66,6 @@ module SimpleForm @input_html_classes = @html_classes.dup @input_html_options = html_options_for(:input, input_html_classes).tap do |o| - o['aria-required'] = true if has_required? o[:readonly] = true if has_readonly? o[:disabled] = true if has_disabled? o[:autofocus] = true if has_autofocus?