From 5e248781a8509ef59c681636175f4832023ae77f Mon Sep 17 00:00:00 2001 From: Darkvater Date: Wed, 18 Oct 2006 14:20:10 +0000 Subject: [PATCH] (svn r6821) -Codechange: For the measurement tool do not show the tooltip when the selection is only a single tile. Seems this was a bit annoying to some ;) --- misc_gui.c | 10 +++---- viewport.c | 78 +++++++++++++++++++++++++++++------------------------- window.h | 2 +- 3 files changed, 48 insertions(+), 42 deletions(-) diff --git a/misc_gui.c b/misc_gui.c index aece882c5f..68e125e4f7 100644 --- a/misc_gui.c +++ b/misc_gui.c @@ -665,18 +665,18 @@ static void TooltipsWndProc(Window *w, WindowEvent *e) * @param str String to be displayed * @param params (optional) up to 5 pieces of additional information that may be * added to a tooltip; currently only supports parameters of {NUM} (integer) */ -void GuiShowTooltipsWithArgs(StringID str, uint paramcount, uint32 params[]) +void GuiShowTooltipsWithArgs(StringID str, uint paramcount, const uint32 params[]) { char buffer[512]; BoundingRect br; + Window *w; uint i; int x, y; - Window *w = FindWindowById(WC_TOOLTIPS, 0); - if (w != NULL) DeleteWindow(w); + DeleteWindowById(WC_TOOLTIPS, 0); - /* We only show measurement tooltips with shift pressed down */ - if (paramcount != 0 && !_patches.measure_tooltip) return; + /* We only show measurement tooltips with patch setting on */ + if (str == STR_NULL || (paramcount != 0 && !_patches.measure_tooltip)) return; for (i = 0; i != paramcount; i++) SetDParam(i, params[i]); GetString(buffer, str); diff --git a/viewport.c b/viewport.c index 55ea9214d1..ba4a13b0a7 100644 --- a/viewport.c +++ b/viewport.c @@ -2110,6 +2110,8 @@ static int CalcHeightdiff(HighLightStyle style, uint distance, TileIndex start_t return (int)(h1 - h0) * 50; } +static const measure_strings_length[] = {STR_NULL, STR_MEASURE_LENGTH, STR_MEASURE_LENGTH_HEIGHTDIFF}; + // while dragging static void CalcRaildirsDrawstyle(TileHighlightData *thd, int x, int y, int method) { @@ -2215,23 +2217,23 @@ static void CalcRaildirsDrawstyle(TileHighlightData *thd, int x, int y, int meth TileIndex t0 = TileVirtXY(thd->selstart.x, thd->selstart.y); TileIndex t1 = TileVirtXY(x, y); uint distance = DistanceManhattan(t0, t1) + 1; - int heightdiff = CalcHeightdiff(b, distance, t0, t1); + byte index = 0; uint params[2]; - /* If we are showing a tooltip for horizontal or vertical drags, - * 2 tiles have a length of 1. To bias towards the ceiling we add - * one before division. It feels more natural to count 3 lengths as 2 */ - if ((b & HT_DIR_MASK) != HT_DIR_X && (b & HT_DIR_MASK) != HT_DIR_Y) { - distance = (distance + 1) / 2; - } + if (distance != 1) { + int heightdiff = CalcHeightdiff(b, distance, t0, t1); + /* If we are showing a tooltip for horizontal or vertical drags, + * 2 tiles have a length of 1. To bias towards the ceiling we add + * one before division. It feels more natural to count 3 lengths as 2 */ + if ((b & HT_DIR_MASK) != HT_DIR_X && (b & HT_DIR_MASK) != HT_DIR_Y) { + distance = (distance + 1) / 2; + } - params[0] = distance; - if (heightdiff == 0) { - GuiShowTooltipsWithArgs(STR_MEASURE_LENGTH, 1, params); - } else { - params[1] = heightdiff; - GuiShowTooltipsWithArgs(STR_MEASURE_LENGTH_HEIGHTDIFF, 2, params); + params[index++] = distance; + if (heightdiff != 0) params[index++] = heightdiff; } + + GuiShowTooltipsWithArgs(measure_strings_length[index], index, params); } thd->selend.x = x; @@ -2294,21 +2296,22 @@ calc_heightdiff_single_direction:; TileIndex t0 = TileVirtXY(sx, sy); TileIndex t1 = TileVirtXY(x, y); uint distance = DistanceManhattan(t0, t1) + 1; - /* With current code passing a HT_LINE style to calculate the height - * difference is enough. However if/when a point-tool is created - * with this method, function should be called with new_style (below) - * instead of HT_LINE | style case HT_POINT is handled specially - * new_style := (_thd.next_drawstyle & HT_RECT) ? HT_LINE | style : _thd.next_drawstyle; */ - int heightdiff = CalcHeightdiff(HT_LINE | style, 0, t0, t1); + byte index = 0; uint params[2]; - params[0] = distance; - if (heightdiff == 0) { - GuiShowTooltipsWithArgs(STR_MEASURE_LENGTH, 1, params); - } else { - params[1] = heightdiff; - GuiShowTooltipsWithArgs(STR_MEASURE_LENGTH_HEIGHTDIFF, 2, params); + if (distance != 1) { + /* With current code passing a HT_LINE style to calculate the height + * difference is enough. However if/when a point-tool is created + * with this method, function should be called with new_style (below) + * instead of HT_LINE | style case HT_POINT is handled specially + * new_style := (_thd.next_drawstyle & HT_RECT) ? HT_LINE | style : _thd.next_drawstyle; */ + int heightdiff = CalcHeightdiff(HT_LINE | style, 0, t0, t1); + + params[index++] = distance; + if (heightdiff != 0) params[index++] = heightdiff; } + + GuiShowTooltipsWithArgs(measure_strings_length[index], index, params); } break; case VPM_X_AND_Y_LIMITED: { /* drag an X by Y constrained rect area */ @@ -2316,13 +2319,17 @@ calc_heightdiff_single_direction:; x = sx + clamp(x - sx, -limit, limit); y = sy + clamp(y - sy, -limit, limit); /* Fallthrough */ - case VPM_X_AND_Y: { /* drag an X by Y area */ + case VPM_X_AND_Y: /* drag an X by Y area */ if (_patches.measure_tooltip) { + static const StringID measure_strings_area[] = { + STR_NULL, STR_NULL, STR_MEASURE_AREA, STR_MEASURE_AREA_HEIGHTDIFF + }; + TileIndex t0 = TileVirtXY(sx, sy); TileIndex t1 = TileVirtXY(x, y); uint dx = abs(TileX(t0) - TileX(t1)) + 1; uint dy = abs(TileY(t0) - TileY(t1)) + 1; - int heightdiff; + byte index = 0; uint params[3]; /* If dragging an area (eg dynamite tool) and it is actually a single @@ -2336,18 +2343,17 @@ calc_heightdiff_single_direction:; } } - heightdiff = CalcHeightdiff(style, 0, t0, t1); + if (dx != 1 || dy != 1) { + int heightdiff = CalcHeightdiff(style, 0, t0, t1); - params[0] = dx; - params[1] = dy; - if (heightdiff == 0) { - GuiShowTooltipsWithArgs(STR_MEASURE_AREA, 2, params); - } else { - params[2] = heightdiff; - GuiShowTooltipsWithArgs(STR_MEASURE_AREA_HEIGHTDIFF, 3, params); + params[index++] = dx; + params[index++] = dy; + if (heightdiff != 0) params[index++] = heightdiff; } + + GuiShowTooltipsWithArgs(measure_strings_area[index], index, params); } - } break; + break; } default: NOT_REACHED(); diff --git a/window.h b/window.h index 876f6a2a4b..9d942e2826 100644 --- a/window.h +++ b/window.h @@ -807,7 +807,7 @@ void CDECL SetWindowWidgetsHiddenState(Window *w, bool hidden_stat, int widgets, void CDECL SetWindowWidgetsLoweredState(Window *w, bool lowered_stat, int widgets, ...); /* misc_gui.c*/ -void GuiShowTooltipsWithArgs(StringID str, uint paramcount, uint params[]); +void GuiShowTooltipsWithArgs(StringID str, uint paramcount, const uint params[]); static inline void GuiShowTooltips(StringID str) { GuiShowTooltipsWithArgs(str, 0, NULL);