Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 344107 - decorate gerrit tasks with review icon
Summary: decorate gerrit tasks with review icon
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P3 enhancement (vote)
Target Milestone: 0.9   Edit
Assignee: Mik Kersten CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-28 06:54 EDT by Benjamin Muskalla CLA
Modified: 2012-06-18 18:07 EDT (History)
1 user (show)

See Also:


Attachments
icon (1004 bytes, image/png)
2011-05-12 16:47 EDT, Benjamin Muskalla CLA
no flags Details
three sizes of the standard glasses icon (356 bytes, image/png)
2011-07-14 01:03 EDT, Mik Kersten CLA
no flags Details
screenshot (5.64 KB, image/png)
2011-07-18 20:06 EDT, Steffen Pingel CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Benjamin Muskalla CLA 2011-04-28 06:54:25 EDT
Gerrit reviews should have a icon decoration to easily distinguish between tasks from other systems and gerrit reviews.
Comment 1 Steffen Pingel CLA 2011-05-07 20:41:28 EDT
We have been very careful with vendor branding in the UI. Generally, we only show that for repositories. Agreed though that we should have a review specific task decoration.
Comment 2 Benjamin Muskalla CLA 2011-05-12 16:46:40 EDT
I think it makes sense to use a generic "review task" decorator across all review implementations.
Comment 3 Benjamin Muskalla CLA 2011-05-12 16:47:12 EDT
Created attachment 195545 [details]
icon

How about this? ;)
Comment 4 Steffen Pingel CLA 2011-05-12 17:14:23 EDT
Not a bad idea. I like it better than the glasses that we have been using elsewhere but it might be hard to distinguish from the standard decoration for stories. I'll bring it up on the next Mylyn call.
Comment 5 Steffen Pingel CLA 2011-06-16 17:31:29 EDT
We discussed this on the call today and had some concerns that the decoration is too ambiguous. The conclusion was to try an eyeglass decoration as an alternative.
Comment 6 Mik Kersten CLA 2011-07-14 01:03:25 EDT
Created attachment 199633 [details]
three sizes of the standard glasses icon
Comment 7 Steffen Pingel CLA 2011-07-15 20:12:49 EDT
Thanks Mik. I'll add that as a decoration.
Comment 8 Mik Kersten CLA 2011-07-18 13:17:02 EDT
Could you post the result?
Comment 9 Steffen Pingel CLA 2011-07-18 20:06:44 EDT
Created attachment 199872 [details]
screenshot
Comment 10 Mik Kersten CLA 2011-07-22 02:21:15 EDT
Not bad.  Kinda hard to see that they are glasses, but at least it's a standard Eclipse icon.
Comment 11 Steffen Pingel CLA 2011-08-11 13:47:28 EDT
Yes, it's not easy to recognize the glasses but it's distinguishes review tasks well and it's easy to remember once you have learned the icon. As per the review on today's call I'll mark this bug as resolved.
Comment 12 Sam Davis CLA 2012-06-18 18:07:29 EDT
I never would have guessed that those were glasses. It seems kind of pointless to use a "standard" icon if it's unrecognizable. Maybe a magnifying glass would be easier to see?