Page MenuHome

UI: Fix T77173: Report Background Colors for 2.83 Release
ClosedPublic

Authored by Hans Goudey (HooglyBoogly) on Jun 2 2020, 5:50 PM.

Details

Summary

This is a temporary solution for T77173 for the 2.83 release. D7203 provides
a more long term solution for future releases.

This adds theme colors for the three report backgrounds, setting them to the color
used in 2.82.

2.822.83Patch
Error
Warning
Info

Diff Detail

Repository
rB Blender

Event Timeline

Hans Goudey (HooglyBoogly) requested review of this revision.Jun 2 2020, 5:50 PM
Hans Goudey (HooglyBoogly) created this revision.

Did you check the bundled themes, does the new color work for these? I realize they were only added in 2.83, so there is no old color we can use.
Besides that this seems fine, although I have to test it to double check still.

Some of the white themes aren't great. Do you want me to include updates to those here?

I'm not actually sure I have commit access to the addons repo though.

Blender light:

diff --git a/release/scripts/presets/interface_theme/blender_light.xml b/release/scripts/presets/interface_theme/blender_light.xml
index f4ec8233df3..dbc7dd782da 100644
--- a/release/scripts/presets/interface_theme/blender_light.xml
+++ b/release/scripts/presets/interface_theme/blender_light.xml
@@ -1052,6 +1052,9 @@
         info_property_text="#ffffff"
         info_operator="#3ace87ff"
         info_operator_text="#ffffff"
+        info_report_error="#990000"
+        info_report_warning="#b36a00"
+        info_report_info="#6080ff"
         >
         <space>
           <ThemeSpaceGeneric

Other theme changes are here: P1442

Julian Eisel (Severin) accepted this revision.EditedJun 2 2020, 8:09 PM

Code looks fine. Testing it (but not every single theme), it also seems fine - well, at least as good as the old version, which is all we wanted :)

This revision is now accepted and ready to land.Jun 2 2020, 8:09 PM
Dalai Felinto (dfelinto) resigned from this revision.Jun 2 2020, 8:26 PM

I won't have for review here, but if Julian is ok, I'm ok too. Thanks Hans