Page MenuHome

Outline color for selected objects
Closed, ResolvedPublicDESIGN

Description

Instances in an object currently have a different outline color. This can be confusing now, because sometimes an object only consists of instances in its evaluated state. That makes it hard to see which object is active.

Event Timeline

Hi Jacques, I was considering working on this too (unless you wanted to), but I was wondering if it'd cause confusion having the instances also share the same color as the original. How would people tell them apart?

Jacques Lucke (JacquesLucke) changed the subtype of this task from "To Do" to "Design".Nov 1 2021, 11:58 AM

Guess that's more of a design task for now.
Personally, I'm not convinced that people have to tell instances apart in the viewport. For me they are just part of the final object. Whether the output is a mesh or a single instance that contains a mesh seems irrelevant in the viewport.

Here is a start to this solution. It doesn't make make geometry components quite as bright as the base component type though.

diff --git a/source/blender/draw/intern/draw_common.c b/source/blender/draw/intern/draw_common.c
index 62d715460bb..a4d98cf352d 100644
--- a/source/blender/draw/intern/draw_common.c
+++ b/source/blender/draw/intern/draw_common.c
@@ -101,10 +101,10 @@ void DRW_globals_update(void)
       gb->colorEditMeshMiddle,
       dot_v3v3(gb->colorEditMeshMiddle, (float[3]){0.3333f, 0.3333f, 0.3333f})); /* Desaturate */
 
-  interp_v4_v4v4(gb->colorDupliSelect, gb->colorBackground, gb->colorSelect, 0.5f);
+  copy_v4_v4(gb->colorDupliSelect, gb->colorSelect);
   /* Was 50% in 2.7x since the background was lighter making it easier to tell the color from
    * black, with a darker background we need a more faded color. */
-  interp_v4_v4v4(gb->colorDupli, gb->colorBackground, gb->colorWire, 0.3f);
+  copy_v4_v4(gb->colorDupli, gb->colorWire);
 
 #ifdef WITH_FREESTYLE
   UI_GetThemeColor4fv(TH_FREESTYLE_EDGE_MARK, gb->colorEdgeFreestyle);