Community
Participate
Working Groups
Created attachment 286608 [details] target See screenshot, using the same icon in two location is confusing IMHO. I suggest to remove the first one.
Created attachment 286609 [details] Target editor without icon
New Gerrit change created: https://git.eclipse.org/r/c/pde/eclipse.pde.ui/+/182106
@Lars I agree that two [practically] identical icons could confuse. But why you want to remove the first one (on the left)? Please have a look on other PDE editors: I can see the icon on the left for many (all?) of them. I would rather suggest to change the right icon to be more expressive or replace this right icon with the hyperlink (gives more chances to hit it from the first click :) )
Created attachment 286614 [details] Silly looking icon Hi Alexander, thanks for the input. Tbh, I think the icons could be removed in all editors as they "only" repeat the icons from the editor tab. WDYT? Is this icon repetion useful to the user?
(In reply to Lars Vogel from comment #4) > Created attachment 286614 [details] > Silly looking icon > > Hi Alexander, thanks for the input. Tbh, I think the icons could be removed > in all editors as they "only" repeat the icons from the editor tab. > > WDYT? Is this icon repetion useful to the user? Well, I can't tell for every user, I can only share my personal experience: after using PDE editors for the last 20 years (OMG), I didn't think until today that there is this left [repeated] icon there. But, generally, [not silly looking] icons make products more quite.
(In reply to Alexander Fedorov from comment #5) > Well, I can't tell for every user, I can only share my personal experience: > after using PDE editors for the last 20 years (OMG), I didn't think until > today that there is this left [repeated] icon there. Same for me, I never (actively) noticed the icon. I think we should remove them, as they "only" repeat the editor icon and use unnecessary screenspace. I will open a new bug for the other editors.
Gerrit change https://git.eclipse.org/r/c/pde/eclipse.pde.ui/+/182106 was merged to [master]. Commit: http://git.eclipse.org/c/pde/eclipse.pde.ui.git/commit/?id=7c38781ea6c387e9b5375daccbbc308a96f532f9