Hide checkbox icons in pie menu's sub-layouts
AbandonedPublic

Authored by Aleksandr Zinovev (raa) on Aug 12 2017, 2:37 PM.

Details

Summary

Toggle button's color should be enough to show it's state.
Looks safe for 2.79

Before and After:

Run this script in Blender to test the patch:

import bpy

class TestPie(bpy.types.Menu):
    bl_label = "Test Pie"

    def draw(self, context):
        layout = self.layout.menu_pie()
        layout.separator()
        layout.separator()
        layout.prop(context.space_data.text, "use_module", toggle=True)
        
        col = layout.column(True)
        col.scale_y = 1.5
        col.prop(context.space_data.text, "use_module", toggle=True)


bpy.utils.register_class(TestPie)
bpy.ops.wm.call_menu_pie(name="TestPie")

Diff Detail

Repository
rB Blender

Not sure about that one… yes, there is color change, but still, entry does not look like a standard toggle button in default theme (looks flat, among other things), so am inclined to believe we should keep checkbox.

That more like (minor) UI design issue, than bug, actually.

Agree, better to use a checkbox here to be clear. Don't think the drawing is clear enough really.

Going ahead and abandoning this patch. If anybody has a different opinion, please speak up. Thanks for the patch nevertheless.