Fix assertion failure when editing a light entity which has _color va…

…lues specified as integers 0-255
jeroenjanssens · Aug 6, 2014 · f77937a · f77937a
1 parent 2ecd7bd
commit f77937a
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/Source/View/ColorEditor.cpp b/Source/View/ColorEditor.cpp
@@ -143,7 +143,15 @@ namespace TrenchBroom {
                 const Model::Entity& entity = **entityIt;
                 const Model::PropertyValue* value = entity.propertyForKey(property());
                 if (value != NULL)
-                    colorSet.insert(Vec3f(*value));
+				{
+					Vec3f color = Vec3f(*value);
+
+					// YIQOrder requires color values in the range [0..1]
+					if (color.x() > 1.0f || color.y() > 1.0f || color.z() > 1.0f)
+						color /= 255.0f;
+
+                    colorSet.insert(color);
+				}
             }
 
             m_colorHistory->setColors(Utility::makeList(colorSet));