Merge pull request #115 from Carifio24/init-border-color

Initialize border color
glue-viz · Jan 8, 2025 · 98c806f · 98c806f
2 parents bf85be6 + 32fd058
commit 98c806f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/glue_plotly/viewers/scatter/layer_state_widget.vue b/glue_plotly/viewers/scatter/layer_state_widget.vue
@@ -57,6 +57,11 @@
                 </div>
             </template>
             <template v-else>
+               <div>
+                    <v-subheader class="pl-0 slider-label">size scaling</v-subheader>
+                    <glue-throttled-slider wait="300" min="0.1" max="10" step="0.01" :value.sync="glue_state.size_scaling"
+                        hide-details />
+                </div>
                 <div>
                       <v-subheader class="pl-0 slider-label">fill markers</v-subheader>
                       <v-switch v-model="glue_state.fill" hide-details style="margin-top: 0" />
@@ -92,11 +97,6 @@
                         </div>
                     </v-menu>
                 </div>
-                <div>
-                    <v-subheader class="pl-0 slider-label">size scaling</v-subheader>
-                    <glue-throttled-slider wait="300" min="0.1" max="10" step="0.01" :value.sync="glue_state.size_scaling"
-                        hide-details />
-                </div>
             </template>
         </template>
         <div class="text-subtitle-2 font-weight-bold" :style="glue_state.markers_visible ? {} : {marginTop: '6px'}">Line</div>

diff --git a/glue_plotly/viewers/scatter/state.py b/glue_plotly/viewers/scatter/state.py
@@ -5,6 +5,6 @@ class PlotlyScatterLayerState(ScatterLayerState):
 
     border_visible = DDCProperty(False, docstring="Whether to show borders on the markers")
     border_size = DDCProperty(1, docstring="The size of the marker borders")
-    border_color = DDCProperty(docstring="The color of the marker borders")
+    border_color = DDCProperty("#000000", docstring="The color of the marker borders")
     border_color_match_layer = DDCProperty(False, docstring="If true, border color options are ignored, "
                                                             "and the border matches the layer")