[FRIO] highlighting effect when calling comment with dark themes too bright

[loma-one]

The coloured highlighting when calling up a comment via the bell is a good function for quickly finding the position.
However, in a dark theme and when the eyes have become accustomed to the darkness, it feels like a laser beam trying to burn out the eyes.

The highlighting is controlled at this point:

friendica/view/theme/frio/js/theme.js

Line 743 in 31b74b1

// Define the colors which are used for highlighting

Question:

• Is there a way to control the effect in a theme via a CSS instruction?
• Alternatively, is it possible to control this effect separately for light and dark themes?

https://loma.ml/display/373ebf56-1166-031a-53cd-b2e686160001

[MrPetovan]

For these to be depending on the theme, the declaration of the variable would have to be in a place that is controlled by the theme. We have some JavaScript variables initialized in template files, however there aren't any provisions to pass scheme-defined variables to any template at the moment, so I'm a little stumped.

[loma-one]

Perhaps it is already sufficient if a value is stored at the relevant code point that represents a compromise for light and dark themes.

I have done it with the values

	// Define the colors which are used for highlighting
	var colWhite = { backgroundColor: "#7f7f7f" };
	var colShiny = { backgroundColor: "#7e763a" };

and it seems to be more pleasant. However, I am sure that the choice of colour can be better.

[MrPetovan]

This is a good first step, however finding values that work in all three official schemes (light, dark and black) is a challenge. Any improvement is welcome though. Would you mind posting screenshots of the new colors in all three schemes?

[loma-one]

This is of course a flowing colour gradient and the effect looks friendlier than shown here.
In principle, only the brightness peaks have been removed here.

[MrPetovan]

Looks fine to me!

[AlfredSK]

World it be possible to do the highlighting with a flashing colored margin instead?

[MrPetovan]

Yes but it would be more work than just tuning the colors of the current highlighting.