Eclipse-Spectrum-Theme icon indicating copy to clipboard operation
Eclipse-Spectrum-Theme copied to clipboard

Heap status foreground colour is hard to read

Open ingomohr opened this issue 5 years ago • 4 comments

Screenshot 2020-06-12 at 00 11 50

Maybe we could give it some red and green(ish) as well :)

ingomohr avatar Jun 11 '20 22:06 ingomohr

Nice catch, I usually have the heap status hidden so I wouldn’t have noticed this :) good suggestion!

AObuchow avatar Jun 11 '20 22:06 AObuchow

Nice catch, I usually have the heap status hidden so I wouldn’t have noticed this :) good suggestion!

Yup, me too. I just switched to another workspace where I have it visible... and there it was ;)

ingomohr avatar Jun 11 '20 22:06 ingomohr

It seems that the colors used are hardcoded to some extent, see HeapStatus.java.

If you use a color picker on the HeapStatus used mem, it gives 160, 160, 160 in RGB values.

		usedMemCol = new Color(display, 160, 160, 160); // gray
		lowMemCol = new Color(display, 255, 70, 70); // medium red
		freeMemCol = new Color(display, 255, 190, 125); // light orange

Also the heap status looks the same no matter what Platform theme is used.

An upstream bug should be made about this.

AObuchow avatar Jun 12 '20 04:06 AObuchow

Upstream bug: https://bugs.eclipse.org/bugs/show_bug.cgi?id=564710

AObuchow avatar Jun 28 '20 03:06 AObuchow