Hi, On 25/03/2013 00:56, Tamito KAJIYAMA wrote: > [...] > BTW, an UI glitch I pointed out in the code review is not fixed yet. > I have replied to your review comment concerning the UI issue. Let me quote > the comment and my reply here: > > https://codereview.appspot.com/7416049/diff/1/release/scripts/startup/bl_ui/space_view3d.py#newcode2508 > >> irieshinsuke 2013/03/21 12:39:58 >> >> Horizontally split row is too narrow to display labels "Freestyle Edge Marks" >> and "Freestyle Face Marks", so both of them are truncated to "Freestyle". I >> think "col = layout.column()" should be inserted here to avoid this glitch. >> >> me 2013/03/23 22:46:08 >> >> I am not fully sure if this change should be made. Any UI label will be >> truncated when the panel width is narrower than the label width. The >> Freestyle >> Edge/Face Mark labels in question can be completely shown if the panel is >> widen. Brecht told me that UI controls should be organized in two columns in >> general, and the Overlays section exactly follows this rule. I tend to be >> reluctant to violate it here. > I am fully open to any decision in this regard and will do whatever required > as long as this issue is a merger stopper. > > With best regards, >
Maybe we could simply shorten the names to something like "Edge FMark" and "Face FMark" ? The tooltips are here to be more explanatory anyway… Also, I would move "Seams" to the first column, when freestyle is enabled. Best regards, Bastien _______________________________________________ Bf-committers mailing list [email protected] http://lists.blender.org/mailman/listinfo/bf-committers
