107685 - Need icons for default mozilla sidebar tab headers

Status: RESOLVED WONTFIX

(SeaMonkey :: Sidebar, defect)

Description

[Samir Gehani]

Per bug 67110, I have the code working to conditionally show an icon next to the
tab title (if specified).  Now I need some icons to include in the product. 
Here are the tabs that we need icons for:

(*) Search
(*) What's Related
(*) Bookmarks
(*) History

Additionally, it would be nice to have icons for the other two default mozilla
tabs but not as high priority as the above icons (IMHO):

(*) Tinderbox
(*) Tinderbox-Ports

The icons can be 16px x 16px gifs, similar to what I used to test:
<http://search.netscape.com/sherlock/SBWeb_02.gif>

You can refer to
<http://lxr.mozilla.org/seamonkey/source/profile/defaults/panels.rdf> for URLs
to the contents of these tabs when designing the icons.

Comment 1

[andreww]

Favicon support is close to being implemented, why not instead of having 
static icons in the sidebar, allow the favicon.ico get fetched from the 
desired sidebar?  This way the server could display the proper  icon, and 
it would allow _all_ sidebars to optionally have icons

See bug 

http://bugzilla.mozilla.org/show_bug.cgi?id=32087

.ico support is already in!  see:
http://bugzilla.mozilla.org/show_bug.cgi?id=18502

And these are already 16X16 pixel images.  And the mechanism for 
displaying the icon would be identical to how websites would show their 
icons.  

Comment 2

[Samir Gehani]

Interesting thought Andrew.  At any rate, the intention always was to allow all
sidebars to optionally have icons: they would specify them in the new extended
window.sidebar.addPanelEx() API as a URL when a user adds a panel from a
website.  Since we now understand the .ico format it would be pretty close to
favicon support except that it could be specified only once.  Maybe we can add
favicon support for sidebar headers and failover to the static ones.

Comment 3

[Alex Bishop]

I think it's important that the icons for the Search, What's Related, Bookmarks
and History tabs should be 'themeless' i.e. not look particularly like they
belong to either Modern or Classic (as they're not going to be theme dependent,
right?).

Comment 4

[andreww]

I dont think they should be "themeless" - since they are in the chrome - I 
think they should behave in the same way that the status bar (taskbar) 
icons at the bottom left do - which is to match their theme.

Comment 5

[marlon bishop]

we need to see the performance impact of this feature before we get started on
creating the icons.

Comment 6

[andreww]

I see some performance metrics here:
http://www.mozilla.org/xpapps/sidebar/

under Perf.

I'm  going to do some testing in the next few days myself to see if I can 
determine the impact of adding images to the sidebar using the js tool  
described there.

of  primary interest look here:
http://www.mozilla.org/xpapps/sidebar/perf/rdf-vs-paint.html

Comment 7

[Ian Pottinger]

adding self to cc list

Comment 8

[Lori Kaplan]

minusing. the feature was cut due to performance impact

Comment 9

[Chris Nalls]

Closing - WONTFIX due to performance degradation.