summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorrubidium <rubidium@openttd.org>2009-01-10 17:13:41 +0000
committerrubidium <rubidium@openttd.org>2009-01-10 17:13:41 +0000
commit7642fd7b347c0497149608422b5992e23ccc0948 (patch)
treec51f972bfdc778b926aedef73f9933ab1fdb3cfa
parent097fddc4542339ee569dd207505e460722c4aa4e (diff)
downloadopenttd-7642fd7b347c0497149608422b5992e23ccc0948.tar.xz
(svn r14969) -Codechange: make drawing trees possible (Alberth)
-rw-r--r--bin/data/openttdd.grfbin460826 -> 460977 bytes
-rw-r--r--bin/data/openttdw.grfbin461016 -> 461169 bytes
-rw-r--r--bin/data/orig_dos.obg2
-rw-r--r--bin/data/orig_dos_de.obg2
-rw-r--r--bin/data/orig_win.obg2
-rw-r--r--src/settings_gui.cpp139
-rw-r--r--src/table/sprites.h5
7 files changed, 122 insertions, 28 deletions
diff --git a/bin/data/openttdd.grf b/bin/data/openttdd.grf
index 67724d0f1..aed7b8a29 100644
--- a/bin/data/openttdd.grf
+++ b/bin/data/openttdd.grf
Binary files differ
diff --git a/bin/data/openttdw.grf b/bin/data/openttdw.grf
index 4cac67e14..ec77f462f 100644
--- a/bin/data/openttdw.grf
+++ b/bin/data/openttdw.grf
Binary files differ
diff --git a/bin/data/orig_dos.obg b/bin/data/orig_dos.obg
index 3c05c2a4d..a77025886 100644
--- a/bin/data/orig_dos.obg
+++ b/bin/data/orig_dos.obg
@@ -24,7 +24,7 @@ TRGI.GRF = da6a6c9dcc451eec88d79211437b76a8
TRGC.GRF = ed446637e034104c5559b32c18afe78d
TRGH.GRF = ee6616fb0e6ef6b24892c58c93d86fc9
TRGT.GRF = e30e8a398ae86c03dc534a8ac7dfb3b6
-OPENTTDD.GRF = ea18d93ca0364c3e66d99dc66b3171ce
+OPENTTDD.GRF = d006d291e15ea34e81a14ca0d6f227bc
[origin]
default = You can find it on your Transport Tycoon Deluxe CD-ROM.
diff --git a/bin/data/orig_dos_de.obg b/bin/data/orig_dos_de.obg
index e81af74b0..8433cb963 100644
--- a/bin/data/orig_dos_de.obg
+++ b/bin/data/orig_dos_de.obg
@@ -24,7 +24,7 @@ TRGI.GRF = da6a6c9dcc451eec88d79211437b76a8
TRGC.GRF = ed446637e034104c5559b32c18afe78d
TRGH.GRF = ee6616fb0e6ef6b24892c58c93d86fc9
TRGT.GRF = fcde1d7e8a74197d72a62695884b909e
-OPENTTDD.GRF = ea18d93ca0364c3e66d99dc66b3171ce
+OPENTTDD.GRF = d006d291e15ea34e81a14ca0d6f227bc
[origin]
default = You can find it on your Transport Tycoon Deluxe CD-ROM.
diff --git a/bin/data/orig_win.obg b/bin/data/orig_win.obg
index 23d6df897..3b498bdc8 100644
--- a/bin/data/orig_win.obg
+++ b/bin/data/orig_win.obg
@@ -24,7 +24,7 @@ TRGIR.GRF = 0c2484ff6be49fc63a83be6ab5c38f32
TRGCR.GRF = 3668f410c761a050b5e7095a2b14879b
TRGHR.GRF = 06bf2b7a31766f048baac2ebe43457b1
TRGTR.GRF = de53650517fe661ceaa3138c6edb0eb8
-OPENTTDW.GRF = d265ad550b7ca83ae94d83492c3cad11
+OPENTTDW.GRF = 0e5b39d1349da3451ba00684722b87b4
[origin]
default = You can find it on your Transport Tycoon Deluxe CD-ROM.
diff --git a/src/settings_gui.cpp b/src/settings_gui.cpp
index 3a3eccc9d..c509f7263 100644
--- a/src/settings_gui.cpp
+++ b/src/settings_gui.cpp
@@ -584,7 +584,8 @@ void ShowGameDifficulty()
new GameDifficultyWindow();
}
-static const int SETTING_HEIGHT = 11; ///< Height of a single patch setting in the tree view
+static const int SETTING_HEIGHT = 11; ///< Height of a single setting in the tree view in pixels
+static const int LEVEL_WIDTH = 15; ///< Indenting width of a sub-page in pixels
/**
* Flags for #PatchEntry
@@ -637,7 +638,7 @@ struct PatchEntry {
uint Length() const;
PatchEntry *FindEntry(uint row, uint *cur_row);
- void Draw(GameSettings *patches_ptr, int x, int y);
+ uint Draw(GameSettings *patches_ptr, int base_x, int base_y, uint first_row, uint max_row, uint cur_row, uint parent_last);
private:
void DrawPatch(GameSettings *patches_ptr, const SettingDesc *sd, int x, int y, int state);
@@ -652,6 +653,8 @@ struct PatchPage {
uint Length() const;
PatchEntry *FindEntry(uint row, uint *cur_row) const;
+
+ uint Draw(GameSettings *patches_ptr, int base_x, int base_y, uint first_row, uint max_row, uint cur_row = 0, uint parent_last = 0) const;
};
@@ -759,15 +762,79 @@ PatchEntry *PatchEntry::FindEntry(uint row_num, uint *cur_row)
}
/**
- * Draw setting value (button + text + current value)
+ * Draw a row in the settings panel.
+ *
+ * See PatchPage::Draw() for an explanation about how drawing is performed.
+ *
+ * The \a parent_last parameter ensures that the vertical lines at the left are
+ * only drawn when another entry follows, that it prevents output like
+ * \verbatim
+ * |-- setting
+ * |-- (-) - Title
+ * | |-- setting
+ * | |-- setting
+ * \endverbatim
+ * The left-most vertical line is not wanted. It is prevented by setting the
+ * appropiate bit in the \a parent_last parameter.
+ *
* @param patches_ptr Pointer to current values of all settings
- * @param x Left-most position in window/panel to start drawing
- * @param y Upper-most position in window/panel to start drawing
+ * @param base_x Left-most position in window/panel to start drawing \a first_row
+ * @param base_y Upper-most position in window/panel to start drawing \a first_row
+ * @param first_row First row number to draw
+ * @param max_row Row-number to stop drawing (the row-number of the row below the last row to draw)
+ * @param cur_row Current row number (internal variable)
+ * @param parent_last Last-field booleans of parent page level (page level \e i sets bit \e i to 1 if it is its last field)
+ * @return Row number of the next row to draw
*/
-void PatchEntry::Draw(GameSettings *patches_ptr, int x, int y)
+uint PatchEntry::Draw(GameSettings *patches_ptr, int base_x, int base_y, uint first_row, uint max_row, uint cur_row, uint parent_last)
{
- assert((this->flags & PEF_KIND_MASK) == PEF_SETTING_KIND);
- DrawPatch(patches_ptr, this->d.entry.setting, x, y, (this->flags & PEF_BUTTONS_MASK));
+ if (cur_row >= max_row) return cur_row;
+
+ int x = base_x;
+ int y = base_y;
+ if (cur_row >= first_row) {
+ int colour = _colour_gradient[COLOUR_ORANGE][4];
+ y = base_y + (cur_row - first_row) * SETTING_HEIGHT; // Compute correct y start position
+
+ /* Draw vertical for parent nesting levels */
+ for (uint lvl = 0; lvl < this->level; lvl++) {
+ if (!HasBit(parent_last, lvl)) GfxDrawLine(x + 4, y, x + 4, y + SETTING_HEIGHT - 1, colour);
+ x += LEVEL_WIDTH;
+ }
+ /* draw own |- prefix */
+ int halfway_y = y + SETTING_HEIGHT / 2;
+ int bottom_y = (flags & PEF_LAST_FIELD) ? halfway_y : y + SETTING_HEIGHT - 1;
+ GfxDrawLine(x + 4, y, x + 4, bottom_y, colour);
+ /* Small horizontal line from the last vertical line */
+ GfxDrawLine(x + 4, halfway_y, x + LEVEL_WIDTH - 4, halfway_y, colour);
+ x += LEVEL_WIDTH;
+ }
+
+ switch(this->flags & PEF_KIND_MASK) {
+ case PEF_SETTING_KIND:
+ if (cur_row >= first_row) {
+ DrawPatch(patches_ptr, this->d.entry.setting, x, y, this->flags & PEF_BUTTONS_MASK);
+ }
+ cur_row++;
+ break;
+ case PEF_SUBTREE_KIND:
+ if (cur_row >= first_row) {
+ DrawSprite((this->d.sub.folded ? SPR_CIRCLE_FOLDED : SPR_CIRCLE_UNFOLDED), PAL_NONE, x, y);
+ DrawString(x + 12, y, this->d.sub.title, TC_FROMSTRING);
+ }
+ cur_row++;
+ if (!this->d.sub.folded) {
+ if (this->flags & PEF_LAST_FIELD) {
+ assert(this->level < sizeof(parent_last));
+ SetBit(parent_last, this->level); // Add own last-field state
+ }
+
+ cur_row = this->d.sub.page->Draw(patches_ptr, base_x, base_y, first_row, max_row, cur_row, parent_last);
+ }
+ break;
+ default: NOT_REACHED();
+ }
+ return cur_row;
}
/**
@@ -865,6 +932,35 @@ PatchEntry *PatchPage::FindEntry(uint row_num, uint *cur_row) const
return pe;
}
+/**
+ * Draw a selected part of the settings page.
+ *
+ * The scrollbar uses rows of the page, while the page data strucure is a tree of #PatchPage and #PatchEntry objects.
+ * As a result, the drawing routing traverses the tree from top to bottom, counting rows in \a cur_row until it reaches \a first_row.
+ * Then it enables drawing rows while traversing until \a max_row is reached, at which point drawing is terminated.
+ *
+ * @param patches_ptr Pointer to current values of all settings
+ * @param base_x Left-most position in window/panel to start drawing of each setting row
+ * @param base_y Upper-most position in window/panel to start drawing of row number \a first_row
+ * @param first_row Number of first row to draw
+ * @param max_row Row-number to stop drawing (the row-number of the row below the last row to draw)
+ * @param cur_row Current row number (internal variable)
+ * @param parent_last Last-field booleans of parent page level (page level \e i sets bit \e i to 1 if it is its last field)
+ * @return Row number of the next row to draw
+ */
+uint PatchPage::Draw(GameSettings *patches_ptr, int base_x, int base_y, uint first_row, uint max_row, uint cur_row, uint parent_last) const
+{
+ if (cur_row >= max_row) return cur_row;
+
+ for (uint i = 0; i < this->num; i++) {
+ cur_row = this->entries[i].Draw(patches_ptr, base_x, base_y, first_row, max_row, cur_row, parent_last);
+ if (cur_row >= max_row) {
+ break;
+ }
+ }
+ return cur_row;
+}
+
static PatchEntry _patches_ui[] = {
PatchEntry("gui.vehicle_speed"),
@@ -1073,13 +1169,8 @@ struct PatchesSelectionWindow : Window {
/* Set up selected category */
this->DrawWidgets();
-
- int x = SETTINGTREE_LEFT_OFFSET;
- int y = SETTINGTREE_TOP_OFFSET;
- for (uint i = this->vscroll.pos; i != page->Length() && this->vscroll.pos + this->vscroll.cap - i > 0; i++) {
- page->entries[i].Draw(patches_ptr, x, y);
- y += SETTING_HEIGHT;
- }
+ page->Draw(patches_ptr, SETTINGTREE_LEFT_OFFSET, SETTINGTREE_TOP_OFFSET,
+ this->vscroll.pos, this->vscroll.pos + this->vscroll.cap);
}
virtual void OnClick(Point pt, int widget)
@@ -1089,9 +1180,6 @@ struct PatchesSelectionWindow : Window {
int y = pt.y - SETTINGTREE_TOP_OFFSET; // Shift y coordinate
if (y < 0) return; // Clicked above first entry
- int x = pt.x - SETTINGTREE_LEFT_OFFSET; // Shift x coordinate
- if (x < 0) return; // Clicked left of the entry
-
byte btn = this->vscroll.pos + y / SETTING_HEIGHT; // Compute which setting is selected
if (y % SETTING_HEIGHT > SETTING_HEIGHT - 2) return; // Clicked too low at the setting
@@ -1101,6 +1189,9 @@ struct PatchesSelectionWindow : Window {
if (pe == NULL) return; // Clicked below the last setting of the page
+ int x = pt.x - SETTINGTREE_LEFT_OFFSET - (pe->level + 1) * LEVEL_WIDTH; // Shift x coordinate
+ if (x < 0) return; // Clicked left of the entry
+
if ((pe->flags & PEF_KIND_MASK) == PEF_SUBTREE_KIND) {
pe->d.sub.folded = !pe->d.sub.folded; // Flip 'folded'-ness of the sub-page
@@ -1233,11 +1324,11 @@ const int PatchesSelectionWindow::SETTINGTREE_TOP_OFFSET = 47;
static const Widget _patches_selection_widgets[] = {
{ WWT_CLOSEBOX, RESIZE_NONE, COLOUR_MAUVE, 0, 10, 0, 13, STR_00C5, STR_018B_CLOSE_WINDOW},
-{ WWT_CAPTION, RESIZE_RIGHT, COLOUR_MAUVE, 11, 381, 0, 13, STR_CONFIG_PATCHES_CAPTION, STR_018C_WINDOW_TITLE_DRAG_THIS},
-{ WWT_PANEL, RESIZE_RIGHT, COLOUR_MAUVE, 0, 381, 14, 41, 0x0, STR_NULL},
-{ WWT_PANEL, RESIZE_RB, COLOUR_MAUVE, 0, 369, 42, 215, 0x0, STR_NULL}, // PATCHSEL_OPTIONSPANEL
-{ WWT_SCROLLBAR, RESIZE_LRB, COLOUR_MAUVE, 370, 381, 42, 203, 0x0, STR_0190_SCROLL_BAR_SCROLLS_LIST}, // PATCHSEL_SCROLLBAR
-{ WWT_RESIZEBOX, RESIZE_LRTB, COLOUR_MAUVE, 370, 381, 204, 215, 0x0, STR_RESIZE_BUTTON}, // PATCHSEL_RESIZE
+{ WWT_CAPTION, RESIZE_RIGHT, COLOUR_MAUVE, 11, 396, 0, 13, STR_CONFIG_PATCHES_CAPTION, STR_018C_WINDOW_TITLE_DRAG_THIS},
+{ WWT_PANEL, RESIZE_RIGHT, COLOUR_MAUVE, 0, 396, 14, 41, 0x0, STR_NULL},
+{ WWT_PANEL, RESIZE_RB, COLOUR_MAUVE, 0, 384, 42, 215, 0x0, STR_NULL}, // PATCHSEL_OPTIONSPANEL
+{ WWT_SCROLLBAR, RESIZE_LRB, COLOUR_MAUVE, 385, 396, 42, 203, 0x0, STR_0190_SCROLL_BAR_SCROLLS_LIST}, // PATCHSEL_SCROLLBAR
+{ WWT_RESIZEBOX, RESIZE_LRTB, COLOUR_MAUVE, 385, 396, 204, 215, 0x0, STR_RESIZE_BUTTON}, // PATCHSEL_RESIZE
{ WWT_TEXTBTN, RESIZE_NONE, COLOUR_YELLOW, 10, 100, 16, 27, STR_CONFIG_PATCHES_GUI, STR_NULL}, // PATCHSEL_INTERFACE
{ WWT_TEXTBTN, RESIZE_NONE, COLOUR_YELLOW, 101, 191, 16, 27, STR_CONFIG_PATCHES_CONSTRUCTION, STR_NULL}, // PATCHSEL_CONSTRUCTION
@@ -1249,7 +1340,7 @@ static const Widget _patches_selection_widgets[] = {
};
static const WindowDesc _patches_selection_desc = {
- WDP_CENTER, WDP_CENTER, 382, 216, 382, 425,
+ WDP_CENTER, WDP_CENTER, 397, 216, 397, 425,
WC_GAME_OPTIONS, WC_NONE,
WDF_STD_TOOLTIPS | WDF_STD_BTN | WDF_DEF_WIDGET | WDF_RESIZABLE,
_patches_selection_widgets,
diff --git a/src/table/sprites.h b/src/table/sprites.h
index 31f59134a..7d5d8184a 100644
--- a/src/table/sprites.h
+++ b/src/table/sprites.h
@@ -49,7 +49,7 @@ enum Sprites {
/* Extra graphic spritenumbers */
SPR_OPENTTD_BASE = 4896,
- OPENTTD_SPRITE_COUNT = 147,
+ OPENTTD_SPRITE_COUNT = 149,
/* Halftile-selection sprites */
SPR_HALFTILE_SELECTION_FLAT = SPR_OPENTTD_BASE,
@@ -73,6 +73,9 @@ enum Sprites {
SPR_PIN_UP = SPR_OPENTTD_BASE + 51, // pin icon
SPR_PIN_DOWN = SPR_OPENTTD_BASE + 52,
+ SPR_CIRCLE_FOLDED = SPR_OPENTTD_BASE + 147, // (+) icon
+ SPR_CIRCLE_UNFOLDED = SPR_OPENTTD_BASE + 148, // (-) icon
+
/* on screen keyboard icons */
SPR_OSK_LEFT = SPR_OPENTTD_BASE + 138,
SPR_OSK_RIGHT = SPR_OPENTTD_BASE + 139,