Hi,

I found that in Linux Mint Cinnamon dark theme (Mint Y Dark Aqua), the drop down menu selection coloring was fixed to use white text with dark gray selection bar, but the main menu still uses black text with dark gray selection bar. Can this be fixed to use white text too?

UPDATE: More: For some reason, with light themes, U++ is using the 'colorization' color for main menu selection bar, whereas all the other applications tend to use the same colors as the drop down menu. I think setting the main menu to use all the same colors as the drop down menu would be the best choice here.

Hi Mirek,

Hope this helps...

With default light theme:

s.look = Image (32, 16)
dk = Color(48, 48, 48)
wh = Color(255, 255, 255)
After IsDark(wh):
dk = Color(48, 48, 48)
wh = Color(255, 255, 255)
s.topitemtext[0] = Color(48, 48, 48)
s.topitemtext[1] = Color(48, 48, 48)
s.topitemtext[2] = Color(33, 33, 33)
s.topitem[0] = 
s.topitem[1] = 
s.topitem[2] = Image (32, 16)
After Diff():
s.topitemtext[0] = Color(48, 48, 48)
s.topitemtext[1] = Color(48, 48, 48)
s.topitemtext[2] = Color(255, 255, 255)
s.topitem[0] = 
s.topitem[1] = 
s.topitem[2] = Color(53, 168, 84)

And dark theme:

s.look = Image (32, 16)
dk = Color(218, 218, 218)
wh = Color(64, 64, 64)
After IsDark(wh):
dk = Color(64, 64, 64)
wh = Color(218, 218, 218)
s.topitemtext[0] = Color(218, 218, 218)
s.topitemtext[1] = Color(218, 218, 218)
s.topitemtext[2] = Color(7, 7, 7)
s.topitem[0] = 
s.topitem[1] = 
s.topitem[2] = Image (32, 16)
After Diff():
s.topitemtext[0] = Color(218, 218, 218)
s.topitemtext[1] = Color(218, 218, 218)
s.topitemtext[2] = Color(7, 7, 7)
s.topitem[0] = 
s.topitem[1] = 
s.topitem[2] = Image (32, 16)

Best regards,

Tom

Hi,

For light theme:

AvgColor(s.topitem[2], SColorFace(), 0) = Color(0, 0, 0)
Diff(AvgColor(s.topitem[2], SColorFace(), 0), s.topitemtext[2]) = 33
SColorHighlight() = Color(53, 168, 84)
SColorHighlightText() = Color(255, 255, 255)

For dark theme:

AvgColor(s.topitem[2], SColorFace(), 0) = Color(245, 245, 245)
Diff(AvgColor(s.topitem[2], SColorFace(), 0), s.topitemtext[2]) = 238
SColorHighlight() = Color(31, 158, 222)
SColorHighlightText() = Color(255, 255, 255)

UPDATE: These are dumped just before if(Diff(...

Best regards,

Tom

Hi,

Here is a crappy shrinked photo attached... cannot take screenshot of the menu as it disappears! Anyway, you will get the picture.

BR, Tom

Hi,

This works here for both light and dark themes:

		s.topitemtext[2] = GetInkColor();
		Over(m, topitem);
		if(Diff(AvgColor(m), s.topitemtext[2]) < 100) {
			//s.topitem[2] = SColorHighlight(); // <<< Commenting this out helps!
			s.topitemtext[2] = SColorHighlightText();
		}
	}

Best regards,

Tom

Also please test with theide too - that has slightly different menu styling required.

mirek wrote on Tue, 23 May 2023 16:23

Also please test with theide too - that has slightly different menu styling required.

Thanks!
Everything seems to work just fine here with Linux Mint and many different themes. Also with TheIDE.

Is there any specific testing requirements for TheIDE, I should try?