Open Bug 1419783 Opened 7 years ago Updated 2 years ago

Add an option in the prefs panel to turn on off auto pretty printing

Categories

(DevTools :: Debugger, enhancement, P5)

57 Branch
enhancement

Tracking

(Not tracked)

People

(Reporter: malikanshul29, Unassigned)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:57.0) Gecko/20100101 Firefox/57.0
Build ID: 20171112125346




Expected results:

Lets add an option to allow a user switch the feature on or off if they like.
Attached patch Patch (obsolete) — Splinter Review
Comment on attachment 8930969 [details] [diff] [review]
Patch

># HG changeset patch
># User Anshul <malikanshul29@gmail.com>
># Date 1511365712 -19800
>#      Wed Nov 22 21:18:32 2017 +0530
># Node ID c8d7421f284dab67c498b3277db7402a89e38d83
># Parent  3a0aac55195fd50ca4b1b41be450bfbaaafc191c
>Added an option in prefs panel for auto prettyprint
>
>diff --git a/devtools/client/framework/toolbox-options.js b/devtools/client/framework/toolbox-options.js
>--- a/devtools/client/framework/toolbox-options.js
>+++ b/devtools/client/framework/toolbox-options.js
>@@ -328,6 +328,11 @@ OptionsPanel.prototype = {
>       label: L10N.getStr("toolbox.options.enableNewDebugger.label"),
>       id: "devtools-new-debugger",
>       parentId: "debugger-options"
>+    }, {
>+      pref: "devtools.debugger.auto-pretty-print",
>+      label: L10N.getStr("toolbox.options.autoPrettyPrint.label"),
>+      id: "devtools-debugger-auto-pretty-print",
>+      parentId: "debugger-options"
>     }];
> 
>     let createPreferenceOption = ({pref, label, id}) => {
>diff --git a/devtools/client/locales/en-US/toolbox.properties b/devtools/client/locales/en-US/toolbox.properties
>--- a/devtools/client/locales/en-US/toolbox.properties
>+++ b/devtools/client/locales/en-US/toolbox.properties
>@@ -204,3 +204,7 @@ toolbox.options.enableNewDebugger.label=
> # LOCALIZATION NOTE (toolbox.options.enableNewConsole.label): Label of the options panel
> # checkbox to enable the new console frontend. Displayed only in Nightly and local builds.
> toolbox.options.enableNewConsole.label=Enable new console frontend
>+
>+# LOCALIZATION NOTE (toolbox.options.autoPrettyPrint.label): Label of the options panel
>+# checkbox to enable auto prettyprint. Displayed only in Nightly and local builds.
>+toolbox.options.autoPrettyPrint.label=Auto prettyprint
Attached patch Latest patchSplinter Review
Attachment #8930969 - Attachment is obsolete: true
Component: Untriaged → Developer Tools
Thanks Anshul!
Severity: normal → enhancement
Status: UNCONFIRMED → NEW
Component: Developer Tools → Developer Tools: Debugger
Ever confirmed: true
Priority: -- → P3
Attachment #8930972 - Flags: review?(jlaster)
Product: Firefox → DevTools
Blocks: dbg-frontend
Priority: P3 → P5
Attachment #8930972 - Flags: review?(jlaster)
Blocks: 1565711
Blocks: 1565713
No longer blocks: 1565711
No longer blocks: 1565713
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: