Bring back roles with value object

src/components/colorbar/attributes.js

@@ -18,6 +18,7 @@ module.exports = overrideAll({
 // TODO: only right is supported currently
 //     orient: {
 //         valType: 'enumerated',
+//         role: 'info',
 //         values: ['left', 'right', 'top', 'bottom'],
 //         dflt: 'right',
 //         description: [

src/plot_api/plot_schema.js

@@ -63,7 +63,7 @@ exports.get = function() {
     return {
         defs: {
             valObjects: valObjectMeta,
-            metaKeys: UNDERSCORE_ATTRS.concat(['description', 'editType', 'impliedEdits']),
+            metaKeys: UNDERSCORE_ATTRS.concat(['description', 'role', 'editType', 'impliedEdits']),
             editType: {
                 traces: editTypes.traces,
                 layout: editTypes.layout
@@ -600,14 +600,14 @@ function getFramesAttributes() {
 }
 
 function formatAttributes(attrs) {
-    mergeValType(attrs);
+    mergeValTypeAndRole(attrs);
     formatArrayContainers(attrs);
     stringify(attrs);
 
     return attrs;
 }
 
-function mergeValType(attrs) {
+function mergeValTypeAndRole(attrs) {
     function makeSrcAttr(attrName) {
         return {
             valType: 'string',
@@ -622,12 +622,17 @@ function mergeValType(attrs) {
     function callback(attr, attrName, attrs) {
         if(exports.isValObject(attr)) {
             if(attr.valType === 'data_array') {
+                // all 'data_array' attrs have role 'data'
+                attr.role = 'data';
                 // all 'data_array' attrs have a corresponding 'src' attr
                 attrs[attrName + 'src'] = makeSrcAttr(attrName);
             } else if(attr.arrayOk === true) {
                 // all 'arrayOk' attrs have a corresponding 'src' attr
                 attrs[attrName + 'src'] = makeSrcAttr(attrName);
             }
+        } else if(isPlainObject(attr)) {
+            // all attrs container objects get role 'object'
+            attr.role = 'object';
         }
     }
 
@@ -646,6 +651,7 @@ function formatArrayContainers(attrs) {
 
         attrs[attrName] = { items: {} };
         attrs[attrName].items[itemName] = attr;
+        attrs[attrName].role = 'object';
     }
 
     exports.crawl(attrs, callback);

src/plots/frame_attributes.js

@@ -39,13 +39,15 @@ module.exports = {
     },
     data: {
         valType: 'any',
+        role: 'object',
         description: [
             'A list of traces this frame modifies. The format is identical to the',
             'normal trace definition.'
         ].join(' ')
     },
     layout: {
         valType: 'any',
+        role: 'object',
         description: [
             'Layout properties which this frame modifies. The format is identical',
             'to the normal layout definition.'

src/traces/barpolar/attributes.js

@@ -24,6 +24,7 @@ module.exports = {
 
     // orientation: {
     //     valType: 'enumerated',
+    //     role: 'info',
     //     values: ['radial', 'angular'],
     //     editType: 'calc+clearAxisTypes',
     //     description: 'Sets the orientation of the bars.'

src/traces/streamtube/attributes.js

@@ -95,6 +95,7 @@ var attrs = {
     //     valType: 'enumerated',
     //     values: ['scaled', 'absolute', 'fixed'],
     //     dflt: 'scaled',
+    //     role: 'info',
     //     editType: 'calc',
     //     description: [
     //         'Sets the mode by which the streamtubes are sized.'

test/jasmine/bundle_tests/plotschema_test.js

@@ -25,6 +25,7 @@ describe('plot schema', function() {
     var isPlainObject = Lib.isPlainObject;
 
     var VALTYPES = Object.keys(valObjects);
+    var ROLES = ['info', 'style', 'data'];
     var editType = plotSchema.defs.editType;
 
     function assertTraceSchema(callback) {
@@ -72,6 +73,26 @@ describe('plot schema', function() {
         );
     });
 
+    it('all attributes should only have valid `role`', function() {
+        assertPlotSchema(
+            function(attr) {
+                if(isValObject(attr)) {
+                    expect(ROLES.indexOf(attr.role) !== -1).toBe(true, attr);
+                }
+            }
+        );
+    });
+
+    it('all nested objects should have the *object* `role`', function() {
+        assertPlotSchema(
+            function(attr, attrName) {
+                if(!isValObject(attr) && isPlainObject(attr) && attrName !== 'items') {
+                    expect(attr.role === 'object').toBe(true);
+                }
+            }
+        );
+    });
+
     it('all attributes should have the required options', function() {
         assertPlotSchema(
             function(attr) {
@@ -94,7 +115,7 @@ describe('plot schema', function() {
                     var opts = valObject.requiredOpts
                         .concat(valObject.otherOpts)
                         .concat([
-                            'valType', 'description',
+                            'valType', 'description', 'role',
                             'editType', 'impliedEdits', 'anim',
                             '_compareAsJSON', '_noTemplating'
                         ]);
@@ -164,8 +185,13 @@ describe('plot schema', function() {
 
             // N.B. the specs below must be satisfied for plotly.py
             expect(isPlainObject(itemsObj)).toBe(true);
+            expect(itemsObj.role).toBeUndefined();
             expect(Object.keys(itemsObj).length).toEqual(1);
             expect(isPlainObject(itemObj)).toBe(true);
+            expect(itemObj.role).toBe('object');
+
+            var role = np.get().role;
+            expect(role).toEqual('object');
         });
     });
 
@@ -197,7 +223,7 @@ describe('plot schema', function() {
         );
     });
 
-    it('deprecated attributes should have a `valType`', function() {
+    it('deprecated attributes should have a `valType` and `role`', function() {
         var DEPRECATED = '_deprecated';
 
         assertPlotSchema(
@@ -208,6 +234,8 @@ describe('plot schema', function() {
 
                         expect(VALTYPES.indexOf(dAttr.valType) !== -1)
                             .toBe(true, attrString + ': ' + dAttrName);
+                        expect(ROLES.indexOf(dAttr.role) !== -1)
+                            .toBe(true, attrString + ': ' + dAttrName);
                     });
                 }
             }
@@ -289,13 +317,15 @@ describe('plot schema', function() {
         expect(plotSchema.defs.metaKeys)
             .toEqual([
                 '_isSubplotObj', '_isLinkedToArray', '_arrayAttrRegexps',
-                '_deprecated', 'description', 'editType', 'impliedEdits'
+                '_deprecated', 'description', 'role', 'editType', 'impliedEdits'
             ]);
     });
 
     it('should list the correct frame attributes', function() {
         expect(plotSchema.frames).toBeDefined();
+        expect(plotSchema.frames.role).toEqual('object');
         expect(plotSchema.frames.items.frames_entry).toBeDefined();
+        expect(plotSchema.frames.items.frames_entry.role).toEqual('object');
     });
 
     it('should list config attributes', function() {
@@ -439,7 +469,7 @@ describe('getTraceValObject', function() {
         // it still returns the attribute itself - but maybe we should only do this
         // for valType: any? (or data_array/arrayOk with just an index)
         [
-            'valType', 'dflt', 'description', 'arrayOk',
+            'valType', 'dflt', 'role', 'description', 'arrayOk',
             'editType', 'min', 'max', 'values'
         ].forEach(function(prop) {
             expect(getTraceValObject({}, ['x', prop]))

test/jasmine/tests/plot_api_test.js

@@ -2877,7 +2877,8 @@ describe('plot_api edit_types', function() {
 
         editTypes.update(flags, {
             valType: 'boolean',
-            dflt: true
+            dflt: true,
+            role: 'style'
         });
 
         expect(flags).toEqual({calc: false, style: true});
@@ -2897,7 +2898,8 @@ describe('plot_api edit_types', function() {
         editTypes.update(flags, {
             editType: 'calc+style',
             valType: 'number',
-            dflt: 1
+            dflt: 1,
+            role: 'style'
         });
 
         expect(flags).toEqual({calc: true, legend: true, style: true});