Closed Bug 685141 Opened 9 years ago Closed 9 years ago

csshtmltree.css & arrows.png are missing from aero theme

Categories

(DevTools :: General, defect)

Product:
DevTools
Component:
General
Platform:
All
Windows 7
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
Firefox 9

People

(Reporter: miker, Assigned: miker)

Details

(Whiteboard: [styleinspector][minotaur][fixed-in-fx-team])

Attachments

(1 file)

[in-fx-team] Missing Aero Stuff patch 1
1.73 KB, patch
msucan
: review+
Details | Diff | Splinter Review 
csshtmltree.css & arrows.png are missing from the aero theme
Assignee: nobody → mratcliffe
Status: NEW → ASSIGNED
I didn't know that there are 2 themes in winstripe
Attachment #558792 - Flags: review?(mihai.sucan)
Comment on attachment 558792 [details] [diff] [review]
[in-fx-team] Missing Aero Stuff patch 1

># HG changeset patch
># Parent 2857288e9a3faa4660e1800a61fb887be2a85e95
># User Michael Ratcliffe <mratcliffe@mozilla.com>
># Date 1315403849 -7200
>
>Bug 685141 - csshtmltree.css & arrows.png are missing from aero theme
>
>diff --git a/browser/themes/winstripe/browser/jar.mn b/browser/themes/winstripe/browser/jar.mn
>--- a/browser/themes/winstripe/browser/jar.mn
>+++ b/browser/themes/winstripe/browser/jar.mn
>@@ -223,16 +223,18 @@ browser.jar:
>         skin/classic/aero/browser/tabview/close.png                  (tabview/close.png)
>         skin/classic/aero/browser/tabview/edit-light.png             (tabview/edit-light.png)
>         skin/classic/aero/browser/tabview/grain.png                  (tabview/grain.png)
>         skin/classic/aero/browser/tabview/search.png                 (tabview/search.png)
>         skin/classic/aero/browser/tabview/stack-expander.png         (tabview/stack-expander.png)
>         skin/classic/aero/browser/tabview/tabview.png                (tabview/tabview.png)
>         skin/classic/aero/browser/tabview/tabview-inverted.png       (tabview/tabview-inverted.png)
>         skin/classic/aero/browser/tabview/tabview.css                (tabview/tabview.css)
>+        skin/classic/aero/browser/devtools/arrows.png                (devtools/arrows.png)
>+        skin/classic/aero/browser/devtools/csshtmltree.css           (devtools/csshtmltree.css)
> #ifdef MOZ_SERVICES_SYNC
>         skin/classic/aero/browser/sync-throbber.png
>         skin/classic/aero/browser/sync-16.png
>         skin/classic/aero/browser/sync-32.png
>         skin/classic/aero/browser/sync-bg.png
>         skin/classic/aero/browser/sync-desktopIcon.png
>         skin/classic/aero/browser/sync-mobileIcon.png
>         skin/classic/aero/browser/sync-notification-24.png
Attachment #558792 - Flags: review?(mihai.sucan) → review+
Huh? I had a comment there:

"Patch looks fine, but I can't test it. I'm on Linux. :)

r+!"

(and I didn't intend for the diff to show in my previous comment)
Whiteboard: [styleinspector][minotaur][land-in-fx-team]
Comment on attachment 558792 [details] [diff] [review]
[in-fx-team] Missing Aero Stuff patch 1

Pushed:
https://hg.mozilla.org/integration/fx-team/rev/7107a577e4f6
Attachment #558792 - Attachment description: Missing Aero Stuff patch 1 → [in-fx-team] Missing Aero Stuff patch 1
Whiteboard: [styleinspector][minotaur][land-in-fx-team] → [styleinspector][minotaur][fixed-in-fx-team]
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
AFAIK, bugs are FIXED only when they land in m-c.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
https://hg.mozilla.org/mozilla-central/rev/7107a577e4f6
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 9
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.