From b575ee8500f9875815180bbfb2d35ec4e46348f0 Mon Sep 17 00:00:00 2001
From: Bryan Housel <bryan@mapbox.com>
Date: Fri, 11 Jan 2019 13:56:16 -0500
Subject: [PATCH] Move formFields `klass` from an argument to a state variable
 (maybe needed in case render can get called different ways)

---
 modules/ui/form_fields.js   | 32 +++++++++++++++++++-------------
 modules/ui/preset_editor.js |  5 +++--
 2 files changed, 22 insertions(+), 15 deletions(-)

diff --git a/modules/ui/form_fields.js b/modules/ui/form_fields.js
index 08a7e2afbe..10d16559f1 100644
--- a/modules/ui/form_fields.js
+++ b/modules/ui/form_fields.js
@@ -7,25 +7,25 @@ import { utilGetSetValue, utilNoAuto } from '../util';
 
 export function uiFormFields(context) {
     var moreCombo = uiCombobox(context, 'more-fields').minItems(1);
+    var _selection = d3_select(null);
+    var _fieldsArr = [];
     var _state = '';
-    var _fieldsArr;
+    var _klass = '';
 
 
-    function formFields(selection, klass) {
-        render(selection, klass);
+    function formFields(selection) {
+        _selection = selection
+            .call(render);
     }
 
+    formFields.tagsChanged = function() {
+        _selection
+            .call(render);
+    };
 
-    formFields.tagsChanged = function() {};
-
-    function render(selection, klass) {
-
-        formFields.tagsChanged = function() {
-            render(selection, klass);
-        };
 
+    function render(selection) {
         var allowedFields = _fieldsArr.filter(function(field) { return field.isAllowed(); });
-
         var shown = allowedFields.filter(function(field) { return field.isShown(); });
         var notShown = allowedFields.filter(function(field) { return !field.isShown(); });
 
@@ -34,7 +34,7 @@ export function uiFormFields(context) {
 
         container = container.enter()
             .append('div')
-            .attr('class', 'form-fields-container ' + (klass || ''))
+            .attr('class', 'form-fields-container ' + (_klass || ''))
             .merge(container);
 
 
@@ -122,7 +122,7 @@ export function uiFormFields(context) {
 
     formFields.fieldsArr = function(val) {
         if (!arguments.length) return _fieldsArr;
-        _fieldsArr = val;
+        _fieldsArr = val || [];
         return formFields;
     };
 
@@ -132,6 +132,12 @@ export function uiFormFields(context) {
         return formFields;
     };
 
+    formFields.klass = function(val) {
+        if (!arguments.length) return _klass;
+        _klass = val;
+        return formFields;
+    };
+
 
     return formFields;
 }
diff --git a/modules/ui/preset_editor.js b/modules/ui/preset_editor.js
index 0c3d46ce62..75ea7e61ad 100644
--- a/modules/ui/preset_editor.js
+++ b/modules/ui/preset_editor.js
@@ -87,8 +87,9 @@ export function uiPresetEditor(context) {
         selection
             .call(formFields
                 .fieldsArr(_fieldsArr)
-                .state(_state),
-            'inspector-inner fillL3');
+                .state(_state)
+                .klass('inspector-inner fillL3')
+            );
 
 
         selection.selectAll('.wrap-form-field input')