fixups for domain attrs

etpinard · etpinard · commit 4d5a1bda72d2 · 2018-01-10T15:06:18.000-05:00
diff --git a/src/plots/domain_attributes.js b/src/plots/domain_attributes.js
@@ -44,27 +44,28 @@ module.exports = function(opts, extra) {
         dflt: [0, 1]
     };
 
-    var namePart = opts.name || '';
-    var contPart = opts.trace ? 'trace' : 'subplot';
+    var namePart = opts.name ? opts.name + ' ' : '';
+    var contPart = opts.trace ? 'trace ' : 'subplot ';
+    var descPart = extra.description ? ' ' + extra.description : '';
 
     return {
         x: extendFlat({}, base, {
             description: [
-                'Sets the horizontal domain of this',
+                'Sets the horizontal domain of this ',
                 namePart,
                 contPart,
                 '(in plot fraction).',
-                extra.description
-            ].join(' ')
+                descPart
+            ].join('')
         }),
         y: extendFlat({}, base, {
             description: [
-                'Sets the vertical domain of this',
+                'Sets the vertical domain of this ',
                 namePart,
                 contPart,
                 '(in plot fraction).',
-                extra.description
-            ].join(' ')
+                descPart
+            ].join('')
         }),
         editType: opts.editType
     };
diff --git a/src/plots/geo/layout/layout_attributes.js b/src/plots/geo/layout/layout_attributes.js
@@ -70,7 +70,7 @@ module.exports = overrideAll({
         description: [
             'Note that geo subplots are constrained by domain.',
             'In general, when `projection.scale` is set to 1.',
-            'a map will fit either its x or y domain, but not both. '
+            'a map will fit either its x or y domain, but not both.'
         ].join(' ')
     }),
 
