"fixed" the combobox issue
[instimg] / src / widgets.c
index 707d515..2d37973 100644 (file)
@@ -285,12 +285,46 @@ struct wgt_widget *wgt_checkbox(struct wgt_window *win, const char *text, int on
        return w;
 }
 
+static void combosize(struct wgt_window *win, HWND cbwnd, const char **items, int num_items,
+               int width, int height, struct wgt_rect *sz)
+{
+       int i, max_width, sum_height;
+       struct wgt_rect textsz;
+
+       wgt_string_size(win, "00", &textsz);
+       max_width = width < 0 && width != WGT_AUTO ? -width : textsz.width;
+
+       /* first get the height of the editbox */
+       sum_height = SendMessage(cbwnd, CB_GETITEMHEIGHT, -1, 0);
+       sz->y = sum_height;
+       /* then add the height of the dropdown list items */
+       /* XXX I can't figure out why, but it's not sufficient to show all items
+        * without some more height, so let's double it...
+        */
+       sum_height *= 2;
+       sum_height += SendMessage(cbwnd, CB_GETITEMHEIGHT, 0, 0) * num_items;
+       sum_height += GetSystemMetrics(SM_CYEDGE) * 2;
+
+       if(width < 0) {
+               int sbar_width;
+               for(i=0; i<num_items; i++) {
+                       wgt_string_size(win, items[i], &textsz);
+                       if(textsz.width > max_width) max_width = textsz.width;
+               }
+               sbar_width = GetSystemMetrics(SM_CXVSCROLL);
+               width = max_width + sbar_width + GetSystemMetrics(SM_CXEDGE) * 2;
+       }
+       if(height < 0) height = sum_height;
+
+       sz->width = width;
+       sz->height = height;
+}
+
 struct wgt_widget *wgt_combo(struct wgt_window *win, const char **items, int num_items,
                        int sel, int x, int y, int width, int height, wgt_callback modfunc)
 {
-       int i, max_width, max_height, sum_height, res;
        struct wgt_widget *w;
-       struct wgt_rect textsz;
+       struct wgt_rect rect;
 
        if(!(w = calloc(1, sizeof *w))) {
                fprintf(stderr, "wgt_combo: failed to allocate widget structure\n");
@@ -298,49 +332,22 @@ struct wgt_widget *wgt_combo(struct wgt_window *win, const char **items, int num
        }
        w->win = win;
 
-       if(!(w->itemlist = malloc(num_items * sizeof *w->itemlist))) {
-               fprintf(stderr, "wgt_combo: failed to allocate item list\n");
-               free(w);
-               return 0;
-       }
-
-       wgt_string_size(win, "00", &textsz);
-       max_width = width < 0 && width != WGT_AUTO ? -width : textsz.width;
-       max_height = height < 0 && height != WGT_AUTO ? -height : textsz.height;
-       sum_height = num_items ? 0 : max_height;
-
-       for(i=0; i<num_items; i++) {
-               wgt_string_size(win, items[i], &textsz);
-               if(textsz.width > max_width) max_width = textsz.width;
-               if(textsz.height > max_height) max_height = textsz.height;
-               sum_height += textsz.height;
-       }
-       if(width < 0) width = max_width * 3 / 2;
-       if(height < 0) height = sum_height * 2;
-
-
        if(!(w->handle = CreateWindow("COMBOBOX", items[0] ? items[0] : "",
                        WS_CHILD | WS_VISIBLE | CBS_DROPDOWNLIST | CBS_HASSTRINGS,
-                       x, y, width, height, win->handle, 0, GetModuleHandle(0), 0))) {
+                       x, y, 16, 16, win->handle, 0, GetModuleHandle(0), 0))) {
                fprintf(stderr, "wgt_combo: failed to create window\n");
                free(w);
                return 0;
        }
 
-       for(i=0; i<num_items; i++) {
-               if(!(w->itemlist[i] = strdup(items[i]))) {
-                       fprintf(stderr, "wgt_combo: failed to allocate item\n");
-                       while(--i >= 0) free(w->itemlist[i]);
-                       DestroyWindow(w->handle);
-                       free(w->itemlist);
-                       free(w);
-                       return 0;
-               }
-               if((res = SendMessage(w->handle, CB_ADDSTRING, 0, (long)items[i])) != i) {
-                       fprintf(stderr, "wgt_combo: failed to add item\n");
-               }
-       }
-       w->num_items = num_items;
+       combosize(win, w->handle, items, num_items, width, height, &rect);
+       w->rect.x = x;
+       w->rect.y = y;
+       w->rect.width = rect.width;
+       w->rect.height = rect.y;        /* single line height for layout purposes */
+       MoveWindow(w->handle, x, y, rect.width, rect.height, FALSE);
+
+       wgt_combo_setitems(w, items, num_items);
 
        if(sel < 0) sel = 0;
        if(sel >= num_items) sel = num_items - 1;
@@ -348,17 +355,68 @@ struct wgt_widget *wgt_combo(struct wgt_window *win, const char **items, int num
                SendMessage(w->handle, CB_SETCURSEL, sel, 0);
        }
 
-       w->rect.x = x;
-       w->rect.y = y;
-       w->rect.width = width;
-       w->rect.height = max_height;
-
        w->cb_modify = modfunc;
        w->next = win->wlist;
        win->wlist = w;
        return w;
 }
 
+void wgt_checkbox_check(struct wgt_widget *w)
+{
+       SendMessage(w->handle, BM_SETCHECK, BST_CHECKED, 0);
+}
+
+void wgt_checkbox_uncheck(struct wgt_widget *w)
+{
+       SendMessage(w->handle, BM_SETCHECK, BST_UNCHECKED, 0);
+}
+
+int wgt_checkbox_checked(struct wgt_widget *w)
+{
+       return SendMessage(w->handle, BM_GETCHECK, 0, 0) == BST_CHECKED;
+}
+
+int wgt_combo_setitems(struct wgt_widget *w, const char **items, int num_items)
+{
+       int i;
+       char **newit;
+       struct wgt_rect size;
+
+       if(!(newit = malloc(num_items * sizeof *newit))) {
+               fprintf(stderr, "wgt_combo_setitems: failed to allocate item list\n");
+               return -1;
+       }
+       for(i=0; i<num_items; i++) {
+               if(!(newit[i] = strdup(items[i]))) {
+                       fprintf(stderr, "wgt_combo_setitems: failed to allocate item\n");
+                       while(--i >= 0) {
+                               free(newit[i]);
+                       }
+                       free(newit);
+                       return -1;
+               }
+       }
+
+       for(i=0; i<w->num_items; i++) {
+               free(w->itemlist[i]);
+       }
+       free(w->itemlist);
+       SendMessage(w->handle, CB_RESETCONTENT, 0, 0);
+
+       w->itemlist = newit;
+       w->num_items = num_items;
+
+       for(i=0; i<num_items; i++) {
+               SendMessage(w->handle, CB_ADDSTRING, 0, (long)w->itemlist[i]);
+       }
+
+       combosize(w->win, w->handle, items, num_items, w->rect.width, WGT_AUTO, &size);
+       MoveWindow(w->handle, w->rect.x, w->rect.y, w->rect.width, size.height, TRUE);
+
+       SendMessage(w->handle, CB_SETCURSEL, 0, 0);
+       return 0;
+}
+
 int wgt_combo_selected(struct wgt_widget *w)
 {
        int sel = SendMessage(w->handle, CB_GETCURSEL, 0, 0);