fix buggy transition from timeline to summary

Fixes #946

Author: starpit

app/plugins/modules/activation-visualizations/lib/util.js

@@ -398,7 +398,7 @@ exports.optionsToString = (options, except) => {
     let str = ''
     for (let key in options) {
         // underscore comes from minimist
-        if (key !== '_' && options[key] !== undefined && key !== 'name' && key !== 'theme'
+        if (key !== '_' && options[key] !== undefined && key !== 'name' && key !== 'theme' && key !== 'timeline' && key !== 't'
             && (!except || !except.find(_ => _ === key))) {
             const dash = key.length === 1 ? '-' : '--',
                   prefix = options[key] === false ? 'no-' : '', // e.g. --no-help

tests/tests/passes/05/activation-grid-view.js

@@ -286,4 +286,13 @@ describe('Activation grid visualization', function() {
     it('should open grid as timeline with grid -t', () => cli.do('grid -t', this.app)
        .then(cli.expectOK)
        .catch(common.oops(this)))
+
+    it('should open grid as timeline with grid -t, then switch to summary', () => cli.do('grid -t', this.app)
+       .then(cli.expectOK)
+       .then(() => this.app.client.click(ui.selectors.SIDECAR_MODE_BUTTON('summary')))
+       .then(() => this.app.client.waitUntil(() => {
+           return this.app.client.getText('.sidecar-header-icon')
+               .then(txt => txt.toLowerCase() === 'summary')
+       }))
+       .catch(common.oops(this)))
 })