Closed Bug 850929 Opened 8 years ago Closed 8 years ago

GCLI needs a simpler way to mark parameters as being options

Categories

(DevTools Graveyard :: Graphic Commandline and Toolbar, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 22

People

(Reporter: jwalker, Assigned: jwalker)

Details

Attachments

(1 file)

var tempSpec = {
    name: 'temp',
    params: [
      {
        name: 'text',
        type: 'string',
        option: true
      }
    ],
    exec: ...
  };


Is better than

  var tempSpec = {
    name: 'temp',
    params: [
      {
        group: 'Options',
        params: [
          {
            name: 'text',
            type: 'string',
          }
        ]
      }
    ],
    exec: ...
  };

option:true gets you the default group name of "Option", or you can specify a custom name with option:"Advanced"
Attached patch v1Splinter Review
Assignee: nobody → jwalker
Status: NEW → ASSIGNED
Attachment #726541 - Flags: review?(paul)
Attachment #726541 - Flags: review?(paul) → review+
https://hg.mozilla.org/mozilla-central/rev/55bea9d170b7
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 22
Product: Firefox → DevTools
Product: DevTools → DevTools Graveyard
You need to log in before you can comment on or make changes to this bug.