From b900fead753504246f49596c684ee0a03e8df112 Mon Sep 17 00:00:00 2001
From: ebb-tide <denizk@gmail.com>
Date: Mon, 2 Jul 2018 17:45:44 -0700
Subject: [PATCH 1/9] Add map as a template variable value type to the backend

---
 bolt/internal/internal.proto | 2 +-
 chronograf.go                | 4 ++--
 server/swagger.json          | 2 +-
 server/templates.go          | 2 +-
 server/templates_test.go     | 4 ++--
 5 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/bolt/internal/internal.proto b/bolt/internal/internal.proto
index 27dec2513c..f11b3bc833 100644
--- a/bolt/internal/internal.proto
+++ b/bolt/internal/internal.proto
@@ -97,7 +97,7 @@ message Template {
 }
 
 message TemplateValue {
-	string type             = 1; // Type can be tagKey, tagValue, fieldKey, csv, measurement, database, constant
+	string type             = 1; // Type can be tagKey, tagValue, fieldKey, csv, map, measurement, database, constant
 	string value            = 2; // Value is the specific value used to replace a template in an InfluxQL query
 	bool selected           = 3; // Selected states that this variable has been picked to use for replacement
 	string key              = 4; // Key is the key for a specific Value if the Template Type is map (optional)
diff --git a/chronograf.go b/chronograf.go
index 8a1c54f47d..3f83f8f950 100644
--- a/chronograf.go
+++ b/chronograf.go
@@ -159,7 +159,7 @@ type Range struct {
 // TemplateValue is a value use to replace a template in an InfluxQL query
 type TemplateValue struct {
 	Value    string `json:"value"`         // Value is the specific value used to replace a template in an InfluxQL query
-	Type     string `json:"type"`          // Type can be tagKey, tagValue, fieldKey, csv, measurement, database, constant, influxql
+	Type     string `json:"type"`          // Type can be tagKey, tagValue, fieldKey, csv, map, measurement, database, constant, influxql
 	Selected bool   `json:"selected"`      // Selected states that this variable has been picked to use for replacement
 	Key      string `json:"key,omitempty"` // Key is the key for the Value if the Template Type is 'map'
 }
@@ -177,7 +177,7 @@ type TemplateID string
 type Template struct {
 	TemplateVar
 	ID    TemplateID     `json:"id"`              // ID is the unique ID associated with this template
-	Type  string         `json:"type"`            // Type can be fieldKeys, tagKeys, tagValues, CSV, constant, measurements, databases, map, influxql
+	Type  string         `json:"type"`            // Type can be fieldKeys, tagKeys, tagValues, csv, constant, measurements, databases, map, influxql
 	Label string         `json:"label"`           // Label is a user-facing description of the Template
 	Query *TemplateQuery `json:"query,omitempty"` // Query is used to generate the choices for a template
 }
diff --git a/server/swagger.json b/server/swagger.json
index bb591d4927..968ef9b2c8 100644
--- a/server/swagger.json
+++ b/server/swagger.json
@@ -4178,7 +4178,7 @@
         },
         "type": {
           "type": "string",
-          "enum": ["csv", "tagKey", "tagValue", "fieldKey", "timeStamp"],
+          "enum": ["csv", "tagKey", "tagValue", "fieldKey", "timeStamp", "map"],
           "description":
             "The type will change the format of the output value. tagKey/fieldKey are double quoted; tagValue are single quoted; csv and timeStamp are not quoted."
         },
diff --git a/server/templates.go b/server/templates.go
index 6f158ac3c3..fcb7a365e1 100644
--- a/server/templates.go
+++ b/server/templates.go
@@ -22,7 +22,7 @@ func ValidTemplateRequest(template *chronograf.Template) error {
 		switch v.Type {
 		default:
 			return fmt.Errorf("Unknown template variable type %s", v.Type)
-		case "csv", "fieldKey", "tagKey", "tagValue", "measurement", "database", "constant", "influxql":
+		case "csv", "map", "fieldKey", "tagKey", "tagValue", "measurement", "database", "constant", "influxql":
 		}
 
 		if template.Type == "map" && v.Key == "" {
diff --git a/server/templates_test.go b/server/templates_test.go
index cd775acea9..4a23fefefd 100644
--- a/server/templates_test.go
+++ b/server/templates_test.go
@@ -69,7 +69,7 @@ func TestValidTemplateRequest(t *testing.T) {
 						{
 							Key:   "key",
 							Value: "value",
-							Type:  "constant",
+							Type:  "map",
 						},
 					},
 				},
@@ -84,7 +84,7 @@ func TestValidTemplateRequest(t *testing.T) {
 					Values: []chronograf.TemplateValue{
 						{
 							Value: "value",
-							Type:  "constant",
+							Type:  "map",
 						},
 					},
 				},

From 45875c7840b3110fa7aa97d7c53d9eec0fc5cf0a Mon Sep 17 00:00:00 2001
From: ebb-tide <denizk@gmail.com>
Date: Mon, 2 Jul 2018 17:49:00 -0700
Subject: [PATCH 2/9] Add a map template variable builder

---
 ui/src/shared/copy/notifications.ts           |   6 +
 .../components/MapTemplateBuilder.tsx         | 150 ++++++++++++++++++
 .../components/TemplatePreviewListItem.tsx    |   1 +
 .../components/TemplateVariableEditor.tsx     |   2 +
 ui/src/tempVars/constants/index.ts            |  15 ++
 ui/src/types/tempVars.ts                      |   3 +
 6 files changed, 177 insertions(+)
 create mode 100644 ui/src/tempVars/components/MapTemplateBuilder.tsx

diff --git a/ui/src/shared/copy/notifications.ts b/ui/src/shared/copy/notifications.ts
index 879ed8e090..6ed110f0b1 100644
--- a/ui/src/shared/copy/notifications.ts
+++ b/ui/src/shared/copy/notifications.ts
@@ -541,6 +541,12 @@ export const notifyInvalidTimeRangeValueInURLQuery = (): Notification => ({
   message: `Invalid URL query value supplied for lower or upper time range.`,
 })
 
+export const notifyInvalidMapType = (): Notification => ({
+  ...defaultErrorNotification,
+  icon: 'cube',
+  message: `Template Variables of map type accept two comma separated values per line`,
+})
+
 export const notifyInvalidZoomedTimeRangeValueInURLQuery = (): Notification => ({
   ...defaultErrorNotification,
   icon: 'cube',
diff --git a/ui/src/tempVars/components/MapTemplateBuilder.tsx b/ui/src/tempVars/components/MapTemplateBuilder.tsx
new file mode 100644
index 0000000000..9ef8b735de
--- /dev/null
+++ b/ui/src/tempVars/components/MapTemplateBuilder.tsx
@@ -0,0 +1,150 @@
+import React, {PureComponent, ChangeEvent} from 'react'
+import {getDeep} from 'src/utils/wrappers'
+import Papa from 'papaparse'
+import _ from 'lodash'
+
+import {ErrorHandling} from 'src/shared/decorators/errors'
+
+import TemplatePreviewList from 'src/tempVars/components/TemplatePreviewList'
+import DragAndDrop from 'src/shared/components/DragAndDrop'
+import {
+  notifyCSVUploadFailed,
+  notifyInvalidMapType,
+} from 'src/shared/copy/notifications'
+
+import {TemplateBuilderProps, TemplateValueType} from 'src/types'
+import {trimAndRemoveQuotes} from 'src/tempVars/utils/parsing'
+
+interface State {
+  templateValuesString: string
+}
+
+@ErrorHandling
+class MapTemplateBuilder extends PureComponent<TemplateBuilderProps, State> {
+  public constructor(props: TemplateBuilderProps) {
+    super(props)
+    const templateValues = props.template.values.map(v => v.value)
+
+    this.state = {
+      templateValuesString: templateValues.join(', '),
+    }
+  }
+
+  public render() {
+    const {onUpdateDefaultTemplateValue, template} = this.props
+    const {templateValuesString} = this.state
+    const pluralizer = template.values.length === 1 ? '' : 's'
+
+    return (
+      <>
+        <div className="form-group col-xs-12">
+          <label>Upload a CSV File</label>
+          <DragAndDrop
+            submitText="Preview"
+            fileTypesToAccept={this.validFileExtension}
+            handleSubmit={this.handleUploadFile}
+            submitOnDrop={true}
+            compact={true}
+          />
+        </div>
+        <div className="form-group col-xs-12">
+          <label>Comma Separated Values</label>
+          <div className="temp-builder--mq-controls">
+            <textarea
+              className="form-control input-sm"
+              value={templateValuesString}
+              onChange={this.handleChange}
+              onBlur={this.handleBlur}
+            />
+          </div>
+        </div>
+        <div className="form-group col-xs-12 temp-builder--results">
+          <p className="temp-builder--validation">
+            Mapping contains <strong>{template.values.length}</strong> key-value
+            pair{pluralizer}
+          </p>
+          {template.values.length > 0 && (
+            <TemplatePreviewList
+              items={template.values}
+              onUpdateDefaultTemplateValue={onUpdateDefaultTemplateValue}
+            />
+          )}
+        </div>
+      </>
+    )
+  }
+
+  private handleUploadFile = (
+    uploadContent: string,
+    fileName: string
+  ): void => {
+    const {template, onUpdateTemplate} = this.props
+
+    const fileExtensionRegex = new RegExp(`${this.validFileExtension}$`)
+    if (!fileName.match(fileExtensionRegex)) {
+      this.props.notify(notifyCSVUploadFailed())
+      return
+    }
+
+    this.setState({templateValuesString: uploadContent})
+
+    const nextValues = this.constructValuesFromString(uploadContent)
+
+    onUpdateTemplate({...template, values: nextValues})
+  }
+
+  private handleBlur = (): void => {
+    const {template, onUpdateTemplate} = this.props
+    const {templateValuesString} = this.state
+
+    const values = this.constructValuesFromString(templateValuesString)
+
+    onUpdateTemplate({...template, values})
+  }
+
+  private get validFileExtension(): string {
+    return '.csv'
+  }
+
+  private handleChange = (e: ChangeEvent<HTMLTextAreaElement>): void => {
+    this.setState({templateValuesString: e.target.value})
+  }
+
+  private constructValuesFromString(templateValuesString) {
+    const {notify} = this.props
+    const parsedTVS = Papa.parse(templateValuesString)
+    const templateValuesData = getDeep<string[][]>(parsedTVS, 'data', [[]])
+
+    if (templateValuesData.length === 0) {
+      return
+    }
+
+    let arrayOfKeys = []
+    let values = []
+    _.forEach(templateValuesData, arr => {
+      if (arr.length === 2 || (arr.length === 3 && arr[2] === '')) {
+        const key = trimAndRemoveQuotes(arr[0])
+        const value = trimAndRemoveQuotes(arr[1])
+
+        if (!arrayOfKeys.includes(key) && key !== '') {
+          values = [
+            ...values,
+            {
+              type: TemplateValueType.Map,
+              value,
+              key,
+              selected: false,
+              localSelected: false,
+            },
+          ]
+          arrayOfKeys = [...arrayOfKeys, key]
+        }
+      } else {
+        notify(notifyInvalidMapType())
+      }
+    })
+    return values
+  }
+}
+
+export default MapTemplateBuilder
diff --git a/ui/src/tempVars/components/TemplatePreviewListItem.tsx b/ui/src/tempVars/components/TemplatePreviewListItem.tsx
index 5d766a5382..f2f1a8a49e 100644
--- a/ui/src/tempVars/components/TemplatePreviewListItem.tsx
+++ b/ui/src/tempVars/components/TemplatePreviewListItem.tsx
@@ -28,6 +28,7 @@ class TemplatePreviewListItem extends PureComponent<Props> {
           active: this.isDefault,
         })}
       >
+        "key"
         {item.value}
         {this.defaultIndicator()}
       </li>
diff --git a/ui/src/tempVars/components/TemplateVariableEditor.tsx b/ui/src/tempVars/components/TemplateVariableEditor.tsx
index 4e297ba567..5d7cfc42de 100644
--- a/ui/src/tempVars/components/TemplateVariableEditor.tsx
+++ b/ui/src/tempVars/components/TemplateVariableEditor.tsx
@@ -23,6 +23,7 @@ import {
 
 import DatabasesTemplateBuilder from 'src/tempVars/components/DatabasesTemplateBuilder'
 import CSVTemplateBuilder from 'src/tempVars/components/CSVTemplateBuilder'
+import MapTemplateBuilder from 'src/tempVars/components/MapTemplateBuilder'
 import MeasurementsTemplateBuilder from 'src/tempVars/components/MeasurementsTemplateBuilder'
 import FieldKeysTemplateBuilder from 'src/tempVars/components/FieldKeysTemplateBuilder'
 import TagKeysTemplateBuilder from 'src/tempVars/components/TagKeysTemplateBuilder'
@@ -66,6 +67,7 @@ interface State {
 const TEMPLATE_BUILDERS = {
   [TemplateType.Databases]: DatabasesTemplateBuilder,
   [TemplateType.CSV]: CSVTemplateBuilder,
+  [TemplateType.Map]: MapTemplateBuilder,
   [TemplateType.Measurements]: MeasurementsTemplateBuilder,
   [TemplateType.FieldKeys]: FieldKeysTemplateBuilder,
   [TemplateType.TagKeys]: TagKeysTemplateBuilder,
diff --git a/ui/src/tempVars/constants/index.ts b/ui/src/tempVars/constants/index.ts
index 1f67cc3a43..2a8b66ec4f 100644
--- a/ui/src/tempVars/constants/index.ts
+++ b/ui/src/tempVars/constants/index.ts
@@ -34,6 +34,10 @@ export const TEMPLATE_TYPES_LIST: TemplateTypesListItem[] = [
     text: 'CSV',
     type: TemplateType.CSV,
   },
+  {
+    text: 'Map',
+    type: TemplateType.Map,
+  },
   {
     text: 'Custom Meta Query',
     type: TemplateType.MetaQuery,
@@ -42,6 +46,7 @@ export const TEMPLATE_TYPES_LIST: TemplateTypesListItem[] = [
 
 export const TEMPLATE_VARIABLE_TYPES = {
   [TemplateType.CSV]: TemplateValueType.CSV,
+  [TemplateType.Map]: TemplateValueType.Map,
   [TemplateType.Databases]: TemplateValueType.Database,
   [TemplateType.Measurements]: TemplateValueType.Measurement,
   [TemplateType.FieldKeys]: TemplateValueType.FieldKey,
@@ -106,6 +111,16 @@ export const DEFAULT_TEMPLATES: DefaultTemplates = {
       query: {},
     }
   },
+  [TemplateType.Map]: () => {
+    return {
+      id: uuid.v4(),
+      tempVar: '',
+      values: [],
+      type: TemplateType.Map,
+      label: '',
+      query: {},
+    }
+  },
   [TemplateType.TagKeys]: () => {
     return {
       id: uuid.v4(),
diff --git a/ui/src/types/tempVars.ts b/ui/src/types/tempVars.ts
index a4ea1e1a8a..a7603f973e 100644
--- a/ui/src/types/tempVars.ts
+++ b/ui/src/types/tempVars.ts
@@ -8,6 +8,7 @@ export enum TemplateValueType {
   Measurement = 'measurement',
   TagValue = 'tagValue',
   CSV = 'csv',
+  Map = 'map',
   Points = 'points',
   Constant = 'constant',
   MetaQuery = 'influxql',
@@ -19,6 +20,7 @@ export interface TemplateValue {
   type: TemplateValueType
   selected: boolean
   localSelected: boolean
+  key?: string
 }
 
 export interface TemplateQuery {
@@ -38,6 +40,7 @@ export enum TemplateType {
   TagKeys = 'tagKeys',
   TagValues = 'tagValues',
   CSV = 'csv',
+  Map = 'map',
   Databases = 'databases',
   MetaQuery = 'influxql',
 }

From 11204b678913bd847fd86d7eb6e1b787c418ceb4 Mon Sep 17 00:00:00 2001
From: ebb-tide <denizk@gmail.com>
Date: Tue, 3 Jul 2018 10:46:26 -0700
Subject: [PATCH 3/9] Add Template value type map to replace function

---
 ui/src/tempVars/utils/replace.ts | 1 +
 1 file changed, 1 insertion(+)

diff --git a/ui/src/tempVars/utils/replace.ts b/ui/src/tempVars/utils/replace.ts
index fd18b054fc..870c277735 100644
--- a/ui/src/tempVars/utils/replace.ts
+++ b/ui/src/tempVars/utils/replace.ts
@@ -89,6 +89,7 @@ const renderTemplate = (query: string, template: Template): string => {
     case TemplateValueType.CSV:
     case TemplateValueType.Constant:
     case TemplateValueType.MetaQuery:
+    case TemplateValueType.Map:
       return replaceAll(q, tempVar, value)
     default:
       return query

From 9d0c16e0fae2ab82542665d9bfafe3afa4f745f9 Mon Sep 17 00:00:00 2001
From: ebb-tide <denizk@gmail.com>
Date: Tue, 3 Jul 2018 10:59:27 -0700
Subject: [PATCH 4/9] Display key value instead of value in Template Control
 Dropdown for map type tempvar

---
 .../tempVars/components/TemplateControlDropdown.tsx  | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/ui/src/tempVars/components/TemplateControlDropdown.tsx b/ui/src/tempVars/components/TemplateControlDropdown.tsx
index f47d3567ea..ce870083de 100644
--- a/ui/src/tempVars/components/TemplateControlDropdown.tsx
+++ b/ui/src/tempVars/components/TemplateControlDropdown.tsx
@@ -6,7 +6,7 @@ import TemplateVariableEditor from 'src/tempVars/components/TemplateVariableEdit
 import {calculateDropdownWidth} from 'src/dashboards/constants/templateControlBar'
 import Authorized, {isUserAuthorized, EDITOR_ROLE} from 'src/auth/Authorized'
 
-import {Template, Source} from 'src/types'
+import {Template, Source, TemplateValueType} from 'src/types'
 
 interface Props {
   template: Template
@@ -43,10 +43,12 @@ class TemplateControlDropdown extends PureComponent<Props, State> {
     } = this.props
     const {isEditing} = this.state
 
-    const dropdownItems = template.values.map(value => ({
-      ...value,
-      text: value.value,
-    }))
+    const dropdownItems = template.values.map(value => {
+      if (value.type === TemplateValueType.Map) {
+        return {...value, text: value.key}
+      }
+      return {...value, text: value.value}
+    })
 
     const dropdownStyle = template.values.length
       ? {minWidth: calculateDropdownWidth(template.values)}

From f90d9731483abfe75827509b227e71bc5ae70e8d Mon Sep 17 00:00:00 2001
From: ebb-tide <denizk@gmail.com>
Date: Tue, 3 Jul 2018 10:59:59 -0700
Subject: [PATCH 5/9] Remove requirement for editor role to enable Template
 Control dropdown

---
 .../tempVars/components/TemplateControlDropdown.tsx  | 12 ++----------
 1 file changed, 2 insertions(+), 10 deletions(-)

diff --git a/ui/src/tempVars/components/TemplateControlDropdown.tsx b/ui/src/tempVars/components/TemplateControlDropdown.tsx
index ce870083de..a842e3b9ab 100644
--- a/ui/src/tempVars/components/TemplateControlDropdown.tsx
+++ b/ui/src/tempVars/components/TemplateControlDropdown.tsx
@@ -4,7 +4,7 @@ import Dropdown from 'src/shared/components/Dropdown'
 import OverlayTechnology from 'src/reusable_ui/components/overlays/OverlayTechnology'
 import TemplateVariableEditor from 'src/tempVars/components/TemplateVariableEditor'
 import {calculateDropdownWidth} from 'src/dashboards/constants/templateControlBar'
-import Authorized, {isUserAuthorized, EDITOR_ROLE} from 'src/auth/Authorized'
+import Authorized, {EDITOR_ROLE} from 'src/auth/Authorized'
 
 import {Template, Source, TemplateValueType} from 'src/types'
 
@@ -33,14 +33,7 @@ class TemplateControlDropdown extends PureComponent<Props, State> {
   }
 
   public render() {
-    const {
-      template,
-      isUsingAuth,
-      meRole,
-      source,
-      onPickTemplate,
-      onCreateTemplate,
-    } = this.props
+    const {template, source, onPickTemplate, onCreateTemplate} = this.props
     const {isEditing} = this.state
 
     const dropdownItems = template.values.map(value => {
@@ -65,7 +58,6 @@ class TemplateControlDropdown extends PureComponent<Props, State> {
           menuClass="dropdown-astronaut"
           useAutoComplete={true}
           selected={localSelectedItem.text}
-          disabled={isUsingAuth && !isUserAuthorized(meRole, EDITOR_ROLE)}
           onChoose={onPickTemplate(template.id)}
         />
         <Authorized requiredRole={EDITOR_ROLE}>

From 3c554095b713284b3f0ce56bfefd718a845ebc94 Mon Sep 17 00:00:00 2001
From: ebb-tide <denizk@gmail.com>
Date: Tue, 3 Jul 2018 13:28:08 -0700
Subject: [PATCH 6/9] Show key value in previews for map type template vals

---
 .../tempVars/components/TemplatePreviewListItem.tsx   | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/ui/src/tempVars/components/TemplatePreviewListItem.tsx b/ui/src/tempVars/components/TemplatePreviewListItem.tsx
index f2f1a8a49e..4060f36392 100644
--- a/ui/src/tempVars/components/TemplatePreviewListItem.tsx
+++ b/ui/src/tempVars/components/TemplatePreviewListItem.tsx
@@ -2,7 +2,7 @@ import React, {PureComponent} from 'react'
 import classNames from 'classnames'
 import {TEMPLATE_PREVIEW_LIST_DIMENSIONS as DIMENSIONS} from 'src/tempVars/constants'
 
-import {TemplateValue} from 'src/types'
+import {TemplateValue, TemplateValueType} from 'src/types'
 
 const {LI_HEIGHT, LI_MARGIN_BOTTOM} = DIMENSIONS
 
@@ -28,7 +28,7 @@ class TemplatePreviewListItem extends PureComponent<Props> {
           active: this.isDefault,
         })}
       >
-        "key"
+        {this.mapTempVarKey}
         {item.value}
         {this.defaultIndicator()}
       </li>
@@ -39,6 +39,13 @@ class TemplatePreviewListItem extends PureComponent<Props> {
     return this.props.item.selected
   }
 
+  private get mapTempVarKey(): string {
+    const {item} = this.props
+    if (item.type === TemplateValueType.Map) {
+      return `${item.key}     -->`
+    }
+  }
+
   private defaultIndicator(): JSX.Element {
     if (this.isDefault) {
       return <div className="temp-builder--default">DEFAULT</div>

From f6b4e1cf48827f2586fd236e6f807449511132b7 Mon Sep 17 00:00:00 2001
From: ebb-tide <denizk@gmail.com>
Date: Tue, 3 Jul 2018 13:57:01 -0700
Subject: [PATCH 7/9] Show key value in builder textArea for map type template
 vals

---
 ui/src/tempVars/components/MapTemplateBuilder.tsx | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/ui/src/tempVars/components/MapTemplateBuilder.tsx b/ui/src/tempVars/components/MapTemplateBuilder.tsx
index 9ef8b735de..cb4b1fa425 100644
--- a/ui/src/tempVars/components/MapTemplateBuilder.tsx
+++ b/ui/src/tempVars/components/MapTemplateBuilder.tsx
@@ -24,9 +24,13 @@ class MapTemplateBuilder extends PureComponent<TemplateBuilderProps, State> {
   public constructor(props: TemplateBuilderProps) {
     super(props)
     const templateValues = props.template.values.map(v => v.value)
+    const templateKeys = props.template.values.map(v => v.key)
+    const templateValuesString = templateKeys
+      .map((v, i) => `${v}, ${templateValues[i]}`)
+      .join('\n')
 
     this.state = {
-      templateValuesString: templateValues.join(', '),
+      templateValuesString,
     }
   }
 

From 32e4d0bf8dedc6ad9a0b413498bcc1988f6bcca7 Mon Sep 17 00:00:00 2001
From: ebb-tide <denizk@gmail.com>
Date: Tue, 3 Jul 2018 14:07:18 -0700
Subject: [PATCH 8/9] Remove offset if displaying an unscrollable number of
 items in templates previews

---
 ui/src/tempVars/components/TemplatePreviewList.tsx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/ui/src/tempVars/components/TemplatePreviewList.tsx b/ui/src/tempVars/components/TemplatePreviewList.tsx
index 90c8b752cc..52b5352c6d 100644
--- a/ui/src/tempVars/components/TemplatePreviewList.tsx
+++ b/ui/src/tempVars/components/TemplatePreviewList.tsx
@@ -42,8 +42,8 @@ class TemplatePreviewList extends PureComponent<Props> {
   private get resultsListHeight() {
     const {items} = this.props
     const count = Math.min(items.length, RESULTS_TO_DISPLAY)
-
-    return count * (LI_HEIGHT + LI_MARGIN_BOTTOM) - OFFSET
+    const scrollOffset = count > RESULTS_TO_DISPLAY ? OFFSET : 0
+    return count * (LI_HEIGHT + LI_MARGIN_BOTTOM) - scrollOffset
   }
 }
 

From b7bae77d009ee3ea65edbe6b29cdf0d7e295c45d Mon Sep 17 00:00:00 2001
From: ebb-tide <denizk@gmail.com>
Date: Tue, 3 Jul 2018 14:17:18 -0700
Subject: [PATCH 9/9] Retain unselected values when saving map type template
 variables

---
 ui/src/dashboards/actions/index.ts | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/ui/src/dashboards/actions/index.ts b/ui/src/dashboards/actions/index.ts
index 49fa24e976..4f8e32ae41 100644
--- a/ui/src/dashboards/actions/index.ts
+++ b/ui/src/dashboards/actions/index.ts
@@ -341,7 +341,10 @@ const removeUnselectedTemplateValues = (
     'templates',
     []
   ).map(template => {
-    if (template.type === TempVarsModels.TemplateType.CSV) {
+    if (
+      template.type === TempVarsModels.TemplateType.CSV ||
+      template.type === TempVarsModels.TemplateType.Map
+    ) {
       return template
     }