Merge pull request 'B2024-007 - Added missing settings to Beaver Valley’s Transition “{Proc Num} (Procedure Number Only)” to fix a null reference error' (#234) from B2024-007_BeaverValley_Transition into Development

Reviewed-on: #234

PROMS/Volian.Controls.Library/DisplayTransition.cs

@@ -208,7 +208,7 @@ namespace Volian.Controls.Library
 			// set other procedure related private variables.  
 			vlnTreeComboSetsFillIn(_CurrentProcedure);
 			cbTranProcsFillIn(_CurrentProcedure);
-			if (_CurrentProcedure.Sections != null)
+			if (_CurrentProcedure.Sections != null && secitm != null)
 			{
 				// if the selected format is using section default, initialize with the section default (find it),
 				// otherwise use current section item.
@@ -987,6 +987,7 @@ namespace Volian.Controls.Library
 			if (prcitm.Sections != null)
 			{
 				foreach (SectionInfo sio in prcitm.Sections)
+				
 				{
 					SectionConfig sc = sio.MyConfig as SectionConfig;
 					if (sc != null && sc.Section_OriginalSteps == "Y")
@@ -1115,6 +1116,8 @@ namespace Volian.Controls.Library
 				this.superToolTipDispTran.SetSuperTooltip(this.listBoxTranFmt, new DevComponents.DotNetBar.SuperTooltipInfo("", "", 
 					"This is where transition types are selected.  This also defines how the transition will look in the text.",
 					null, null, DevComponents.DotNetBar.eTooltipColor.Gray));
+			// B2024-007 color of transition panel was not toggling properly
+			groupPanelTranFmt.Style.BackColor = !HasDefault && NeedsDefault ? Color.Red : Color.Orange;
 			//btnTranSave.Enabled = allowSave;
 			//if (CurTrans != null && selii != null)
 			//{