Closed Bug 595165 Opened 14 years ago Closed 14 years ago

Search icon is missing for Tab Candy

Categories

(Firefox Graveyard :: Panorama, defect)

x86
All
defect
Not set
minor

Tracking

(Not tracked)

VERIFIED FIXED
Firefox 4.0b7

People

(Reporter: vladmaniac, Assigned: raymondlee)

References

Details

Attachments

(2 files, 2 obsolete files)

Attached image screenshot
Steps: 
1. Browse in some different tabs
2. Open Tab Candy 
3. Inspect search icon 

================================================================================

Missing search icon of Tab Candy for win7(32). 

Works with Ubuntu 10.04(64) and MAC OS X 10.6(32) 

Note: See screenshot
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → DUPLICATE
juan, 

I meant that there is no magnifying glass for win7 in the icon, as there is for mac and linux. 

I referred to it as general "search icon".
Status: RESOLVED → REOPENED
OS: Windows 7 → All
Resolution: DUPLICATE → ---
Attached patch v1 (obsolete) — Splinter Review
Attachment #474663 - Flags: feedback?(ian)
Attached patch V1 (obsolete) — Splinter Review
Attachment #474663 - Attachment is obsolete: true
Attachment #474665 - Flags: feedback?(ian)
Attachment #474663 - Flags: feedback?(ian)
Blocks: 594094
Comment on attachment 474665 [details] [diff] [review]
V1

>         skin/classic/aero/browser/tabbrowser/tab-arrow-left.png                 (tabbrowser/tab-arrow-left.png)
>         skin/classic/aero/browser/tabbrowser/tabDragIndicator.png               (tabbrowser/tabDragIndicator.png)
>         skin/classic/aero/browser/tabview/edit-light.png             (tabview/edit-light.png)
>         skin/classic/aero/browser/tabview/edit.png                   (tabview/edit.png)
>         skin/classic/aero/browser/tabview/new-tab.png                (tabview/new-tab.png)
>         skin/classic/aero/browser/tabview/tabview.css                (tabview/tabview.css)
>         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/search.png                 (tabview/search.png)

Move this as well as stack-expander.png before tabview.css.
Attachment #474665 - Flags: review+
Attached patch v1Splinter Review
Attachment #474665 - Attachment is obsolete: true
Attachment #474687 - Flags: review?(dao)
Attachment #474665 - Flags: feedback?(ian)
Comment on attachment 474687 [details] [diff] [review]
v1

Strictly speaking tabview.css should really precede tabview.png.
Attachment #474687 - Flags: review?(dao) → review+
Attachment #474687 - Flags: approval2.0?
Assignee: nobody → raymond
Comment on attachment 474687 [details] [diff] [review]
v1

a=beltzner
Attachment #474687 - Flags: approval2.0? → approval2.0+
http://hg.mozilla.org/mozilla-central/rev/713ca0604cc0
Status: REOPENED → RESOLVED
Closed: 14 years ago14 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 4.0b6
verified on recent nightly builds of minefield
Status: RESOLVED → VERIFIED
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: