Bug 343031 - There is no icon for java stack trace console
Summary: There is no icon for java stack trace console
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Debug (show other bugs)
Version: 3.7   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 4.10 M3   Edit
Assignee: Sarika Sinha CLA
QA Contact:
URL:
Whiteboard:
Keywords: bugday, helpwanted
Depends on:
Blocks:
 
Reported: 2011-04-16 03:36 EDT by Ewgeni Grigorov CLA
Modified: 2018-11-19 23:07 EST (History)
4 users (show)

See Also:


Attachments
an initial icon suggestion in grey scale (357 bytes, image/gif)
2011-04-16 03:38 EDT, Ewgeni Grigorov CLA
no flags Details
java stack trace console with icon (8.31 KB, image/png)
2017-12-27 15:19 EST, Andrey Loskutov CLA
no flags Details
svg file (1.49 KB, image/svg+xml)
2018-10-08 08:28 EDT, Sarika Sinha CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Ewgeni Grigorov CLA 2011-04-16 03:36:19 EDT
Build Identifier: CVS, 16.04.2011

The icon is not available in "Open Console".

Reproducible: Always
Comment 1 Ewgeni Grigorov CLA 2011-04-16 03:38:18 EDT
Created attachment 193412 [details]
an initial icon suggestion in grey scale
Comment 2 Andrey Loskutov CLA 2017-12-27 15:14:03 EST
Ewgeni, if you are still interested, please update your icon to something matching other Eclipse icons L&F.

Meanwhile I believe that we have already a matching icon (used for exception breakpoints), which I would like to set to the console. Having no icon doesn't look nice, and with the icon it is much easier to find. See attachment 272047 [details] 
 to bug 529248 how it looks today.
Comment 3 Eclipse Genie CLA 2017-12-27 15:17:47 EST
New Gerrit change created: https://git.eclipse.org/r/114774
Comment 4 Andrey Loskutov CLA 2017-12-27 15:19:12 EST
Created attachment 272050 [details]
java stack trace console with icon

