428098 - Most Visited item in Bookmarks Toolbar does not use a smart search icon

Status: VERIFIED FIXED

(Firefox :: Theme, defect)

Description

[Andrés Delfino]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9pre) Gecko/2008040907 Minefield/3.0pre
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9pre) Gecko/2008040907 Minefield/3.0pre

Most Visited item in Bookmarks Toolbar does not use a smart search icon.

Reproducible: Always

Steps to Reproduce:
1. See the Most Visited item in Bookmarks Toolbar.
Actual Results:  
The item uses a normal folder as an icon.

Expected Results:  
The item should use a smart search folder as an icon.

Comment 1

[Andrés Delfino]

Attachment: Screenshot in Windows XP.

Comment 2

[Andrés Delfino]

Nominating as blocking since all this work around Library should be as consistent as possible, since it's new stuff that the user will experiment with.

Comment 3

[Andrés Delfino]

Attachment: Screenshot in Ubuntu 7.10.

Comment 4

[Wesley Johnston (:wesj)]

Attachment: Adds query attribute to toolbar buttons

Looks like the query attribute isn't being checked/added here. Quick patch to just add this attribute check.

Comment 5

[Mike Connor [:mconnor]]

Comment on attachment 314771 [details] [diff] [review]
Adds query attribute to toolbar buttons

r=mconnor, thanks!

Comment 6

[Mike Beltzner [:beltzner, not reading bugmail]]

Comment on attachment 314771 [details] [diff] [review]
Adds query attribute to toolbar buttons

a=beltzner, thanks!

Comment 7

[:Gavin Sharp [email: [e-mail]]]

mozilla/browser/components/places/content/toolbar.xml 	1.151