(svn r14168) -Codechange: Make dropdown 'auto_width' a separate parameter, so that a minimum width can be specified.

1 files changed, 8 insertions, 7 deletions

diff --git a/src/widgets/dropdown.cpp b/src/widgets/dropdown.cpp
index f04c72764..23f87a2cf 100644
--- a/src/widgets/dropdown.cpp
+++ b/src/widgets/dropdown.cpp
@@ -229,7 +229,7 @@ struct DropdownWindow : Window {
 	}
 };
 
-void ShowDropDownList(Window *w, DropDownList *list, int selected, int button, uint width, bool instant_close)
+void ShowDropDownList(Window *w, DropDownList *list, int selected, int button, uint width, bool auto_width, bool instant_close)
 {
 	bool is_dropdown_menu_shown = w->IsWidgetLowered(button);
 
@@ -250,17 +250,16 @@ void ShowDropDownList(Window *w, DropDownList *list, int selected, int button, u
 	/* The preferred position is just below the dropdown calling widget */
 	int top = w->top + wi->bottom + 1;
 
-	bool auto_width = (width == UINT_MAX);
+	if (width == 0) width = wi->right - wi->left + 1;
+
+	uint max_item_width = 0;
 
 	if (auto_width) {
 		/* Find the longest item in the list */
-		width = 0;
 		for (DropDownList::const_iterator it = list->begin(); it != list->end(); ++it) {
 			const DropDownListItem *item = *it;
-			width = max(width, item->Width() + 5);
+			max_item_width = max(max_item_width, item->Width() + 5);
 		}
-	} else if (width == 0) {
-		width = wi->right - wi->left + 1;
 	}
 
 	/* Total length of list */
@@ -297,10 +296,12 @@ void ShowDropDownList(Window *w, DropDownList *list, int selected, int button, u
 			scroll = true;
 			/* Add space for the scroll bar if we automatically determined
 			 * the width of the list. */
-			if (auto_width) width += 12;
+			max_item_width += 12;
 		}
 	}
 
+	if (auto_width) width = max(width, max_item_width);
+
 	DropdownWindow *dw = new DropdownWindow(
 		w->left + wi->left,
 		top,