The default bug view has changed. See this FAQ.

Allow the source code editor of Scratchpad & StyleEditor to be themed

RESOLVED FIXED in Firefox 13

Status

()

Firefox
Developer Tools
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Alfred Kayser, Assigned: VD)

Tracking

({dev-doc-needed})

Trunk
Firefox 13
dev-doc-needed
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [sourceeditor][orion][good first bug][mentor=msucan])

Attachments

(2 attachments, 6 obsolete attachments)

(Reporter)

Description

5 years ago
The source editor (orion) currently only includes 'chrome://browser/content/orion-mozilla.css'. This means that third party themes, such as Walnut and Nautipolis cannot style the style editor nor scratchpad.

Please add a way to include a 'chrome://browser/skin/orion-mozilla.css'.
Possibly with the default theme with an import to the content version.
@import "chrome://browser/content/orion-mozilla.css"
Sounds like a good idea! Thanks Alfred!
Whiteboard: [sourceeditor][orion][good first bug][mentor=msucan]
(Assignee)

Comment 2

5 years ago
HI! I would like to work on this bug! But this is my first bug! Could you guide me on how to proceed? I have pulled the source code from the repository!
(In reply to VD from comment #2)
> HI! I would like to work on this bug! But this is my first bug! Could you
> guide me on how to proceed? I have pulled the source code from the
> repository!

Hello VD! Thanks for your interest to help us with this bug! I am glad to help you make the Source Editor themeable.

We have taken the discussion to IRC and VD is already working on a patch! Looking forward to his patch!
Assignee: nobody → vandhanaa91
Status: NEW → ASSIGNED
Version: unspecified → Trunk
(Assignee)

Comment 4

5 years ago
Created attachment 592479 [details] [diff] [review]
Patch for Bug 721324 - Allow the source code editor of Scratchpad & StyleEditor to be themed
(Assignee)

Comment 5

5 years ago
hope this helps :)
(Assignee)

Updated

5 years ago
Attachment #592479 - Flags: review?(mihai.sucan)
(Assignee)

Comment 6

5 years ago
Created attachment 592482 [details] [diff] [review]
Patch for Bug 721324 - Allow the source code editor of Scratchpad & StyleEditor to be themed
Attachment #592479 - Attachment is obsolete: true
Attachment #592479 - Flags: review?(mihai.sucan)
Attachment #592482 - Flags: review?(mihai.sucan)
Comment on attachment 592482 [details] [diff] [review]
Patch for Bug 721324 - Allow the source code editor of Scratchpad & StyleEditor to be themed

Review of attachment 592482 [details] [diff] [review]:
-----------------------------------------------------------------

Path looks good. Thank you very much for your contribution VD! Awesome!

Dão: should we keep the content stylesheet? I think we can work only with the theme CSS.

::: browser/devtools/sourceeditor/orion/mozilla.css
@@ +5,4 @@
>  }
>  
>  .view {
> + }

Nit: you add an unneeded space here.
Attachment #592482 - Flags: review?(mihai.sucan)
Attachment #592482 - Flags: review?(dao)
Attachment #592482 - Flags: review+
Comment on attachment 592482 [details] [diff] [review]
Patch for Bug 721324 - Allow the source code editor of Scratchpad & StyleEditor to be themed

