X-Git-Url: https://git.ucc.asn.au/?a=blobdiff_plain;f=Usermode%2FApplications%2Faxwin3_src%2FWM%2Frenderers%2Fmenu.c;h=a183de005593c020deee31913fbe37616aa139da;hb=fae63a3ca20775dbf083ba601bea317570e3dd64;hp=7e286709bf94ca6ea10eccbde1a5f0559a9a6acc;hpb=b1488ae9beae34068d38d00e191b90ef0bf812f3;p=tpg%2Facess2.git

diff --git a/Usermode/Applications/axwin3_src/WM/renderers/menu.c b/Usermode/Applications/axwin3_src/WM/renderers/menu.c
index 7e286709..a183de00 100644
--- a/Usermode/Applications/axwin3_src/WM/renderers/menu.c
+++ b/Usermode/Applications/axwin3_src/WM/renderers/menu.c
@@ -168,7 +168,7 @@ void Renderer_Menu_Redraw(tWindow *Window)
 			w, ciMenu_ItemHeight,
 			gMenu_Font,
 			cMenu_LabelColour,
-			item->Label
+			item->Label, -1
 			);
 		// Underline
 		if(item->UnderlineW)
@@ -188,7 +188,7 @@ void Renderer_Menu_Redraw(tWindow *Window)
 				w, ciMenu_ItemHeight,
 				gMenu_Font,
 				cMenu_ShortcutColour,
-				item->Shortcut
+				item->Shortcut, -1
 				);
 		}
 		
@@ -275,33 +275,31 @@ int Renderer_Menu_int_AddItem(tWindow *Window, int Length, const tMenuMsg_AddIte
 	}
 	else
 	{
-		char	tmp = item->Label[item->KeyOffset];
 		// Get width of preceding substring
-		item->Label[item->KeyOffset] = '\0';
-		WM_Render_GetTextDims(NULL, item->Label, &item->UnderlineX, NULL);
+		WM_Render_GetTextDims(NULL, item->Label, item->KeyOffset, &item->UnderlineX, NULL);
 		// Get the width of the underlined character
-		// TODO: Fix for high UTF-8 characters
-		item->Label[item->KeyOffset] = tmp;
-		tmp = item->Label[item->KeyOffset+1];
-		item->Label[item->KeyOffset+1] = '\0';
+		// NOTE: 1 makes only one character be parsed, even if it is >1 byte long
 		WM_Render_GetTextDims(
-			NULL, item->Label+item->KeyOffset,
+			NULL, item->Label+item->KeyOffset, 1,
 			&item->UnderlineW, NULL
 			);
-		item->Label[item->KeyOffset+1] = tmp;
 	}
 	// - Labels
-	WM_Render_GetTextDims(NULL, item->Label, &item->LabelWidth, NULL);
+	WM_Render_GetTextDims(NULL, item->Label, -1, &item->LabelWidth, NULL);
 	if(item->Shortcut)
-		WM_Render_GetTextDims(NULL, item->Shortcut, &item->ShortcutWidth, NULL);
+		WM_Render_GetTextDims(NULL, item->Shortcut, -1, &item->ShortcutWidth, NULL);
 	else
 		item->ShortcutWidth = 0;
-	
+
+	// Get maximum lengths (to determine the size of the menu	
 	if( item->LabelWidth > info->MaxLabelWidth )
 		info->MaxLabelWidth = item->LabelWidth;
 	if( item->ShortcutWidth > info->MaxShortcutWidth )
 		info->MaxShortcutWidth = item->ShortcutWidth;
 	
+	// Update width
+	// TODO: Check, do I want to resize down too?
+	// TODO: Take into account padding too
 	if( info->MaxLabelWidth + info->MaxShortcutWidth + ciMenu_Gap > info->CachedW )
 	{
 		info->CachedW = ciMenu_LeftPadding + info->MaxLabelWidth
@@ -313,6 +311,9 @@ int Renderer_Menu_int_AddItem(tWindow *Window, int Length, const tMenuMsg_AddIte
 	return 0;
 }
 
+/**
+ * \brief Convert coordinates into an item index
+ */
 int Renderer_Menu_int_GetItemByPos(tWindow *Window, tMenuWindowInfo *Info, int X, int Y)
 {
 	 int	i;