Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 343982

Summary: [client] new icons to match visual style
Product: [ECD] Orion Reporter: Susan McCourt <susan>
Component: ClientAssignee: Susan McCourt <susan>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: bokowski, john.arthorne, libingw, malgorzata.tomczyk, mamacdon, Mike_Wilson, Szymon.Brandys
Version: 0.2   
Target Milestone: 0.2   
Hardware: PC   
OS: Windows 7   
Whiteboard:
Attachments:
Description Flags
Show Git Log on the Git Repo view
none
The clock on the file picker none

Description Susan McCourt CLA 2011-04-27 11:31:28 EDT
It looks like we'll get some cycles to rework the icons in light of the current style.  I'll be making a list early in M8 of the icons needed, based on what is already in Orion.  Please make a note here if you have new icons coming.
Comment 1 Susan McCourt CLA 2011-05-05 13:58:53 EDT
Regarding icons.
They are useful for the local actions that appear inside the actions column, where space is tight.  Also nice to include in the menus.

But I'm starting to think that for the page level commands, we should be rendering them all as text links.  There are typically only two or three icons at that level anyway, and we want the user to be able to find the common page tasks easily without having to decipher an icon.
Comment 2 Susan McCourt CLA 2011-05-05 15:09:27 EDT
(In reply to comment #1)
> But I'm starting to think that for the page level commands, we should be
> rendering them all as text links. 

mockups posted to bug 344876
Comment 3 Susan McCourt CLA 2011-05-05 16:15:51 EDT
I released the "text only commands on the page toolbar" as part of bug 344876.  We'll still design icons for some of the commands which also appear in other places, or if we think they might in the future.   But we'll prioritize accordingly.
Comment 4 Susan McCourt CLA 2011-05-10 18:49:20 EDT
This is the list I've sent the graphic designer.  If you don't see an icon you need in this list, please annotate this bug.

Navigator
1.  Add (as in Add favorite, or Add site)
2.  "Make Favorite" (star)
3.  "Rename"
4.  "Download" or "Export" to zip
5.  "Delete"
6.  "Upload" or "Import" as zip
7.  "SFTP Import" (may use generic import and use the menu words to distinguish)
8.  "SFTP Export" (may use generic export and use menu words to distinguish)
9.  "New File"
10.  "New Folder"
11.  Checkboxes in checked and unchecked state
12.  Tree twisties in three states:  closed/collapsed, opening, open/expanded

Editor (No icons, only the text "Save" command)
Sites  (Only existing icons, need to get rid of floppy disk on "Save" button)

Plugins
13.  Refresh

Repositories
14.  Link to folder

Git Log
15.  Tag

Git Status
Per discussion in bug 336116, I did not ask for icons for actions against the entire commit (commit) or the entire branch (push, pull, fetch, merge, branch).  We most likely will use text links for these actions.  To my knowledge, none of these appear "in-line" in an items list where we would need a compact representation.  So I've only listed the ones that annotate items.
16.  Stage all
17.  Stage
18.  Side by side compare
19.  Unstage 
20.  Unstage all

Git Remote
21.  Incoming
22.  Outgoing
Comment 5 Boris Bokowski CLA 2011-05-11 10:17:52 EDT
(In reply to comment #4)
> Git Status
> ...  So I've only listed the ones that annotate items.

Makes sense. Do we need an icon for "replace with latest from branch/HEAD" since this would be a per-file action?
Comment 6 Mark Macdonald CLA 2011-05-17 12:22:08 EDT
(In reply to comment #5)

Just added:

Sites:
1. Move up
2. Move down
Comment 7 Mark Macdonald CLA 2011-05-17 12:32:28 EDT
(In reply to comment #6)
Also, do we want to use icons to represent resources or model objects (eg. a site configuration, a plugin, a service)? I've been using an icon from Eclipse to represent a site config.
Comment 8 John Arthorne CLA 2011-05-17 13:23:13 EDT
I added an Icon for Help in the toolbar. I think it looks fine but just mentioning it here for completeness.
Comment 9 Susan McCourt CLA 2011-05-17 15:03:44 EDT
Added requests for the following:

23. Help Icon
24. Move up in list
25. Move down in list
26. Replace with remote version of file (for git-status)
Comment 10 Szymon Brandys CLA 2011-05-20 09:08:50 EDT
We would need the following icons:
1. Git Clone UI
- Repo icon
- Branches and Branch icons
- Remotes and Remote icons
- Tags icon (Tag icon was already requested)

2. Git Checkout operations

I think that for adding/removing branches and remotes I can use the same icons we have in the file navigator and the favorites list i.e. '+' and 'x'.
Comment 11 Szymon Brandys CLA 2011-05-20 09:11:13 EDT
I can't see a request for diff icons. Both status and log pages would need icons showing:
- modify change
- addition change
- removing change
Comment 12 Susan McCourt CLA 2011-05-23 12:27:21 EDT
Szymon, are there CVS icons you are using today?  I've been sending the designer the icons we are currently using as a way to guide the effort.
Comment 13 Szymon Brandys CLA 2011-05-23 12:44:54 EDT
(In reply to comment #12)
> Szymon, are there CVS icons you are using today?  I've been sending the designer
> the icons we are currently using as a way to guide the effort.

> - modify change
> - addition change
> - removing change

are taken from the git status UI. I don't know where we have them from.

> 1. Git Clone UI
> - Repo icon
> - Branches and Branch icons
> - Remotes and Remote icons
> - Tags icon (Tag icon was already requested)

We could use CVS icons for repo, branch(es) and tag(s). There is no CVS icon for remote(s) though.

> 2. Git Checkout operations

There is no CVS icon for this operation. I used EGit one.
Comment 14 Susan McCourt CLA 2011-05-25 12:25:59 EDT
moving styling bugs to RC1
Comment 15 Susan McCourt CLA 2011-06-02 16:18:36 EDT
Update on icon status:
The first round from the icon designer wasn't impressively different than what we already have.  Linda is working with her on establishing a distinctive style that would be noticeably different/improved.  It is looking promising.

But there is a chance we won't get them all done in time, so our fallback is that icons won't change.  A middle ground is that icons that appear on the page (hovering actions, in the lists/trees) get done first and those that appear in menus would not be done.

Given this, we need to ensure the proper IP is in place for the icons that don't come from Eclipse Platform.

Szymon - you mentioned you took an icon (checkout) from eGit.  Are there any other icons you have obtained from a non-Eclipse Platform project??

Re: new(er) git icons.  Now that some of the actions have been added to the git remote page, it would seem we need to request icons for push, pull, merge.  I took screencaps of the latest UI and sent an updated request to the designer.  These are the additional git icons needed.  Szymon, please post here if you think I have forgotten anything.

27.  Repository
28.  Branch (singular)
29.  Branches (plural)
30.  Remote (singular)
31.  Remotes (plural)
32.  Checkout
33.  Push
34.  Fetch
35.  Merge
36.  Modification
37.  Addition
38.  Removal
Comment 16 Susan McCourt CLA 2011-06-02 16:26:36 EDT
(In reply to comment #13)
> > - modify change
> > - addition change
> > - removing change

Libing - can you verify where these icons came from?
Comment 17 Susan McCourt CLA 2011-06-15 01:46:37 EDT
released all new icons.
Please everyone check all your pages.
I checked the obvious stuff but might not have hit all hovering actions, etc.  
I tried to delete unused files in this pass also and it's possible that someone is still relying on a file I deleted.

I talked to Simon about this and we agreed it's better to push the change and then have everyone check their pages then to try to source code review the name changes first.  The testing is more important.
Comment 18 Szymon Brandys CLA 2011-06-15 03:45:30 EDT
Created attachment 198000 [details]
Show Git Log on the Git Repo view

The new icons look very nice. I wonder if we should replace "Show Git Log" on the clone page with an icon too. Otherwise the Git pages look good.
Comment 19 Szymon Brandys CLA 2011-06-15 03:47:50 EDT
Created attachment 198001 [details]
The clock on the file picker

Was this icon also replaced?
Comment 20 Susan McCourt CLA 2011-06-15 10:14:36 EDT
(In reply to comment #18)
> Created attachment 198000 [details]
> Show Git Log on the Git Repo view
> 
> The new icons look very nice. I wonder if we should replace "Show Git Log" on
> the clone page with an icon too. Otherwise the Git pages look good.

This has always seemed a bit weird to me, but I think for now just reordering the commands so that the link appears at the end would make it look better.

(In reply to comment #19)
> Created attachment 198001 [details]
> The clock on the file picker
> 
> Was this icon also replaced?

No, none of the dojo tree icons were replaced.  I'll open a bug for this, as we also have some old school looking icons in the plugin page.