fixed markup #2

Author: Eugene Tupikov

examples/views/multiple-input.php

@@ -46,7 +46,7 @@
 
 <h3>Multiple columns</h3>
 <?php
-echo $form->field($model, 'schedule')->widget(MultipleInput::className(), [
+echo $form->field($model, 'schedule')->label(false)->widget(MultipleInput::className(), [
     'id' => 'examplemodel-schedule',
     'max' => 4,
     'allowEmptyList' => true,
@@ -58,6 +58,7 @@
         }
         return $options;
     },
+    'cloneButton' => true,
     'columns' => [
         [
             'name'  => 'user_id',

src/assets/src/css/multiple-input.css

@@ -59,6 +59,10 @@ table.multiple-input-list tr > th {
     padding-left: 5px;
 }
 
+.multiple-input-list.list-renderer .list-cell__button:last-child {
+    padding-right: 15px;
+}
+
 .multiple-input-list.list-renderer tbody .list-cell__button .btn{
     margin-top: 25px;
 }

src/assets/src/js/jquery.multipleInput.js

@@ -151,7 +151,7 @@
                     // fetch default attribute options from active from attribute
                     if (typeof attribute === 'object') {
                         $.each(attribute, function (key, value) {
-                            if (['id', 'input', 'container'].indexOf(key) == -1) {
+                            if (['id', 'input', 'container'].indexOf(key) === -1) {
                                 defaultAttributeOptions[key] = value;
                             }
                         });
@@ -199,7 +199,7 @@
 
         remove: function (index) {
             var row = null;
-            if (index != undefined) {
+            if (index !== undefined) {
                 row = $(this).find('.js-input-remove:eq(' + index + ')');
             } else {
                 row = $(this).find('.js-input-remove').last();
@@ -239,7 +239,7 @@
             template  = settings.template,
             inputList = $wrapper.children('.multiple-input-list').first();
 
-        if (settings.max != null && getCurrentIndex($wrapper) >= settings.max) {
+        if (settings.max !== null && getCurrentIndex($wrapper) >= settings.max) {
             return;
         }
 
@@ -287,10 +287,10 @@
             if (values) {
                 var val = values[index];
 
-                if (tag == 'INPUT' || tag == 'TEXTAREA') {
+                if (tag === 'INPUT' || tag === 'TEXTAREA') {
                     obj.val(val);
-                } else if (tag == 'SELECT') {
-                    if (val && val.indexOf('option') != -1) {
+                } else if (tag === 'SELECT') {
+                    if (val && val.indexOf('option') !== -1) {
                         obj.append(val);
                     } else {
                         var option = obj.find('option[value="' + val + '"]');
@@ -362,7 +362,7 @@
 
 
         // do not add attribute which are not the part of widget
-        if (wrapper.length == 0) {
+        if (wrapper.length === 0) {
             return;
         }
 

src/assets/src/js/jquery.multipleInput.min.js

@@ -57,6 +57,11 @@ public function renderHeader()
 
         $content = [];
         $content[] = Html::tag('td', ' ');
+
+        if ($this->cloneButton) {
+            $content[] = Html::tag('td', ' ');
+        }
+
         $content[] = Html::tag('td', $button, [
             'class' => 'list-cell__button',
         ]);

src/renderers/ListRenderer.php

@@ -58,8 +58,6 @@ public function renderHeader()
             $cells[] = $this->renderHeaderCell($column);
         }
 
-
-
         if ($this->max === null || ($this->max >= 1 && $this->max !== $this->min)) {
             $button = $this->isAddButtonPositionHeader() ? $this->renderAddButton() : '';