From 7108756ea193b442760a40959d71a3a1f51f7f3f Mon Sep 17 00:00:00 2001
From: Kyle Conroy <kyleconroy@gmail.com>
Date: Mon, 18 Oct 2021 15:53:04 -0400
Subject: [PATCH 1/3] vertical divider and padding between toolbar items

---
 jdaviz/app.vue | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/jdaviz/app.vue b/jdaviz/app.vue
index 3470c6982c..5ebc9bcdd9 100644
--- a/jdaviz/app.vue
+++ b/jdaviz/app.vue
@@ -2,6 +2,8 @@
   <v-app id="web-app" class="jdaviz">
     <v-app-bar color="primary" dark :dense="state.settings.dense_toolbar" flat app absolute clipped-right>
       <j-tooltip :tipid="item.name" v-for="item in state.tool_items">
+        <v-divider v-if="['g-data-tools', 'g-subset-tools'].indexOf(item.name) === -1" vertical style="margin: 0px 10px"></v-divider>
+        <v-divider v-else-if="item.name === 'g-subset-tools'" vertical style="margin: 0px 10px; border-width: 0"></v-divider>
         <jupyter-widget :widget="item.widget" :key="item.name"></jupyter-widget>
       </j-tooltip>
       <v-toolbar-items>
@@ -250,4 +252,9 @@ a:active {
   background-color: #c75109 !important;
 }
 
+
+.v-divider.theme--dark {
+  /* make the v-divider standout more */
+  border-color: hsla(0,0%,100%,.35) !important;
+}
 </style>

From 31677d5e095a2f8f5f433e0f038b361938d4f3f0 Mon Sep 17 00:00:00 2001
From: Kyle Conroy <kyleconroy@gmail.com>
Date: Tue, 12 Oct 2021 17:41:30 -0400
Subject: [PATCH 2/3] remove duplicate divider in redshift slider

(and small tweak to vertical alignment by hiding textless hint under select UI element)
---
 jdaviz/app.vue                                                | 4 ++++
 .../specviz/plugins/redshift_slider/redshift_slider.vue       | 3 +--
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/jdaviz/app.vue b/jdaviz/app.vue
index 5ebc9bcdd9..06c8cc0d94 100644
--- a/jdaviz/app.vue
+++ b/jdaviz/app.vue
@@ -257,4 +257,8 @@ a:active {
   /* make the v-divider standout more */
   border-color: hsla(0,0%,100%,.35) !important;
 }
+
+.no-hint .v-text-field__details {
+  display: none !important;
+}
 </style>
diff --git a/jdaviz/configs/specviz/plugins/redshift_slider/redshift_slider.vue b/jdaviz/configs/specviz/plugins/redshift_slider/redshift_slider.vue
index 3446d4b180..72cf92b0f4 100644
--- a/jdaviz/configs/specviz/plugins/redshift_slider/redshift_slider.vue
+++ b/jdaviz/configs/specviz/plugins/redshift_slider/redshift_slider.vue
@@ -13,9 +13,8 @@
       :step="slider_step"
     >
     <template v-slot:prepend>
-      <v-divider vertical></v-divider>
       <v-select
-        class="pl-md-4"
+        class="pl-md-4 no-hint"
         :items="['Redshift', 'RV (km/s)']"
         v-model="slider_type"
         dense>

From 1366710b2ee34d9b047005ec91bf39734c321b15 Mon Sep 17 00:00:00 2001
From: Kyle Conroy <kyleconroy@gmail.com>
Date: Wed, 20 Oct 2021 08:54:07 -0400
Subject: [PATCH 3/3] move divider out of tooltip

---
 jdaviz/app.vue | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/jdaviz/app.vue b/jdaviz/app.vue
index 06c8cc0d94..45b491f30e 100644
--- a/jdaviz/app.vue
+++ b/jdaviz/app.vue
@@ -1,12 +1,15 @@
 <template>
   <v-app id="web-app" class="jdaviz">
     <v-app-bar color="primary" dark :dense="state.settings.dense_toolbar" flat app absolute clipped-right>
-      <j-tooltip :tipid="item.name" v-for="item in state.tool_items">
+      <v-toolbar-items v-for="item in state.tool_items">
         <v-divider v-if="['g-data-tools', 'g-subset-tools'].indexOf(item.name) === -1" vertical style="margin: 0px 10px"></v-divider>
         <v-divider v-else-if="item.name === 'g-subset-tools'" vertical style="margin: 0px 10px; border-width: 0"></v-divider>
-        <jupyter-widget :widget="item.widget" :key="item.name"></jupyter-widget>
-      </j-tooltip>
+        <j-tooltip :tipid="item.name">
+          <jupyter-widget :widget="item.widget" :key="item.name"></jupyter-widget>
+        </j-tooltip>
+      </v-toolbar-items>
       <v-toolbar-items>
+        <v-divider v-if="config === 'mosviz'" vertical style="margin: 0px 10px"></v-divider>
         <j-tooltip v-if="config === 'mosviz'" tipid="lock-row-toggle">
           <v-btn 
             icon