(In reply to Eclipse Genie from comment #3)
> New Gerrit change created: https://git.eclipse.org/r/114774

The attached picture shows how it looks like after the patch.
Comment 6 Eclipse Genie CLA 2017-12-28 03:39:44 EST
New Gerrit change created: https://git.eclipse.org/r/114790
Comment 8 Sarika Sinha CLA 2018-01-03 04:42:21 EST
The new icon does not have any connection with console. If we don't have a new icon we can at least have  the console icon with "J" from the Java Exception icon instead of simply reusing Java exception icon.
Comment 9 Andrey Loskutov CLA 2018-01-03 04:56:46 EST
(In reply to Sarika Sinha from comment #8)
> The new icon does not have any connection with console. If we don't have a
> new icon we can at least have  the console icon with "J" from the Java
> Exception icon instead of simply reusing Java exception icon.

I thought to add something like this but then decided against it. 

The context of all the menu entries is the console. There is no need to emphasize this again in the icon, especially because any attempt to add some extras to the main "console" icon part will result in about 5 x 5 pixel small "something", like OSGI console icon. So we will end with few indistinguishable console icons, similar to the OSGI console icon, and this does not make much sense for me. Instead, being already in the context of the console, we can emphasize what is shown *inside* the opened page, and this is in the "java stack trace" console the exception stack trace. So I've ended up with the "J!" icon shown for Java exceptions. Of course we could use something else for the icon, but I would really do not recommend to add a "console" to it.
Comment 10 Sarika Sinha CLA 2018-01-03 05:13:11 EST
Java Stack should not be directly associated to Java exception. We can paste any stack trace, It might be just coming from  a log or to understand the flow.

I see that OSGI icon does not look clear but CVS one is very clear and we should have all the icons in a similar style.
Comment 11 Sarika Sinha CLA 2018-01-04 21:59:45 EST
If we don't have a proper icon before M5, we should revert this. As we can not change this multiple times and confuse user.
Comment 12 Eclipse Genie CLA 2018-01-05 04:29:11 EST
New Gerrit change created: https://git.eclipse.org/r/114978
Comment 13 Andrey Loskutov CLA 2018-01-05 04:32:03 EST
(In reply to Sarika Sinha from comment #11)
> If we don't have a proper icon before M5, we should revert this. As we can
> not change this multiple times and confuse user.

(In reply to Eclipse Genie from comment #12)
> New Gerrit change created: https://git.eclipse.org/r/114978

I've explained my thoughts in comment 9, so I will revert the changes.

If someone wants to add a new icon: see the patch above where it should be configured.
Comment 15 Michael Keppler CLA 2018-08-14 04:06:25 EDT
What about reusing the stackframe image from junit? http://git.eclipse.org/c/jdt/eclipse.jdt.ui.git/plain/org.eclipse.jdt.junit/icons/full/eview16/stackframe.png

Regarding whether to have the "screen" in all console icons: At least in my Eclipse installation the majority of available consoles does _not_ have this screen (e.g. m2e, spotbugs, xtext, ...). I tend to agree with Andrey, these icons are shown only in a single drop down menu where the context "console" is already completely clear. Those consoles also don't appear in Ctrl-3 searches (which would be an argument for showing the generic console background in the icon).
Comment 16 Andrey Loskutov CLA 2018-10-03 04:03:22 EDT
(In reply to Michael Keppler from comment #15)
> What about reusing the stackframe image from junit?
> http://git.eclipse.org/c/jdt/eclipse.jdt.ui.git/plain/org.eclipse.jdt.junit/
> icons/full/eview16/stackframe.png
> 
> Regarding whether to have the "screen" in all console icons: At least in my
> Eclipse installation the majority of available consoles does _not_ have this
> screen (e.g. m2e, spotbugs, xtext, ...). I tend to agree with Andrey, these
> icons are shown only in a single drop down menu where the context "console"
> is already completely clear. Those consoles also don't appear in Ctrl-3
> searches (which would be an argument for showing the generic console
> background in the icon).

Sarika, WDYT? I would be OK with the icon.
Comment 17 Sarika Sinha CLA 2018-10-04 04:40:05 EDT
(In reply to Andrey Loskutov from comment #16)
> (In reply to Michael Keppler from comment #15)
> > What about reusing the stackframe image from junit?
> > http://git.eclipse.org/c/jdt/eclipse.jdt.ui.git/plain/org.eclipse.jdt.junit/
> > icons/full/eview16/stackframe.png
> > 
> > Regarding whether to have the "screen" in all console icons: At least in my
> > Eclipse installation the majority of available consoles does _not_ have this
> > screen (e.g. m2e, spotbugs, xtext, ...). I tend to agree with Andrey, these
> > icons are shown only in a single drop down menu where the context "console"
> > is already completely clear. Those consoles also don't appear in Ctrl-3
> > searches (which would be an argument for showing the generic console
> > background in the icon).
> 
> Sarika, WDYT? I would be OK with the icon.

Let me work on having the overlay "J" icon  from JDT UI and overlay it on Console or the stackframe icon, let's see how does it come.

If anyone else wants to try any overlay, you can.

Does it sound ok?
Comment 18 Eclipse Genie CLA 2018-10-08 08:25:25 EDT
New Gerrit change created: https://git.eclipse.org/r/130612
Comment 19 Sarika Sinha CLA 2018-10-08 08:27:45 EDT
(In reply to Eclipse Genie from comment #18)
> New Gerrit change created: https://git.eclipse.org/r/130612

This is based on the initial icon design. Overlay will not work as it is provided in plugin.xml.

If this icon is acceptable, I am attaching the svg file and someone can help to generate 2x file from platform utility.
Comment 20 Sarika Sinha CLA 2018-10-08 08:28:37 EDT
Created attachment 276161 [details]
svg file
Comment 21 Michael Keppler CLA 2018-10-08 13:39:27 EDT
Seems okay to me. Funny enough however, only with your new icon I now realized there are 2 different kinds of similar console icons. Yours and the CVS console icon use a sharp rectangle in dark blue, new console and OSGI Host console use a rounded rectangle, light blue (like the console view itself). I'm happy with all of those. :)
Comment 22 Andrey Loskutov CLA 2018-10-08 17:43:18 EDT
I personally do not like the icon (reason already explained) but I'm for a progress on this bug, just because no icon is worse than some one.
Comment 24 Eclipse Genie CLA 2018-11-19 05:57:09 EST
New Gerrit change created: https://git.eclipse.org/r/132663
Comment 25 Sarika Sinha CLA 2018-11-19 06:06:04 EST
Released svg and png files.
Comment 27 Sarika Sinha CLA 2018-11-19 23:07:28 EST
Verified using
Eclipse SDK

Version: 2018-12 (4.10)
Build id: I20181119-0600