diff options
author | Campbell Barton <ideasman42@gmail.com> | 2019-06-27 14:19:17 +0300 |
---|---|---|
committer | Campbell Barton <ideasman42@gmail.com> | 2019-06-27 14:20:29 +0300 |
commit | 7f5a21177a058eab33a2ba1707ef3fc965a32cd7 (patch) | |
tree | d0b63de62b1193691d491996caed8f87b89a5c24 /source/blender | |
parent | 0f983e8540529025d27fd9197f3d6e95ed476d3d (diff) |
Cleanup: de-duplicate flag checks for button colors
Diffstat (limited to 'source/blender')
-rw-r--r-- | source/blender/editors/interface/interface_widgets.c | 49 |
1 files changed, 21 insertions, 28 deletions
diff --git a/source/blender/editors/interface/interface_widgets.c b/source/blender/editors/interface/interface_widgets.c index 6c9c36c0d7f..a6c8697ec2c 100644 --- a/source/blender/editors/interface/interface_widgets.c +++ b/source/blender/editors/interface/interface_widgets.c @@ -2526,22 +2526,27 @@ static void widget_state(uiWidgetType *wt, int state, int drawflag) wt->wcol = *(wt->wcol_theme); + const char *color_blend = NULL; + if (drawflag & UI_BUT_ANIMATED_CHANGED) { + color_blend = wcol_state->inner_changed_sel; + } + else if (state & UI_BUT_ANIMATED_KEY) { + color_blend = wcol_state->inner_key_sel; + } + else if (state & UI_BUT_ANIMATED) { + color_blend = wcol_state->inner_anim_sel; + } + else if (state & UI_BUT_DRIVEN) { + color_blend = wcol_state->inner_driven_sel; + } + else if (state & UI_BUT_OVERRIDEN) { + color_blend = wcol_state->inner_overridden_sel; + } + if (state & UI_SELECT) { copy_v4_v4_char(wt->wcol.inner, wt->wcol.inner_sel); - if (drawflag & UI_BUT_ANIMATED_CHANGED) { - widget_state_blend(wt->wcol.inner, wcol_state->inner_changed_sel, wcol_state->blend); - } - else if (state & UI_BUT_ANIMATED_KEY) { - widget_state_blend(wt->wcol.inner, wcol_state->inner_key_sel, wcol_state->blend); - } - else if (state & UI_BUT_ANIMATED) { - widget_state_blend(wt->wcol.inner, wcol_state->inner_anim_sel, wcol_state->blend); - } - else if (state & UI_BUT_DRIVEN) { - widget_state_blend(wt->wcol.inner, wcol_state->inner_driven_sel, wcol_state->blend); - } - else if (state & UI_BUT_OVERRIDEN) { - widget_state_blend(wt->wcol.inner, wcol_state->inner_overridden_sel, wcol_state->blend); + if (color_blend != NULL) { + widget_state_blend(wt->wcol.inner, color_blend, wcol_state->blend); } copy_v3_v3_char(wt->wcol.text, wt->wcol.text_sel); @@ -2551,20 +2556,8 @@ static void widget_state(uiWidgetType *wt, int state, int drawflag) } } else { - if (drawflag & UI_BUT_ANIMATED_CHANGED) { - widget_state_blend(wt->wcol.inner, wcol_state->inner_changed, wcol_state->blend); - } - else if (state & UI_BUT_ANIMATED_KEY) { - widget_state_blend(wt->wcol.inner, wcol_state->inner_key, wcol_state->blend); - } - else if (state & UI_BUT_ANIMATED) { - widget_state_blend(wt->wcol.inner, wcol_state->inner_anim, wcol_state->blend); - } - else if (state & UI_BUT_DRIVEN) { - widget_state_blend(wt->wcol.inner, wcol_state->inner_driven, wcol_state->blend); - } - else if (state & UI_BUT_OVERRIDEN) { - widget_state_blend(wt->wcol.inner, wcol_state->inner_overridden, wcol_state->blend); + if (color_blend != NULL) { + widget_state_blend(wt->wcol.inner, color_blend, wcol_state->blend); } if (state & UI_ACTIVE) { /* mouse over? */ |