Fixed tab index handling in slider (#10315)

Change-Id: I84611b5cc0a4f8520d1013562ba650f1f92663c4

client/src/com/vaadin/client/ui/slider/SliderConnector.java

@@ -71,6 +71,7 @@ public class SliderConnector extends AbstractFieldConnector implements
         getWidget().setFeedbackValue(getState().value);
 
         getWidget().buildBase();
+        getWidget().setTabIndex(getState().tabIndex);
     }
 
 }

uitest/src/com/vaadin/tests/fields/TabIndexes.html

@@ -96,6 +96,11 @@
 	<td>vaadin=runcomvaadintestsfieldsTabIndexes::PID_Sfield-16/domChild[0]@tabIndex</td>
 	<td>0</td>
 </tr>
+<tr>
+	<td>assertAttribute</td>
+	<td>vaadin=runcomvaadintestsfieldsTabIndexes::PID_Sfield-17@tabIndex</td>
+	<td>0</td>
+</tr>
 <tr>
 	<td>click</td>
 	<td>vaadin=runcomvaadintestsfieldsTabIndexes::/VVerticalLayout[0]/VOrderedLayout$Slot[1]/VVerticalLayout[0]/VOrderedLayout$Slot[1]/VHorizontalLayout[0]/VOrderedLayout$Slot[1]/VButton[0]/domChild[0]/domChild[0]</td>
@@ -181,6 +186,11 @@
 	<td>vaadin=runcomvaadintestsfieldsTabIndexes::PID_Sfield-16/domChild[0]@tabIndex</td>
 	<td>1</td>
 </tr>
+<tr>
+	<td>assertAttribute</td>
+	<td>vaadin=runcomvaadintestsfieldsTabIndexes::PID_Sfield-17@tabIndex</td>
+	<td>1</td>
+</tr>
 <tr>
 	<td>click</td>
 	<td>vaadin=runcomvaadintestsfieldsTabIndexes::/VVerticalLayout[0]/VOrderedLayout$Slot[1]/VVerticalLayout[0]/VOrderedLayout$Slot[1]/VHorizontalLayout[0]/VOrderedLayout$Slot[2]/VButton[0]/domChild[0]/domChild[0]</td>
@@ -266,6 +276,11 @@
 	<td>vaadin=runcomvaadintestsfieldsTabIndexes::PID_Sfield-16/domChild[0]@tabIndex</td>
 	<td>16</td>
 </tr>
+<tr>
+	<td>assertAttribute</td>
+	<td>vaadin=runcomvaadintestsfieldsTabIndexes::PID_Sfield-17@tabIndex</td>
+	<td>17</td>
+</tr>
 <tr>
 	<td>click</td>
 	<td>vaadin=runcomvaadintestsfieldsTabIndexes::/VVerticalLayout[0]/VOrderedLayout$Slot[1]/VVerticalLayout[0]/VOrderedLayout$Slot[1]/VHorizontalLayout[0]/VOrderedLayout$Slot[3]/VButton[0]/domChild[0]/domChild[0]</td>
@@ -274,81 +289,86 @@
 <tr>
 	<td>assertAttribute</td>
 	<td>vaadin=runcomvaadintestsfieldsTabIndexes::PID_Sfield-1/domChild[1]/domChild[1]@tabIndex</td>
-	<td>16</td>
+	<td>17</td>
 </tr>
 <tr>
 	<td>assertAttribute</td>
 	<td>vaadin=runcomvaadintestsfieldsTabIndexes::PID_Sfield-2/domChild[0]@tabIndex</td>
-	<td>15</td>
+	<td>16</td>
 </tr>
 <tr>
 	<td>assertAttribute</td>
 	<td>vaadin=runcomvaadintestsfieldsTabIndexes::PID_Sfield-3/domChild[0]@tabIndex</td>
-	<td>14</td>
+	<td>15</td>
 </tr>
 <tr>
 	<td>assertAttribute</td>
 	<td>vaadin=runcomvaadintestsfieldsTabIndexes::PID_Sfield-4/domChild[0]@tabIndex</td>
