Page MenuHome

Patch for T57974 : Remove background fade and keep them longer
Needs ReviewPublic

Authored by Lucas Boutrot (thornydre) on Sun, Nov 25, 12:07 AM.

Details

Summary

Hi,

I gave this one a try, hope this is fine. Now the background stays until the message disappears.

One thing I found out by testing is that the warning color makes the message unreadable :

If you want I can try to change the background color, or the font color ?

Cheers !

Diff Detail

Repository
rB Blender

Event Timeline

I was not sure but I think all this can be removed as well.

Interesting. Yes, the readability is an issue. I guess we should update the theme to fix this issue. @Pablo Vazquez (pablovazquez): Will you look into the theme thing here?

The text just uses the text color from the statusbar. The simplest solution would be to make the info/warning/error colors darker as in the task, and hardcode the text color to be white. Alternatively we could add all of those colors to the theme, but I'm not sure it's worth it.

@Brecht Van Lommel (brecht) If the text color was to be hardcoded to anything I feel like it should be hardcoded to be black, which stands out on more colors that white would.

@William Reynish (billreynish) So should I hardcode the text color for this one ?

Yes, black should be readable against most colors. I'm not se keen on hardcoding colors though - I wonder if there's a theme color we can use here instead. Well, for now a hardcoded black color would be ok - we can replace it with something else if it's a problem.

I can try adding a theme color for the report text color maybe ? But I'll try to find if there is another one available first

Can't we just use these colors if we don't already?

If text is then not readable, we can just update the default theme.