UI: Re-order Properties tabs to follow the data hierarchy
authorWilliam Reynish <billrey@me.com>
Fri, 12 Apr 2019 10:04:59 +0000 (12:04 +0200)
committerWilliam Reynish <billrey@me.com>
Fri, 12 Apr 2019 10:04:59 +0000 (12:04 +0200)
- Group tabs together based on the data-flow
- Puts Physics and Particles next to Modifiers, with which they interrelate

See D4612

Reviewers: pablovazquez

source/blender/makesrna/intern/rna_space.c

index bc7aaa1..a3a55a7 100644 (file)
@@ -1331,10 +1331,6 @@ static const EnumPropertyItem *rna_SpaceProperties_context_itemf(
                RNA_enum_items_add_value(&item, &totitem, buttons_context_items, BCONTEXT_OBJECT);
        }
 
-       if (sbuts->pathflag & (1 << BCONTEXT_CONSTRAINT)) {
-               RNA_enum_items_add_value(&item, &totitem, buttons_context_items, BCONTEXT_CONSTRAINT);
-       }
-
        if (sbuts->pathflag & (1 << BCONTEXT_MODIFIER)) {
                RNA_enum_items_add_value(&item, &totitem, buttons_context_items, BCONTEXT_MODIFIER);
        }
@@ -1343,6 +1339,18 @@ static const EnumPropertyItem *rna_SpaceProperties_context_itemf(
                RNA_enum_items_add_value(&item, &totitem, buttons_context_items, BCONTEXT_SHADERFX);
        }
 
+       if (sbuts->pathflag & (1 << BCONTEXT_PARTICLE)) {
+               RNA_enum_items_add_value(&item, &totitem, buttons_context_items, BCONTEXT_PARTICLE);
+       }
+
+       if (sbuts->pathflag & (1 << BCONTEXT_PHYSICS)) {
+               RNA_enum_items_add_value(&item, &totitem, buttons_context_items, BCONTEXT_PHYSICS);
+       }
+
+       if (sbuts->pathflag & (1 << BCONTEXT_CONSTRAINT)) {
+               RNA_enum_items_add_value(&item, &totitem, buttons_context_items, BCONTEXT_CONSTRAINT);
+       }
+
        if (sbuts->pathflag & (1 << BCONTEXT_DATA)) {
                RNA_enum_items_add_value(&item, &totitem, buttons_context_items, BCONTEXT_DATA);
                (item + totitem - 1)->icon = sbuts->dataicon;
@@ -1360,16 +1368,12 @@ static const EnumPropertyItem *rna_SpaceProperties_context_itemf(
                RNA_enum_items_add_value(&item, &totitem, buttons_context_items, BCONTEXT_MATERIAL);
        }
 
-       if (sbuts->pathflag & (1 << BCONTEXT_TEXTURE)) {
-               RNA_enum_items_add_value(&item, &totitem, buttons_context_items, BCONTEXT_TEXTURE);
-       }
-
-       if (sbuts->pathflag & (1 << BCONTEXT_PARTICLE)) {
-               RNA_enum_items_add_value(&item, &totitem, buttons_context_items, BCONTEXT_PARTICLE);
+               if (totitem) {
+               RNA_enum_item_add_separator(&item, &totitem);
        }
 
-       if (sbuts->pathflag & (1 << BCONTEXT_PHYSICS)) {
-               RNA_enum_items_add_value(&item, &totitem, buttons_context_items, BCONTEXT_PHYSICS);
+       if (sbuts->pathflag & (1 << BCONTEXT_TEXTURE)) {
+               RNA_enum_items_add_value(&item, &totitem, buttons_context_items, BCONTEXT_TEXTURE);
        }
 
        RNA_enum_item_end(&item, &totitem);