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 <[email protected]> -----Original Message----- From: Thomas Dinges Sent: Wednesday, April 10, 2013 9:32 PM To: [email protected] 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 > [email protected] > http://lists.blender.org/mailman/listinfo/bf-blender-cvs -- Thomas Dinges Blender Developer, Artist and Musician www.dingto.org _______________________________________________ Bf-committers mailing list [email protected] http://lists.blender.org/mailman/listinfo/bf-committers _______________________________________________ Bf-committers mailing list [email protected] http://lists.blender.org/mailman/listinfo/bf-committers
