[Bf-committers] [Bf-blender-cvs] SVN commit: /data/svn/bf-blender [55940] trunk/blender/release/scripts/ startup/bl_ui/properties_freestyle.py: Added to the Freestyle panel in the Render buttons a button to switch to the Render Layers context .

[Tamito KAJIYAMA]

Hi Thomas,

I fully agree with you that this kind of irregular UI solutions should be
limited to a very few cases, since otherwise we would end up having
many similar buttons to change properties window contexts.

I also see the point made by the original issue report [#34934], and
as I wrote in a reply this is a compromise resulting from a trade-off
between space consumption and user guidance.

Note that the added label and context switch button are within the
Freestyle tab in the Render buttons.  When the Freestyle tab is closed
we don't lose anything, compared to the situation before the additions
of the UI items.

I am open to any other possible solution in this regard.  I would much
appreciate additional discussions.

Regards,

-- 
KAJIYAMA, Tamito <rd6t-kjym at asahi-net.or.jp>


-----Original Message----- 
From: Thomas Dinges
Sent: Wednesday, April 10, 2013 9:32 PM
To: bf-committers at blender.org
Subject: Re: [Bf-committers] [Bf-blender-cvs] SVN commit: /data/svn/bf-blender [55940] trunk/blender/release/scripts/ 
startup/bl_ui/properties_freestyle.py: Added to the Freestyle panel in the Render buttons a button to switch to the Render Layers 
context .

Hi Tamito,
I am not so happy with this solution.

The only place where we use this, is the modifier stack at the moment.
One could argue, that such a button or notice could be added to a lot
more places. Imho, we should just document where the settings are, that
should be fine.

I understand that for long time Freestyle users this might be a bit
confusing atm, but on the other hand we should really avoid adding such
special cases in the UI to jump from one place to another, just because
all settings are not in 1 place.

Best regards,
Thomas

Am 10.04.2013 22:20, schrieb Tamito Kajiyama:
> Revision: 55940
>            http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=55940
> Author:   kjym3
> Date:     2013-04-10 20:20:20 +0000 (Wed, 10 Apr 2013)
> Log Message:
> -----------
> Added to the Freestyle panel in the Render buttons a button to switch to the Render Layers context.
>
> Modified Paths:
> --------------
>      trunk/blender/release/scripts/startup/bl_ui/properties_freestyle.py
>
> Modified: trunk/blender/release/scripts/startup/bl_ui/properties_freestyle.py
> ===================================================================
> --- trunk/blender/release/scripts/startup/bl_ui/properties_freestyle.py 2013-04-10 20:05:11 UTC (rev 55939)
> +++ trunk/blender/release/scripts/startup/bl_ui/properties_freestyle.py 2013-04-10 20:20:20 UTC (rev 55940)
> @@ -60,7 +60,9 @@
>           row.active = (rd.line_thickness_mode == 'ABSOLUTE')
>           row.prop(rd, "line_thickness")
>
> -        layout.label(text="Line style settings are found in the Render Layers context")
> +        row = layout.row()
> +        row.label(text="Line style settings are found in the Render Layers context")
> +        row.operator("wm.properties_context_change", text="", icon='BUTS').context = 'RENDER_LAYER'
>
>
>   # Render layer properties
>
> _______________________________________________
> Bf-blender-cvs mailing list
> Bf-blender-cvs at blender.org
> http://lists.blender.org/mailman/listinfo/bf-blender-cvs


-- 
Thomas Dinges
Blender Developer, Artist and Musician

www.dingto.org

_______________________________________________
Bf-committers mailing list
Bf-committers at blender.org
http://lists.blender.org/mailman/listinfo/bf-committers