Repository: struts
Updated Branches:
  refs/heads/master ad367777c -> 533b236fd


WW-4739 <s:reset> tag does not properly interpret the attribute tabindex


Project: http://git-wip-us.apache.org/repos/asf/struts/repo
Commit: http://git-wip-us.apache.org/repos/asf/struts/commit/533b236f
Tree: http://git-wip-us.apache.org/repos/asf/struts/tree/533b236f
Diff: http://git-wip-us.apache.org/repos/asf/struts/diff/533b236f

Branch: refs/heads/master
Commit: 533b236fdbfb0df3a3b1ea459e88e69f1c3b3ec7
Parents: ad36777
Author: Aleksandr Mashchenko <amashche...@apache.org>
Authored: Thu Feb 2 21:56:53 2017 +0200
Committer: Aleksandr Mashchenko <amashche...@apache.org>
Committed: Thu Feb 2 21:56:53 2017 +0200

----------------------------------------------------------------------
 core/src/main/resources/template/simple/reset.ftl              | 6 ++++++
 .../test/java/org/apache/struts2/views/jsp/ui/ResetTest.java   | 2 ++
 .../test/resources/org/apache/struts2/views/jsp/ui/Reset-1.txt | 2 +-
 .../test/resources/org/apache/struts2/views/jsp/ui/Reset-3.txt | 2 +-
 4 files changed, 10 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/struts/blob/533b236f/core/src/main/resources/template/simple/reset.ftl
----------------------------------------------------------------------
diff --git a/core/src/main/resources/template/simple/reset.ftl 
b/core/src/main/resources/template/simple/reset.ftl
index 7e7481f..67d6505 100644
--- a/core/src/main/resources/template/simple/reset.ftl
+++ b/core/src/main/resources/template/simple/reset.ftl
@@ -37,6 +37,9 @@
 <#if parameters.disabled!false>
  disabled="disabled"<#rt/>
 </#if>
+<#if parameters.tabindex?has_content>
+ tabindex="${parameters.tabindex?html}"<#rt/>
+</#if>
 <#include 
"/${parameters.templateDir}/${parameters.expandTheme}/scripting-events.ftl"/>
 <#include 
"/${parameters.templateDir}/${parameters.expandTheme}/common-attributes.ftl" />
 <#include 
"/${parameters.templateDir}/${parameters.expandTheme}/dynamic-attributes.ftl" />
@@ -69,6 +72,9 @@
 <#if parameters.disabled!false>
  disabled="disabled"<#rt/>
 </#if>
+<#if parameters.tabindex?has_content>
+ tabindex="${parameters.tabindex?html}"<#rt/>
+</#if>
 <#include 
"/${parameters.templateDir}/${parameters.expandTheme}/scripting-events.ftl" />
 <#include 
"/${parameters.templateDir}/${parameters.expandTheme}/common-attributes.ftl" />
 <#include 
"/${parameters.templateDir}/${parameters.expandTheme}/dynamic-attributes.ftl" />

http://git-wip-us.apache.org/repos/asf/struts/blob/533b236f/core/src/test/java/org/apache/struts2/views/jsp/ui/ResetTest.java
----------------------------------------------------------------------
diff --git a/core/src/test/java/org/apache/struts2/views/jsp/ui/ResetTest.java 
b/core/src/test/java/org/apache/struts2/views/jsp/ui/ResetTest.java
index 913ead8..6db4adf 100644
--- a/core/src/test/java/org/apache/struts2/views/jsp/ui/ResetTest.java
+++ b/core/src/test/java/org/apache/struts2/views/jsp/ui/ResetTest.java
@@ -59,6 +59,7 @@ public class ResetTest extends AbstractUITagTest {
         tag.setLabel("mylabel");
         tag.setName("myname");
         tag.setValue("%{foo}");
+        tag.setTabindex("1");
 
         tag.doStartTag();
         tag.doEndTag();
@@ -75,6 +76,7 @@ public class ResetTest extends AbstractUITagTest {
         tag.setType("button");
         tag.setName("myname");
         tag.setValue("%{foo}");
+        tag.setTabindex("1");
 
         tag.doStartTag();
         tag.doEndTag();

http://git-wip-us.apache.org/repos/asf/struts/blob/533b236f/core/src/test/resources/org/apache/struts2/views/jsp/ui/Reset-1.txt
----------------------------------------------------------------------
diff --git 
a/core/src/test/resources/org/apache/struts2/views/jsp/ui/Reset-1.txt 
b/core/src/test/resources/org/apache/struts2/views/jsp/ui/Reset-1.txt
index 699faf7..4d6b7fc 100644
--- a/core/src/test/resources/org/apache/struts2/views/jsp/ui/Reset-1.txt
+++ b/core/src/test/resources/org/apache/struts2/views/jsp/ui/Reset-1.txt
@@ -1,3 +1,3 @@
 <tr>
-    <td colspan="2"><div class="formButton"><input type="reset" name="myname" 
value="bar"/></div></td>
+    <td colspan="2"><div class="formButton"><input type="reset" name="myname" 
value="bar" tabindex="1"/></div></td>
 </tr>

http://git-wip-us.apache.org/repos/asf/struts/blob/533b236f/core/src/test/resources/org/apache/struts2/views/jsp/ui/Reset-3.txt
----------------------------------------------------------------------
diff --git 
a/core/src/test/resources/org/apache/struts2/views/jsp/ui/Reset-3.txt 
b/core/src/test/resources/org/apache/struts2/views/jsp/ui/Reset-3.txt
index a9f63c0..f956573 100644
--- a/core/src/test/resources/org/apache/struts2/views/jsp/ui/Reset-3.txt
+++ b/core/src/test/resources/org/apache/struts2/views/jsp/ui/Reset-3.txt
@@ -1,3 +1,3 @@
 <tr>
-    <td colspan="2"><div class="formButton"><button type="reset" name="myname" 
value="bar">bar</button></div></td>
+    <td colspan="2"><div class="formButton"><button type="reset" name="myname" 
value="bar" tabindex="1">bar</button></div></td>
 </tr>

Reply via email to