-	<td>13</td>
+	<td>14</td>
 </tr>
 <tr>
 	<td>assertAttribute</td>
 	<td>vaadin=runcomvaadintestsfieldsTabIndexes::PID_Sfield-5@tabIndex</td>
-	<td>12</td>
+	<td>13</td>
 </tr>
 <tr>
 	<td>assertAttribute</td>
 	<td>vaadin=runcomvaadintestsfieldsTabIndexes::PID_Sfield-6/domChild[0]@tabIndex</td>
-	<td>11</td>
+	<td>12</td>
 </tr>
 <tr>
 	<td>assertAttribute</td>
 	<td>vaadin=runcomvaadintestsfieldsTabIndexes::PID_Sfield-7/domChild[0]@tabIndex</td>
-	<td>10</td>
+	<td>11</td>
 </tr>
 <tr>
 	<td>assertAttribute</td>
 	<td>vaadin=runcomvaadintestsfieldsTabIndexes::PID_Sfield-8/domChild[0]/domChild[0]@tabIndex</td>
-	<td>9</td>
+	<td>10</td>
 </tr>
 <tr>
 	<td>assertAttribute</td>
 	<td>vaadin=runcomvaadintestsfieldsTabIndexes::PID_Sfield-9/domChild[1]/domChild[1]@tabIndex</td>
-	<td>8</td>
+	<td>9</td>
 </tr>
 <tr>
 	<td>assertAttribute</td>
 	<td>vaadin=runcomvaadintestsfieldsTabIndexes::PID_Sfield-10/domChild[1]@tabIndex</td>
-	<td>7</td>
+	<td>8</td>
 </tr>
 <tr>
 	<td>assertAttribute</td>
 	<td>vaadin=runcomvaadintestsfieldsTabIndexes::PID_Sfield-11/domChild[1]@tabIndex</td>
-	<td>6</td>
+	<td>7</td>
 </tr>
 <tr>
 	<td>assertAttribute</td>
 	<td>vaadin=runcomvaadintestsfieldsTabIndexes::PID_Sfield-12@tabIndex</td>
-	<td>5</td>
+	<td>6</td>
 </tr>
 <tr>
 	<td>assertAttribute</td>
 	<td>vaadin=runcomvaadintestsfieldsTabIndexes::PID_Sfield-13@tabIndex</td>
-	<td>4</td>
+	<td>5</td>
 </tr>
 <tr>
 	<td>assertAttribute</td>
 	<td>vaadin=runcomvaadintestsfieldsTabIndexes::PID_Sfield-14@tabIndex</td>
-	<td>3</td>
+	<td>4</td>
 </tr>
 <tr>
 	<td>assertAttribute</td>
 	<td>vaadin=runcomvaadintestsfieldsTabIndexes::PID_Sfield-15/domChild[1]@tabIndex</td>
-	<td>2</td>
+	<td>3</td>
 </tr>
 <tr>
 	<td>assertAttribute</td>
 	<td>vaadin=runcomvaadintestsfieldsTabIndexes::PID_Sfield-16/domChild[0]@tabIndex</td>
+	<td>2</td>
+</tr>
+<tr>
+	<td>assertAttribute</td>
+	<td>vaadin=runcomvaadintestsfieldsTabIndexes::PID_Sfield-17@tabIndex</td>
 	<td>1</td>
 </tr>
 </tbody></table>

uitest/src/com/vaadin/tests/fields/TabIndexes.java

@@ -22,6 +22,7 @@ import com.vaadin.ui.OptionGroup;
 import com.vaadin.ui.PasswordField;
 import com.vaadin.ui.PopupDateField;
 import com.vaadin.ui.RichTextArea;
+import com.vaadin.ui.Slider;
 import com.vaadin.ui.Table;
 import com.vaadin.ui.TextArea;
 import com.vaadin.ui.TextField;
@@ -121,7 +122,7 @@ public class TabIndexes extends AbstractTestUI {
         fields.add(new TextArea());
         fields.add(new RichTextArea());
         fields.add(new CheckBox());
-        // fields.add(new Slider());
+        fields.add(new Slider());
 
         clearTabIndexes.click();