Bug 506679 - Replace "(locked)" suffix by a lock icon in content-types
Summary: Replace "(locked)" suffix by a lock icon in content-types
Status: CLOSED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.6   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 4.8   Edit
Assignee: Ian Pun CLA
QA Contact: Mickael Istria CLA
URL:
Whiteboard:
Keywords: bugday, Documentation, helpwanted
Depends on:
Blocks:
 
Reported: 2016-10-28 04:10 EDT by Mickael Istria CLA
Modified: 2017-07-24 14:53 EDT (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Mickael Istria CLA 2016-10-28 04:10:53 EDT
The content-type preference page shows (locked) suffix after locked content-type/file associations. It should rather use the nice lock icon that's available for progress reporting,
Comment 1 Patrik Suzzi CLA 2016-11-01 06:56:36 EDT
Could you please provide a reference or image to the lock icon that's available for progress reporting?
Comment 2 Mickael Istria CLA 2016-11-01 16:11:42 EDT
(In reply to Patrik Suzzi from comment #1)
> Could you please provide a reference or image to the lock icon that's
> available for progress reporting?

https://github.com/eclipse/eclipse.platform.ui/blob/master/bundles/org.eclipse.e4.ui.progress/icons/full/progress/lockedstate.png
Comment 3 Mickael Istria CLA 2016-11-23 05:04:35 EST
@Tony: some time ago, I was wondering whether the Eclipse Platform could host a silhouette icon. This "lock" would actually be the opposite use-case of the silhouette one: user defined and modifiable content would be using the silhouette icon, whereas system non modifiable would use the "lock" icon.
So it would be interesting if those icons were using similar "style". WDYT?
Comment 4 Eclipse Genie CLA 2016-11-25 18:06:43 EST
New Gerrit change created: https://git.eclipse.org/r/85813
Comment 5 Ian Pun CLA 2016-11-25 18:08:11 EST
I've added the lock icon to the beginning of the cell item and removed the "(locked)" tags as well. 

It is currently grabbing the image directly from the Progress Manager image key, so it is best to discuss where we can copy over this icon or if this is adequate.
Comment 6 Mickael Istria CLA 2016-11-28 09:40:08 EST
@Ian: I've got an alternative idea: what about using Unicode character for lock  
Comment 7 Mickael Istria CLA 2016-11-28 09:49:50 EST
(In reply to Mickael Istria from comment #6)
> @Ian: I've got an alternative idea: what about using Unicode character for
> lock

Unless I did something wrong, it seems like bugzilla doesn't support Unicode...
Comment 8 Ian Pun CLA 2016-11-28 10:05:28 EST
(In reply to Mickael Istria from comment #7)
> (In reply to Mickael Istria from comment #6)
> > @Ian: I've got an alternative idea: what about using Unicode character for
> > lock
> 
> Unless I did something wrong, it seems like bugzilla doesn't support
> Unicode...

Hmm, apparently bugzilla states that its supported:

https://www.bugzilla.org/features/#unicode

how are you testing it to see if it supports it?
Comment 9 Mickael Istria CLA 2016-11-28 10:06:31 EST
(In reply to Ian Pun from comment #8)
> > Unless I did something wrong, it seems like bugzilla doesn't support
> > Unicode...
> 
> Hmm, apparently bugzilla states that its supported:
> https://www.bugzilla.org/features/#unicode
> how are you testing it to see if it supports it?

See https://bugs.eclipse.org/bugs/show_bug.cgi?id=508300
Comment 10 Ian Pun CLA 2016-11-28 10:15:57 EST
(In reply to Mickael Istria from comment #9)
> (In reply to Ian Pun from comment #8)
> > > Unless I did something wrong, it seems like bugzilla doesn't support
> > > Unicode...
> > 
> > Hmm, apparently bugzilla states that its supported:
> > https://www.bugzilla.org/features/#unicode
> > how are you testing it to see if it supports it?
> 
> See https://bugs.eclipse.org/bugs/show_bug.cgi?id=508300

I think (at least for now) we should stick to the lock image instead of moving to the Emoji, just so that it matches with the rest of the UI, unless we make a serious effort to move to emojis for a large portion of the platform.

what do you think?
Comment 11 Mickael Istria CLA 2016-11-28 10:17:34 EST
(In reply to Ian Pun from comment #10)
> I think (at least for now) we should stick to the lock image instead of
> moving to the Emoji, just so that it matches with the rest of the UI, unless
> we make a serious effort to move to emojis for a large portion of the
> platform.
> 
> what do you think?

Yeah, I think it makes sense, especially for this case when an icon already exists. The value of emojis is higher for icons that don't exist in the Eclipse world.
Comment 12 Mickael Istria CLA 2017-07-03 14:38:36 EDT
@Ian: could you please update the documentation about content-type preference page to mention this icon rather than the "(locked)" text?
Comment 13 Mickael Istria CLA 2017-07-03 14:43:31 EDT
On the same topic, it would be nice to place a silhouette for user-defined association.
Comment 15 Ian Pun CLA 2017-07-24 14:53:13 EDT
Patch has been merged into master. Closing.