search

Chakroun-Anas / turbo-console-log

593 stars 142 forks source link

Configuration names should have a common prefix #50

Closed SharpSeeEr closed 5 years ago

SharpSeeEr commented 5 years ago

The four settings should have a common prefix added to avoid naming conflicts and allowing easier searching. Instead of:

{
  "wrapLogMessage": false,
  "logMessagePrefix": "TCL",
  "addSemicolonInTheEnd": false,
  "quote": false
}

Add prefix:

{
  "turboConsoleLog.wrapLogMessage": false,
  "turboConsoleLog.logMessagePrefix": "TCL",
  "turboConsoleLog.addSemicolonInTheEnd": false,
  "turboConsoleLog.quote": false
}

Or optionally displayed as:

{
  "turboConsoleLog": {
    "wrapLogMessage": false,
    "logMessagePrefix": "TCL",
    "addSemicolonInTheEnd": false,
    "quote": false
  }
}

This makes finding and editing the appropriate settings much easier by simply searching for the name of the extension. Pull request incoming.

Chakroun-Anas commented 5 years ago

Hello,

Your PR is merged to master, thank you so much for helping me making this extension better.

Regards.