changeset 815:ce8f71217f45

Bugfix CustomView selection during dive mode: The CustomView selection during dive mode was limited to two pages, while meanwhile three are needed to show all options. The problem did not showup in surface mode selection. Rootcause was a static definition instead of the dynamic one used in surface mode. In addition with every page shift a new page was created which might cause display errors after some times. To solve this the pages are now updated instead of rebuilding the pages.
author Ideenmodellierer
date Sun, 03 Sep 2023 18:03:49 +0200 (15 months ago)
parents 26ae9e8f24fd
children c4ee952b9425
files Discovery/Inc/tMenu.h Discovery/Inc/tStructure.h Discovery/Src/tMenu.c Discovery/Src/tMenuEditCustom.c Discovery/Src/tMenuSystem.c
diffstat 5 files changed, 52 insertions(+), 11 deletions(-) [+]
line wrap: on
line diff
--- a/Discovery/Inc/tMenu.h	Sun Sep 03 17:58:12 2023 +0200
+++ b/Discovery/Inc/tMenu.h	Sun Sep 03 18:03:49 2023 +0200
@@ -87,6 +87,7 @@
 
 void selectPage(uint32_t selection);
 
+void resetLineMask(uint32_t lineId);
 void disableLine(uint32_t lineId);
 void enableLine(uint32_t lineId);
 
--- a/Discovery/Inc/tStructure.h	Sun Sep 03 17:58:12 2023 +0200
+++ b/Discovery/Inc/tStructure.h	Sun Sep 03 18:03:49 2023 +0200
@@ -340,6 +340,13 @@
 #define StMSYS5_ResetBluetooth	_MB(2,8,6,12,0)
 #define StMSYS5_SetSampleIndx   _MB(2,8,6,13,0)
 
+#define StMSYS_Custom0			_MB(2,8,1,0,0)
+#define StMSYS_Custom1			_MB(2,8,2,0,0)
+#define StMSYS_Custom2			_MB(2,8,3,0,0)
+#define StMSYS_Custom3			_MB(2,8,4,0,0)
+#define StMSYS_Custom4			_MB(2,8,5,0,0)
+#define StMSYS_Custom5			_MB(2,8,6,0,0)
+
  /* PAGE 9 */
 
  # define StMCustom 	_MB(2,9,0,0,0)
--- a/Discovery/Src/tMenu.c	Sun Sep 03 17:58:12 2023 +0200
+++ b/Discovery/Src/tMenu.c	Sun Sep 03 18:03:49 2023 +0200
@@ -252,6 +252,15 @@
 	}
 }
 
+void resetLineMask(uint32_t lineId)
+{
+	SStateList idList;
+	get_idSpecificStateList(lineId, &idList);
+	if(idList.page < MAXPAGES)
+	{
+		menu.disableLineMask[idList.page] = 0;
+	}
+}
 void enableLine(uint32_t lineId)
 {
 	SStateList idList;
@@ -480,9 +489,13 @@
     if(line == 0)
         line = 1;
 
-    if(menu.disableLineMask[page] & ( 1 << line))
+    while(menu.disableLineMask[page] & ( 1 << line))
     {
     	line++;
+    	if(line > menu.linesAvailableForPage[page])
+    	{
+    	    line = 1;
+    	}
     }
 
     if(line > menu.linesAvailableForPage[page])
@@ -712,6 +725,11 @@
         tMSP_refresh(text, &tabPosition, subtext);
         update_content_with_new_frame(page, text, tabPosition, subtext);
         break;
+    case StMSYS:
+    	tMSystem_refresh(0, text, &tabPosition, NULL);
+    	update_content_with_new_frame(page, text, tabPosition, subtext);
+    	break;
+
     default:
         break;
     }
--- a/Discovery/Src/tMenuEditCustom.c	Sun Sep 03 17:58:12 2023 +0200
+++ b/Discovery/Src/tMenuEditCustom.c	Sun Sep 03 18:03:49 2023 +0200
@@ -44,7 +44,6 @@
 
 
 #define CV_PER_PAGE  		(5u)	/* number of cv selections shown at one page */
-#define CV_SUBPAGE_MAX		(2u)	/* max number of customer view selection pages */
 #define MAX_BACKLIGHT_BOOST (2u)	/* max number of backlight levels which may be increased during focus state */
 
 #define MAX_FOCUS_LIMITER	(2u)	/* max number for reducing the spot used for focus detection */
@@ -930,22 +929,19 @@
 {
 	static uint8_t customviewsSubpage = 0;
 	SSettings *pSettings = settingsGetPointer();
-	char text[MAX_PAGE_TEXTSIZE];
-	uint16_t tabPosition;
-	uint32_t id;
 
+	customviewsSubpageMax = (tHome_getNumberOfAvailableCVs(cv_changelist) / CV_PER_PAGE) + 1;
 
 	if((line == 6) || (cv_changelist[customviewsSubpage * 5 + line-1] == CVIEW_END))		/* select next set of views */
 	{
 		customviewsSubpage++;
-		if(customviewsSubpage == CV_SUBPAGE_MAX)
+		if(customviewsSubpage == customviewsSubpageMax)
 		{
 			customviewsSubpage = 0;
 		}
 		set_CustomsviewsSubpage(customviewsSubpage);
 		/* rebuild the selection page with the next set of customer views */
-		id = tMSystem_refresh(0, text, &tabPosition, NULL);
-		tM_build_page(id, text, tabPosition, NULL);
+		updateSpecificMenu(StMSYS);
 		openMenu(0);
 	}
 	else
--- a/Discovery/Src/tMenuSystem.c	Sun Sep 03 17:58:12 2023 +0200
+++ b/Discovery/Src/tMenuSystem.c	Sun Sep 03 18:03:49 2023 +0200
@@ -59,6 +59,8 @@
     *tab = 300;
     *subtext = 0;
 
+    resetLineMask(StMSYS);
+
     // dive mode
     if(actual_menu_content != MENU_SURFACE)
     {
@@ -80,17 +82,34 @@
 					TXT_2BYTE, customview_TXT2BYTE_helper(id));
         	}
         }
-        text[textPointer++] = TXT_2BYTE;
-        text[textPointer++] = TXT2BYTE_ButtonNext;
-        text[textPointer] = 0;
 
         for(;i<5;i++)	/* clear empty lines in case menu shows less than 5 entries */
         {
         	text[textPointer++]='\n';
         	text[textPointer++]='\r';
         	text[textPointer] = 0;
+        	switch(i)
+			{
+        		case 0:	disableLine(StMSYS_Custom0);
+        			break;
+        		case 1:	disableLine(StMSYS_Custom1);
+        		    break;
+        		case 2:	disableLine(StMSYS_Custom2);
+        			break;
+        		case 3:	disableLine(StMSYS_Custom3);
+        		    break;
+        		case 4:	disableLine(StMSYS_Custom4);
+        		    break;
+        		default:
+        		case 5:	disableLine(StMSYS_Custom5);
+        		    break;
+			}
         }
 
+        text[textPointer++] = TXT_2BYTE;
+        text[textPointer++] = TXT2BYTE_ButtonNext;
+        text[textPointer] = 0;
+
         return StMSYS;
     }