Bug 478367 - [UCOSP] Add glasses decorator icon to Mylyn commons
Summary: [UCOSP] Add glasses decorator icon to Mylyn commons
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: unspecified   Edit
Hardware: All Linux
: P3 enhancement (vote)
Target Milestone: 3.18   Edit
Assignee: Vaughan Hilts CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 478362
  Show dependency tree
 
Reported: 2015-09-24 20:51 EDT by Vaughan Hilts CLA
Modified: 2015-11-13 19:16 EST (History)
1 user (show)

See Also:


Attachments
glasses icon (177 bytes, image/png)
2015-09-25 13:10 EDT, Sam Davis CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Vaughan Hilts CLA 2015-09-24 20:51:28 EDT
This icon will be required for other tasks, so we're adding it in this review so it can be used elsewhere.
Comment 1 Vaughan Hilts CLA 2015-09-24 20:52:48 EDT
@Sam: A few questions and assumptions I worked off...

1) Can this decorator be overlapped on multiple of the icon types or is it it's own kind of incoming icon? (The latter is kind of how it's used right now, versus a stock glasses icon)
Comment 2 Eclipse Genie CLA 2015-09-24 20:54:17 EDT
New Gerrit change created: https://git.eclipse.org/r/56685
Comment 3 Sam Davis CLA 2015-09-25 13:06:32 EDT
I changed the assignee to Vaughan since you pushed a review. You should set the status to assigned when you start working on something so nobody else does.

I'm not sure if I understand the question, but you have to assume any of the existing decorators could be shown at the same time as this new one (but only one of them), so it should look OK when combined with any of them.
Comment 4 Sam Davis CLA 2015-09-25 13:10:16 EDT
Created attachment 256849 [details]
glasses icon

It follows that the decorator should be just the glasses icon by itself. Here's the icon I used to create the mockup.
Comment 5 Vaughan Hilts CLA 2015-10-12 17:10:43 EDT
Can someone other than Sam look at this, please? It'd be good to be able to get it merged in as Bug 478364 will depend on it.
Comment 7 Sam Davis CLA 2015-11-13 19:16:52 EST
.