[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [51623] trunk/blender: Camera: bring Title Safe display up to date, making it a Safe Areas option that

Brecht Van Lommel brechtvanlommel at pandora.be
Thu Oct 25 15:31:52 CEST 2012


Revision: 51623
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=51623
Author:   blendix
Date:     2012-10-25 13:31:45 +0000 (Thu, 25 Oct 2012)
Log Message:
-----------
Camera: bring Title Safe display up to date, making it a Safe Areas option that
shows both title safe and action safe areas following more modern standards.

Patch #32822 by Harley Acheson, full description:

Our current "title safe" camera display option is anachronistic. It shows a
border of 10% on all edges, which used to be the recommended title safe area
for 4:3 content on standard definition CRT televisions. However we are very
unlikely to create new projects that output for SD TV at that aspect ratio.

This patch change the option to "safe areas" with and indicates the
"title safe" area (also known as "graphic safe") as well as the "action safe"
area. "Title Safe" is an area visible by all reasonably maintained sets, where
text was certain not to be cut off. "Action Safe" is a larger area that
represented where a "perfect" set (with high precision to allow less
overscanning) would cut the image off.

The current recommendation for Action Safe is 3.5% on all edges, which is the
maxiumum overscan for TVs now. The recommended title safe is now 5% vertically
and 10% horizontally for content that is of wider aspect ratio than 4:3. The
reason for the difference between horizontal versus vertical margin is because
wider content would be letterboxed on an older 4:3 television, giving it
additional margin.

Modified Paths:
--------------
    trunk/blender/release/scripts/startup/bl_ui/properties_data_camera.py
    trunk/blender/source/blender/editors/space_view3d/view3d_draw.c
    trunk/blender/source/blender/makesrna/intern/rna_camera.c

Modified: trunk/blender/release/scripts/startup/bl_ui/properties_data_camera.py
===================================================================
--- trunk/blender/release/scripts/startup/bl_ui/properties_data_camera.py	2012-10-25 13:12:37 UTC (rev 51622)
+++ trunk/blender/release/scripts/startup/bl_ui/properties_data_camera.py	2012-10-25 13:31:45 UTC (rev 51623)
@@ -175,7 +175,7 @@
         col = split.column()
         col.prop(cam, "show_limits", text="Limits")
         col.prop(cam, "show_mist", text="Mist")
-        col.prop(cam, "show_title_safe", text="Title Safe")
+        col.prop(cam, "show_title_safe", text="Safe Areas")
         col.prop(cam, "show_sensor", text="Sensor")
         col.prop(cam, "show_name", text="Name")
 

Modified: trunk/blender/source/blender/editors/space_view3d/view3d_draw.c
===================================================================
--- trunk/blender/source/blender/editors/space_view3d/view3d_draw.c	2012-10-25 13:12:37 UTC (rev 51622)
+++ trunk/blender/source/blender/editors/space_view3d/view3d_draw.c	2012-10-25 13:31:45 UTC (rev 51623)
@@ -1060,7 +1060,7 @@
 
 static void drawviewborder(Scene *scene, ARegion *ar, View3D *v3d)
 {
-	float fac, a;
+	float fac, hmargin, vmargin;
 	float x1, x2, y1, y2;
 	float x1i, x2i, y1i, y2i;
 	float x3, y3, x4, y4;
@@ -1214,20 +1214,15 @@
 		}
 
 		if (ca->flag & CAM_SHOWTITLESAFE) {
-			fac = 0.1;
-
-			a = fac * (x2 - x1);
-			x1 += a;
-			x2 -= a;
-
-			a = fac * (y2 - y1);
-			y1 += a;
-			y2 -= a;
-
 			UI_ThemeColorBlendShade(TH_WIRE, TH_BACK, 0.25, 0);
 
-			uiSetRoundBox(UI_CNR_ALL);
-			uiDrawBox(GL_LINE_LOOP, x1, y1, x2, y2, 12.0);
+			hmargin = 0.1 * (x2 - x1);
+			vmargin = 0.05 * (y2 - y1);
+			uiDrawBox(GL_LINE_LOOP, x1+hmargin, y1+vmargin, x2-hmargin, y2-vmargin, 2.0);
+
+			hmargin = 0.035 * (x2 - x1);
+			vmargin = 0.035 * (y2 - y1);
+			uiDrawBox(GL_LINE_LOOP, x1+hmargin, y1+vmargin, x2-hmargin, y2-vmargin, 2.0);
 		}
 		if (ca && (ca->flag & CAM_SHOWSENSOR)) {
 			/* determine sensor fit, and get sensor x/y, for auto fit we

Modified: trunk/blender/source/blender/makesrna/intern/rna_camera.c
===================================================================
--- trunk/blender/source/blender/makesrna/intern/rna_camera.c	2012-10-25 13:12:37 UTC (rev 51622)
+++ trunk/blender/source/blender/makesrna/intern/rna_camera.c	2012-10-25 13:31:45 UTC (rev 51623)
@@ -258,7 +258,7 @@
 
 	prop = RNA_def_property(srna, "show_title_safe", PROP_BOOLEAN, PROP_NONE);
 	RNA_def_property_boolean_sdna(prop, NULL, "flag", CAM_SHOWTITLESAFE);
-	RNA_def_property_ui_text(prop, "Show Title Safe", "Show indicators for the title safe zone in Camera view");
+	RNA_def_property_ui_text(prop, "Show Safe Areas", "Show TV title safe and action safe zones in Camera view");
 	RNA_def_property_update(prop, NC_OBJECT | ND_DRAW, NULL);
 
 	prop = RNA_def_property(srna, "show_name", PROP_BOOLEAN, PROP_NONE);




More information about the Bf-blender-cvs mailing list