[Bf-blender-cvs] [e386d627f6c] greasepencil-object: Cleanup: Remove duplicate code

Antonio Vazquez noreply at git.blender.org
Mon Jul 16 09:33:25 CEST 2018


Commit: e386d627f6c0b9bd4ff6da6d3e470fc087d23fe0
Author: Antonio Vazquez
Date:   Mon Jul 16 09:32:54 2018 +0200
Branches: greasepencil-object
https://developer.blender.org/rBe386d627f6c0b9bd4ff6da6d3e470fc087d23fe0

Cleanup: Remove duplicate code

===================================================================

M	release/scripts/startup/bl_ui/space_toolsystem_toolbar.py

===================================================================

diff --git a/release/scripts/startup/bl_ui/space_toolsystem_toolbar.py b/release/scripts/startup/bl_ui/space_toolsystem_toolbar.py
index 099e9b6fb35..3c860d7e1cb 100644
--- a/release/scripts/startup/bl_ui/space_toolsystem_toolbar.py
+++ b/release/scripts/startup/bl_ui/space_toolsystem_toolbar.py
@@ -986,16 +986,24 @@ class _defs_uv_select:
         )
 
 class _defs_gpencil_paint:
+    @classmethod
+    def draw_color_selector(cls, context, layout):
+        brush = context.active_gpencil_brush
+        gp_settings = brush.gpencil_settings
+        ts = context.tool_settings
+        row = layout.row(align=True)
+        row.prop(ts, "use_gpencil_compact_list", text="", icon="SORTSIZE")
+        if ts.use_gpencil_compact_list is True:
+            row.template_ID(gp_settings, "material")
+        else:
+            row.template_greasepencil_color(gp_settings, "material", rows=3, cols=8, scale=0.8)
+
     @classmethod
     def draw_settings_common(cls, context, layout, tool):
         ob = context.active_object
-        nrow = 3
-        ncol = 8
-        scale = 0.80
         if ob and ob.mode == 'GPENCIL_PAINT':
             brush = context.active_gpencil_brush
             gp_settings = brush.gpencil_settings
-            ts = context.tool_settings
 
             if gp_settings.gpencil_brush_type == 'ERASE':
                 row = layout.row()
@@ -1006,12 +1014,7 @@ class _defs_gpencil_paint:
                 row.prop(brush, "size", text="Thickness")
                 row.prop(gp_settings, "gpencil_fill_simplyfy_level", text="Simplify")
 
-                row = layout.row(align=True)
-                row.prop(ts, "use_gpencil_compact_list", text="", icon="SORTSIZE")
-                if ts.use_gpencil_compact_list is True:
-                    row.template_ID(gp_settings, "material")
-                else:
-                    row.template_greasepencil_color(gp_settings, "material", rows=nrow, cols=ncol, scale=scale)
+                _defs_gpencil_paint.draw_color_selector(context, layout)
 
                 row = layout.row(align=True)
                 row.prop(gp_settings, "gpencil_fill_draw_mode", text="")
@@ -1025,12 +1028,7 @@ class _defs_gpencil_paint:
                 row.prop(gp_settings, "pen_strength", slider=True)
                 row.prop(gp_settings, "use_strength_pressure", text="", icon='STYLUS_PRESSURE')
 
-                row = layout.row(align=True)
-                row.prop(ts, "use_gpencil_compact_list", text="", icon="SORTSIZE")
-                if ts.use_gpencil_compact_list is True:
-                    row.template_ID(gp_settings, "material")
-                else:
-                    row.template_greasepencil_color(gp_settings, "material", rows=nrow, cols=ncol, scale=scale)
+                _defs_gpencil_paint.draw_color_selector(context, layout)
 
     @staticmethod
     def generate_from_brushes(context):



More information about the Bf-blender-cvs mailing list