X-Git-Url: http://winboard.nl/cgi-bin?a=blobdiff_plain;f=xaw%2Fxoptions.c;h=6a193a00ecd7e0b2e911059d8bd3342ad44db3d7;hb=9d84fbc414dd9e4112e6ba42c9eca6c2244be6db;hp=d0fdafc64fdd2e44ee4b891e8216ec7499c58cbe;hpb=204df43a817c927d4db73f81cd4b4ce5075def6b;p=xboard.git

diff --git a/xaw/xoptions.c b/xaw/xoptions.c
index d0fdafc..6a193a0 100644
--- a/xaw/xoptions.c
+++ b/xaw/xoptions.c
@@ -201,7 +201,8 @@ LoadListBox (Option *opt, char *emptyText, int n1, int n2)
 {
     static char *dummyList[2];
     dummyList[0] = emptyText; // empty listboxes tend to crash X, so display user-supplied warning string instead
-    XawListChange(opt->handle, *(char*)opt->target ? opt->target : dummyList, 0, 0, True);
+    XawListChange(opt->handle, *(char**)opt->target ? opt->target : dummyList, 0, 0, True);
+//printf("listbox data = %x\n", opt->target);
 }
 
 int
@@ -405,14 +406,68 @@ format_accel (char *input)
     key = strdup(input);
   else
     key = strdup(++test); // remove ">"
+  if(strlen(key) == 1) key[0] = ToUpper(key[0]);
 
   output = realloc(output, strlen(output) + strlen(_(key))+2);
-  strncat(output, _(key), strlen(key) +1);
+  strncat(output, _(key), strlen(_(key)) +1);
 
   free(key);
   return output;
 }
 
+int
+pixlen (char *s)
+{
+#if 0
+    int dummy;
+    XCharStruct overall;
+    XTextExtents(messageFontStruct, s, strlen(s), &dummy, &dummy, &dummy, &overall);
+    return overall.width;
+#else
+    float tot = 0;
+    while(*s) switch(*s++) {
+	case '.': tot += 0.45; break;
+	case ' ': tot += 0.55; break;
+	case 'i': tot += 0.45; break;
+	case 'l': tot += 0.45; break;
+	case 'j': tot += 0.45; break;
+	case 'f': tot += 0.45; break;
+	case 'I': tot += 0.45; break;
+	case 't': tot += 0.45; break;
+	case 'k': tot += 0.83; break;
+	case 's': tot += 0.83; break;
+	case 'x': tot += 0.83; break;
+	case 'z': tot += 0.83; break;
+	case 'r': tot += 0.55; break;
+	case 'w': tot += 1.3; break;
+	case 'm': tot += 1.3; break;
+	case 'A': tot += 1.3; break;
+	case 'C': tot += 1.3; break;
+	case 'D': tot += 1.3; break;
+	case 'G': tot += 1.3; break;
+	case 'H': tot += 1.3; break;
+	case 'N': tot += 1.3; break;
+	case 'V': tot += 1.3; break;
+	case 'X': tot += 1.3; break;
+	case 'Y': tot += 1.3; break;
+	case 'Z': tot += 1.3; break;
+	case 'M': tot += 1.6; break;
+	case 'W': tot += 1.6; break;
+	case 'B': tot += 1.1; break;
+	case 'E': tot += 1.1; break;
+	case 'F': tot += 1.1; break;
+	case 'K': tot += 1.1; break;
+	case 'P': tot += 1.1; break;
+	case 'R': tot += 1.1; break;
+	case 'S': tot += 1.1; break;
+	case 'O': tot += 1.4; break;
+	case 'Q': tot += 1.4; break;
+	default:  tot++;
+    }
+    return tot;
+#endif
+}
+
 static Widget
 CreateComboPopup (Widget parent, Option *opt, int n, int fromList, int def)
 {   // fromList determines if the item texts are taken from a list of strings, or from a menu table
@@ -421,16 +476,19 @@ CreateComboPopup (Widget parent, Option *opt, int n, int fromList, int def)
     Arg arg;
     MenuItem *mb = (MenuItem *) opt->choice;
     char **list = (char **) opt->choice;
-    int maxlength=0;
+    int maxlength=0, menuLen[1000];
 
 
     if(list[0] == NULL) return NULL; // avoid empty menus, as they cause crash
     menu = XtCreatePopupShell(opt->name, simpleMenuWidgetClass, parent, NULL, 0);
 
     if(!fromList)
-      for (i=0; mb[i].string; i++)
-	if (maxlength < strlen(_(mb[i].string)) )
-	  maxlength = strlen(_(mb[i].string) );
+      for (i=0; mb[i].string; i++) if(mb[i].accel) {
+	int len = pixlen(_(mb[i].string));
+	menuLen[i] = len;
+	if (maxlength < len )
+	  maxlength = len;
+      }
 
     for (i=0; 1; i++)
       {
@@ -444,12 +502,13 @@ CreateComboPopup (Widget parent, Option *opt, int n, int fromList, int def)
 	    char *menuname = opt->min & NO_GETTEXT ? msg : _(msg);
 	    char *accel = format_accel(mb[i].accel);
 	    size_t len;
-	    int fill = maxlength - strlen(menuname) +2+strlen(accel);
+//	    int fill = maxlength - strlen(menuname) +2+strlen(accel);
+	    int fill = (maxlength - menuLen[i] + 3)*1.8;
 
-	    len = strlen(menuname)+fill+1;
+	    len = strlen(menuname)+fill+strlen(accel)+1;
 	    label = malloc(len);
 
-	    snprintf(label,len,"%s%*s",menuname,fill,accel);
+	    snprintf(label,len,"%s%*s%s",menuname,fill," ",accel);
 	    free(accel);
 	  }
 	else
@@ -1231,7 +1290,7 @@ GenericPopUp (Option *option, char *title, DialogClass dlgNr, DialogClass parent
 	  XtSetArg(args[j], XtNheight, (Dimension) (wp[dlgNr]->height));  j++;
 	  XtSetArg(args[j], XtNwidth,  (Dimension) (wp[dlgNr]->width));  j++;
 	}
-	if(wp[dlgNr]->width > 0 && wp[dlgNr]->y > 0) {
+	if(wp[dlgNr]->x > 0 && wp[dlgNr]->y > 0) {
 	  XtSetArg(args[j], XtNx, (Position) (wp[dlgNr]->x));  j++;
 	  XtSetArg(args[j], XtNy, (Position) (wp[dlgNr]->y));  j++;
 	}