From ee4e6cc91c447975ca118b5a006709aefceba67c Mon Sep 17 00:00:00 2001 From: Hakan Seven Date: Wed, 8 Jul 2026 16:31:37 +0300 Subject: [PATCH] refactor(properties): Color above Layer; drop the Invisible toggle MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Reorder the General group so Color precedes Layer. Remove the "Invisible" row: the entity visibility flag (DXF code 60) is driven by dynamic-block visibility states, not a manual per-object toggle — setting it from the panel hid the object with no easy way to reselect it. The flag itself is unchanged (dynamic blocks still set it, render/save still honour it, and it stays settable programmatically). Co-Authored-By: Claude Opus 4.8 --- src/scene/cache/properties.rs | 19 +++++-------------- 1 file changed, 5 insertions(+), 14 deletions(-) diff --git a/src/scene/cache/properties.rs b/src/scene/cache/properties.rs index 5e7293819..59ff13d46 100644 --- a/src/scene/cache/properties.rs +++ b/src/scene/cache/properties.rs @@ -31,16 +31,16 @@ pub fn general_section(entity: &EntityType) -> PropSection { field: "handle", value: PropValue::ReadOnly(common.handle.value().to_string()), }, - Property { - label: "Layer".into(), - field: "layer", - value: PropValue::LayerChoice(common.layer.clone()), - }, Property { label: "Color".into(), field: "color", value: PropValue::ColorChoice(common.color), }, + Property { + label: "Layer".into(), + field: "layer", + value: PropValue::LayerChoice(common.layer.clone()), + }, Property { label: "Linetype".into(), field: "linetype", @@ -90,15 +90,6 @@ pub fn general_section(entity: &EntityType) -> PropSection { .push(crate::entities::common::edit_prop("Thickness", "thickness", t)); } - section.props.push(Property { - label: "Invisible".into(), - field: "invisible", - value: PropValue::BoolToggle { - field: "invisible", - value: common.invisible, - }, - }); - section }