>--- a/browser/devtools/sourceeditor/orion/mozilla.css	Mon Jan 23 23:06:58 2012 +0100
>+++ b/browser/devtools/sourceeditor/orion/mozilla.css	Sun Jan 29 19:54:09 2012 +0800
>@@ -1,129 +1,86 @@
> /* Any copyright is dedicated to the Public Domain.
>    http://creativecommons.org/publicdomain/zero/1.0/ */
> 
>-.viewContainer {
>-  background: #cddae5; /* This will be seen as the continuation of the ruler */
>-  font-family: monospace;
>-  font-size: inherit; /* inherit browser's default monospace font size */
>+.viewContainer {  
> }

Please remove any empty rules.

Please use 'hg copy' to move the mozilla.css contents to the new files (before modifying mozilla.css).

>--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
>+++ b/browser/themes/winstripe/devtools/orion-mozilla.css	Sun Jan 29 19:54:09 2012 +0800
>@@ -0,0 +1,127 @@
>+/* Any copyright is dedicated to the Public Domain.
>+   http://creativecommons.org/publicdomain/zero/1.0/ */

Should use the MPL here. http://www.mozilla.org/MPL/headers/

>--- a/browser/themes/winstripe/jar.mn	Mon Jan 23 23:06:58 2012 +0100
>+++ b/browser/themes/winstripe/jar.mn	Sun Jan 29 19:54:09 2012 +0800
>@@ -105,16 +105,17 @@ browser.jar:
>         skin/classic/browser/tabview/tabview.png                    (tabview/tabview.png)
>         skin/classic/browser/tabview/tabview-inverted.png           (tabview/tabview-inverted.png)
>         skin/classic/browser/tabview/tabview.css                    (tabview/tabview.css)
>         skin/classic/browser/devtools/common.css                    (devtools/common.css)
>         skin/classic/browser/devtools/arrows.png                    (devtools/arrows.png)
>         skin/classic/browser/devtools/goto-mdn.png                  (devtools/goto-mdn.png)
>         skin/classic/browser/devtools/csshtmltree.css               (devtools/csshtmltree.css)
>         skin/classic/browser/devtools/gcli.css                      (devtools/gcli.css)
>+	skin/classic/browser/devtools/orion-mozilla.css                    (devtools/orion-mozilla.css)

nit: indentation is off.

Please name the new files orion.css rather than orion-mozilla.css.
Attachment #592482 - Flags: review?(dao) → review-
(Assignee)

Comment 9

5 years ago
Created attachment 592665 [details] [diff] [review]
Patch for Bug 721324
(Assignee)

Updated

5 years ago
Attachment #592665 - Attachment description: P → Patch for Bug 721324
Attachment #592665 - Attachment is patch: true
Attachment #592665 - Flags: review?(mihai.sucan)
(Assignee)

Updated

5 years ago
Attachment #592482 - Attachment is obsolete: true
(Assignee)

Updated

5 years ago
Attachment #592665 - Flags: review?(dao)
(Assignee)

Comment 10

5 years ago
Created attachment 592671 [details] [diff] [review]
Patch for Bug 721324 - Allow the source code editor of Scratchpad & StyleEditor to be themed
Attachment #592671 - Flags: review?(mihai.sucan)
Attachment #592671 - Flags: review?(dao)
(Assignee)

Updated

5 years ago
Attachment #592665 - Attachment is obsolete: true
Attachment #592665 - Flags: review?(mihai.sucan)
Attachment #592665 - Flags: review?(dao)
(Assignee)

Comment 11

5 years ago
Created attachment 592677 [details] [diff] [review]
Patch for Bug 721324 - Allow the source code editor of Scratchpad & StyleEditor to be themed
Attachment #592671 - Attachment is obsolete: true
Attachment #592671 - Flags: review?(mihai.sucan)
Attachment #592671 - Flags: review?(dao)
Attachment #592677 - Flags: review?(mihai.sucan)
Attachment #592677 - Flags: review?(dao)
(Assignee)

Comment 12

5 years ago
Created attachment 592685 [details] [diff] [review]
Patch for Bug 721324 - Allow the source code editor of Scratchpad & StyleEditor to be themed
Attachment #592677 - Attachment is obsolete: true
Attachment #592677 - Flags: review?(mihai.sucan)
Attachment #592677 - Flags: review?(dao)
Attachment #592685 - Flags: review?(mihai.sucan)
Attachment #592685 - Flags: review?(dao)
Comment on attachment 592685 [details] [diff] [review]
Patch for Bug 721324 - Allow the source code editor of Scratchpad & StyleEditor to be themed

>--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
>+++ b/browser/themes/winstripe/devtools/orion.css	Mon Jan 30 21:42:48 2012 +0800

I expected 'hg copy' to create this file based on browser/devtools/sourceeditor/orion/mozilla.css rather than /dev/null. Did this not work?
(Assignee)

Comment 14

5 years ago
Hi! I had initially done a hg add and then removed those files and did a hg copy of mozilla.css. Not sure why its still /dev/null!
(Assignee)

Comment 15

5 years ago
Created attachment 592725 [details] [diff] [review]
Fresh patch after a hg copy of mozilla.css - no change to orion.css
Attachment #592725 - Flags: review?(mihai.sucan)
Attachment #592725 - Flags: review?(dao)

Updated

5 years ago
Attachment #592725 - Flags: review?(mihai.sucan)
Comment on attachment 592685 [details] [diff] [review]
Patch for Bug 721324 - Allow the source code editor of Scratchpad & StyleEditor to be themed

Review of attachment 592685 [details] [diff] [review]:
-----------------------------------------------------------------

Patch looks good for me! Thanks VD for your work and sorry for all the trouble!

Dão: the hg copy trick does not seem to do what we would expect - could be because the copy happens to three locations. Can you please r+ this patch from our contributor? I appreciate his work and I believe that this patch should land, unless there are other problems with it. Thank you!
Attachment #592685 - Flags: review?(mihai.sucan) → review+

Updated

5 years ago
Attachment #592685 - Flags: review?(dao) → review+

Updated

5 years ago
Attachment #592725 - Attachment is obsolete: true
Attachment #592725 - Flags: review?(dao)
Created attachment 594765 [details] [diff] [review]
[in-fx-team] minor update

VD: I removed orion-mozilla.css entirely from browser/devtools - not much point in keeping it.

. and I landed your patch! See:
https://hg.mozilla.org/integration/fx-team/rev/e31fff76aeff

Thank you very much for your contribution! Much appreciated! Keep up the good work!

Updated

5 years ago
Whiteboard: [sourceeditor][orion][good first bug][mentor=msucan] → [sourceeditor][orion][good first bug][mentor=msucan][fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/e31fff76aeff
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Whiteboard: [sourceeditor][orion][good first bug][mentor=msucan][fixed-in-fx-team] → [sourceeditor][orion][good first bug][mentor=msucan]
Target Milestone: --- → Firefox 13
Please document the new themeable Source Editor (Orion) CSS files in Firefox 13. Thank you!
Keywords: dev-doc-needed
You need to log in before you can comment on or make changes to this bug.