Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 370481 - specialized icons for banner, toolbars and slideouts
Summary: specialized icons for banner, toolbars and slideouts
Status: RESOLVED FIXED
Alias: None
Product: Orion
Classification: ECD
Component: Client (show other bugs)
Version: 0.4   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 0.4 RC3   Edit
Assignee: Susan McCourt CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-02-02 14:03 EST by Susan McCourt CLA
Modified: 2012-02-21 20:48 EST (History)
3 users (show)

See Also:
ken_walker: review+
mamacdon: review+


Attachments
pic (1.37 KB, image/png)
2012-02-21 16:30 EST, Mark Macdonald CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Susan McCourt CLA 2012-02-02 14:03:59 EST
Per bug 360986, we will rarely use icons in the toolbars and slideouts.  Only special purpose controls/navigations will be used.  We will want these to be a different style than the yellow.  Here is the working list:

- close (X).  For the slideout and for closing the notification messages that
appear in the slideout.  Not yellow.
- OK/Submit (check mark).  For the slideout.  Not yellow.
- More (...) for the slideout.  Not yellow.
- Favorite icon in banner?? Not yellow???
- Navigation arrows - up, down, left, right.  These arrows will be used in
toolbars when an arrow makes more sense than words (next search result, etc.). 
Need non yellow versions.
- Operations running animated icon (we are currently using the eclipse one,
might be good enough)
- Operations none.  A button to open the current operations list when none are
running.
- Operation ended in error.
- Operation ended in warning.
- help (this may go in the user menu so not sure yet it's needed.  See bug 367784)
Comment 1 Susan McCourt CLA 2012-02-03 02:41:34 EST
need a polished/centered sep.gif
Comment 2 Susan McCourt CLA 2012-02-06 18:20:43 EST
Pairing down the list after meeting with Linda.
We've looked at the current implementation and concluded:
- the arrow icons in yellow are looking fine
- we will use text in the slideout for OK/Submit and More rather than icons
- we need better error/warning/info slideout icons than the old eclipse ones that we are using.

So the working list of icons is now:
- close (X)
- Favorite icon (Star, but muted for banner, not yellow)
- error (for notification slideout)
- warning (for notification slideout)
- info (for notification slideout)
- progress (for notification slideout)
- operations running animated icon 

Re: operations none, progress/info, progress/warning, progress/error.  I'm not convinced these are necessary.  Opened bug 370782.
Comment 3 Susan McCourt CLA 2012-02-10 16:49:09 EST
we've received everything except the busy icons.
Comment 4 Susan McCourt CLA 2012-02-10 17:45:10 EST
- the favorites icon for the banner seems too small.
- also need to tweak the style so it gets the consistent hover that buttons get
Comment 5 Susan McCourt CLA 2012-02-21 14:43:09 EST
remote branch
origin/iconsprites04RC3

contains the last set of icons from the graphic designers:

- git active branch
- the new, cool, groovy "busy" icon
- a small menu twistie for dojo menus (so that the double chevron is only used for menus with no text at all)

Changes to source:
- replaced various icon files, added new ones
- commands.css - change default dojo menu twistie and add "textless" style for chevron
- ide.css - change busy icons and add margin so they aren't butted up against the top of the toolbar
- commands.js - use "textless" to get chevron on textless menu
- commonHTMLFragments.js - use "textless" to get chevron on user menu
- textSearcher.js - switch command definitions from using image files to using css sprite classes (that were already there).  This prevents an image loading race condition that was sometimes causing the find/replace slideout to flicker/change size


Observed improvements:
- Related pages has a small twistie, not the honking chevron
- Orion busy icon, better positioning in toolbar
- Find/replace dialog, no "resize bump" in image loading
Comment 6 Susan McCourt CLA 2012-02-21 14:46:42 EST
commit SHA is
2b53d738c20727fefbe6dacd780986f35588e49c
(if you prefer open commit vs. going to the log)
Comment 7 Ken Walker CLA 2012-02-21 15:53:56 EST
As a project lead I +1 this for RC3
Comment 8 Mark Macdonald CLA 2012-02-21 16:30:39 EST
Created attachment 211369 [details]
pic

The only possible issue I saw is that the mini-chevron appears somewhat smushed against the right-hand border when you hover over a dropdown. This was not the case with the old honking big chevron
Comment 9 Susan McCourt CLA 2012-02-21 19:25:07 EST
pushed fix.
Comment 10 Susan McCourt CLA 2012-02-21 20:47:12 EST
(In reply to comment #8)

> The only possible issue I saw is that the mini-chevron appears somewhat smushed
> against the right-hand border when you hover over a dropdown. This was not the
> case with the old honking big chevron

I fixed this in the spriting class.  The dojo image overrides require a little hand-crafting anyway so I added a margin around the little twistie.
Comment 11 Susan McCourt CLA 2012-02-21 20:48:13 EST
(In reply to comment #10)
> (In reply to comment #8)
> 
> > The only possible issue I saw is that the mini-chevron appears somewhat smushed
> > against the right-hand border when you hover over a dropdown. This was not the
> > case with the old honking big chevron
> 
> I fixed this in the spriting class.  The dojo image overrides require a little
> hand-crafting anyway so I added a margin around the little twistie.

in the spriting *pass* (not class).  
The margin for the small twistie gets added in bug 371262.

Thanks, Mark.