Volian/SourceCode
8
0
Fork 0
Code Issues Pull Requests 1 Projects Releases Wiki Activity

B2023-078 When using the split screen PROMS was crashing when the user closed a procedure tab on the other split screen side which didn’t have the current editing focus. PROMS was trying to reposition to the procedure tab that had editing focus, we were not taking into account that saved currently active procedure tab could be in a different split screen.

Browse Source
This commit is contained in:
John Jenko 2023-08-30 10:55:07 -04:00
parent ad476d8de4
commit 10821a478b
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
PROMS/Volian.Controls.Library/DisplayTabControl.cs
View File

@ -827,7 +827,11 @@ namespace Volian.Controls.Library
private DisplayTabItem FindRemainingTab(Bar myBar)
{
// C2023-004: Proms reverts to first tab rather than active tab/save last selected tab is and use to reset to it
if (LastSelectedDisplayTabItem != null && !_RemovedDisplayTabItems.Contains(LastSelectedDisplayTabItem))
// B2023-078 Added the check for LastSelectedDisplayTabItem is in myBar. This fixes a crash issue where user has as split screen
// each with procedure tabs and closes a procedure tab in the split screen that does not currently have focus.
// that other screen has its own myBar list which does not contain the LastSelectedDisplayTabItem references.
// In that case we want to run through the foreach loop to find a tab in the currently active split screen side
if (LastSelectedDisplayTabItem != null && myBar.Items.Contains(LastSelectedDisplayTabItem) && !_RemovedDisplayTabItems.Contains(LastSelectedDisplayTabItem))
return LastSelectedDisplayTabItem;
foreach (DisplayTabItem itm in myBar.Items)
{