Merge pull request #80 from TravisTheTechie/master

Adding `TabIndex` and `AutoFocus`.

Author: robdmoore

ChameleonForms.Example/Views/ExampleForms/Form1.cshtml

@@ -1,5 +1,6 @@
 @using ChameleonForms.Example.Forms.Components
 @using ChameleonForms.Templates.TwitterBootstrap3
+@using ChameleonForms.Component.Config
 @model ViewModelExample
 
 @{
@@ -15,12 +16,12 @@
 @using (var f = Html.BeginChameleonForm(method: FormMethod.Post, enctype: EncType.Multipart))
 {
     <p>@f.LabelFor(m => m.SomeCheckbox).Label("Are you ready for: ") @f.FieldElementFor(m => m.SomeCheckbox) @f.ValidationMessageFor(m => m.SomeCheckbox)</p>
-    <p>@f.FieldElementFor(m => m.RequiredStringField)</p>
+    <p>@f.FieldElementFor(m => m.RequiredStringField).TabIndex(4)</p>
     using (var s = f.BeginSection("My Section!", InstructionalText(), new{@class = "aClass"}.ToHtmlAttributes()))
     {
-        using (var ff = s.BeginFieldFor(m => m.RequiredStringField, Field.Configure().Attr("data-some-attr", "value")))
+        using (var ff = s.BeginFieldFor(m => m.RequiredStringField, Field.Configure().Attr("data-some-attr", "value").TabIndex(3)))
         {
-            @ff.FieldFor(m => m.NestedField).Attr("data-attr1", "value")
+            @ff.FieldFor(m => m.NestedField).Attr("data-attr1", "value").TabIndex(2)
             @ff.FieldFor(m => m.SomeEnum).Attr("data-attr1", "value")
         }
         @s.FieldFor(m => m.SomeCheckbox).AsDropDown()
@@ -29,7 +30,7 @@
             @ss.FieldFor(m => m.FileUpload).Attr("data-attr1", "value")
         }
         @s.FieldFor(m => m.RequiredStringField).OverrideFieldHtml(new MvcHtmlString("Custom html <b>she-yeah</b>!"))
-        @s.FieldFor(m => m.TextAreaField).Cols(60).Rows(5).Label("Some Label")
+        @s.FieldFor(m => m.TextAreaField).Cols(60).Rows(5).Label("Some Label").AutoFocus().TabIndex(1)
         @s.FieldFor(m => m.SomeCheckbox).InlineLabel("Some label").WithHint("Format: XXX")
         @s.FieldFor(m => m.SomeCheckbox).AsRadioList().WithTrueAs("True").WithFalseAs("False")
         @s.FieldFor(m => m.ListId)

ChameleonForms.Tests/Component/Config/FieldConfigurationShould.Set_autofocus_attribute.approved.html

@@ -0,0 +1 @@
+ autofocus="autofocus"

ChameleonForms.Tests/Component/Config/FieldConfigurationShould.Set_tabindex_attribute.approved.html

@@ -0,0 +1 @@
+ tabindex="44"

ChameleonForms.Tests/Component/Config/FieldConfigurationTests.cs

@@ -96,6 +96,22 @@ public void Set_placeholder_attribute()
             HtmlApprovals.VerifyHtml(fc.Attributes.ToHtmlString());
         }
 
+        [Test]
+        public void Set_autofocus_attribute()
+        {
+            var fc = Field.Configure().AutoFocus();
+
+            HtmlApprovals.VerifyHtml(fc.Attributes.ToHtmlString());
+        }
+
+        [Test]
+        public void Set_tabindex_attribute()
+        {
+            var fc = Field.Configure().TabIndex(44);
+
+            HtmlApprovals.VerifyHtml(fc.Attributes.ToHtmlString());
+        }
+
         [Test]
         public void Set_inline_label()
         {

ChameleonForms/ChameleonForms.csproj

@@ -75,6 +75,7 @@
   <ItemGroup>
     <Compile Include="Attributes\ExistsInAttribute.cs" />
     <Compile Include="Component\Config\FieldConfiguration.cs" />
+    <Compile Include="Component\Config\FieldConfigurationExtensions.cs" />
     <Compile Include="Component\Config\ReadonlyFieldConfiguration.cs" />
     <Compile Include="Component\FieldParent.cs" />
     <Compile Include="Component\Navigation.cs" />

ChameleonForms/Component/Config/FieldConfigurationExtensions.cs

@@ -0,0 +1,33 @@
+namespace ChameleonForms.Component.Config
+{
+    /// <summary>
+    /// Provides additional configuration options to FieldConfiguration
+    /// </summary>
+    public static class FieldConfigurationExtensions
+    {
+        /// <summary>
+        /// Sets the tab index of a given field
+        /// </summary>
+        /// <param name="config">Field configuration to update</param>
+        /// <param name="index">Tab index to be set</param>
+        /// <returns>The instance of IFieldConfiguration passed in to continue chaining things</returns>
+        public static IFieldConfiguration TabIndex(this IFieldConfiguration config, int index)
+        {
+            if (config != null)
+                return config.Attr("tabindex", index);
+            return null;
+        }
+
+        /// <summary>
+        /// Applys the autofocus attribute to a given field
+        /// </summary>
+        /// <param name="config">Field configuration to modify</param>
+        /// <returns>The instance of IFieldConfiguration passed in to continue chaining things</returns>
+        public static IFieldConfiguration AutoFocus(this IFieldConfiguration config)
+        {
+            if (config != null)
+                return config.Attr("autofocus", "autofocus");
+            return null;
+        }
+    }
+}