Note: There are a few cases of duplicates in user autocompletion which are being worked on.

The command line background graphic should be moved to the devtools directory

RESOLVED FIXED in Firefox 13



Developer Tools
6 years ago
6 years ago


(Reporter: jwalker, Assigned: lcamacho)


Firefox 13

Firefox Tracking Flags

(Not tracked)


(Whiteboard: [good first bug][mentor=dao][lang=css])


(1 attachment, 1 obsolete attachment)

Comment 1

6 years ago
We're doing developer tool prioritization, filter on 'brontozaur'
to ignore the spam.
Priority: -- → P2
Assignee: nobody → jwalker
Blocks: 689605
Priority: P2 → --
Priority: -- → P2
Blocks: 703275
Blocks: 659052
No longer blocks: 689605
Assignee: jwalker → nobody
Whiteboard: [good-first-bug]
No longer blocks: 659052

Comment 2

6 years ago
this look easy, can I take this bug?
(In reply to leonard.camacho from comment #2)
> this look easy, can I take this bug?


You'll find all references to commandline.png here:

You need to 'hg copy' toolkit/themes/winstripe/global/icons/commandline.png to browser/themes/gnomestripe/devtools/commandline.png, then 'hg move' toolkit/themes/winstripe/global/icons/commandline.png to browser/themes/winstripe/devtools/commandline.png and toolkit/themes/pinstripe/global/icons/commandline.png to browser/themes/pinstripe/devtools/commandline.png.

The commandline.png entries in toolkit/themes/winstripe/global/ and toolkit/themes/pinstripe/global/ need to be removed and added to browser/themes/winstripe/, browser/themes/pinstripe/ and browser/themes/gnomestripe/

Finally, change all chrome://global/skin/icons/commandline.png references to chrome://browser/skin/devtools/commandline.png.

Comment 4

6 years ago
Created attachment 594871 [details] [diff] [review]

First try, @dao I dont know who need to review this, can you help me with that?


6 years ago
Assignee: nobody → leonard.camacho
Comment on attachment 594871 [details] [diff] [review]

The code changes look fine, but something seems to be wrong with how you moved or diffed commandline.png. For instance, I see "Binary file browser/themes/gnomestripe/devtools/commandline.png has changed" but no actual data.

Can you try hg diff -g?

Comment 6

6 years ago
It tells me this about an image

diff --git a/toolkit/themes/winstripe/global/icons/commandline.png b/browser/themes/winstripe/devtools/commandline.png
rename from toolkit/themes/winstripe/global/icons/commandline.png
rename to browser/themes/winstripe/devtools/commandline.png
This looks better. Could you please generate the full patch with -g and attach it here?


6 years ago
Whiteboard: [good-first-bug] → [good first bug][mentor=dao][lang=css]

Comment 8

6 years ago
Created attachment 594875 [details] [diff] [review]
patchV1 generated with hg diff -g
Attachment #594871 - Attachment is obsolete: true
Comment on attachment 594875 [details] [diff] [review]
patchV1 generated with hg diff -g

I hope this applies the right way (toolkit/themes/pinstripe/global/icons/commandline.png needs to be copied, then moved, the other way around obviously won't work). I'll check this after I got some sleep.
Attachment #594875 - Flags: review?(dao)
Comment on attachment 594875 [details] [diff] [review]
patchV1 generated with hg diff -g

Works. Thanks for doing this!
Attachment #594875 - Flags: review?(dao) → review+
Target Milestone: --- → Firefox 13
looks like the file has not been moved?
or better, for some reason win64 can't find it
I removed the aero icon from, that was a leftover

Comment 15

6 years ago
@mak thanks I didn't see that one
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.