cpp/openttd-patchpack/source Changeset - r28762:67d8065b03e1

Changeset - r28762:67d8065b03e1

Parent rev.

Child rev.

[Not reviewed]

master

0 2 0

Peter Nelson - 10 months ago 2024-02-14 17:23:03
peter1138@openttd.org

Codechange: Update window's widget lookup map when switching displayed plane.

2 files changed with 9 insertions and 0 deletions:

src/widget.cpp

src/widget_type.h

0 comments (0 inline, 0 general)

src/widget.cpp

➞

Show inline comments

@@ @@ -1265,14 +1265,19 @@ void NWidgetStacked::AssignSizePosition( @@
 		child_wid->AssignSizePosition(sizing, x + child_pos_x, y + child_pos_y, child_width, child_height, rtl);
+	}
+}
 void NWidgetStacked::FillWidgetLookup(WidgetLookup &widget_lookup)
+{
 	/* We need to update widget_lookup later. */
 	this->widget_lookup = &widget_lookup;
 	if (this->index >= 0) widget_lookup[this->index] = this;
 	NWidgetContainer::FillWidgetLookup(widget_lookup);
 	/* In case widget IDs are repeated, make sure Window::GetWidget works on displayed widgets. */
 	if (static_cast<size_t>(this->shown_plane) < this->children.size()) this->children[shown_plane]->FillWidgetLookup(widget_lookup);
+}
 void NWidgetStacked::Draw(const Window *w)
+{
 	if (this->shown_plane >= SZSP_BEGIN) return;
@@ @@ -1297,12 +1302,14 @@ NWidgetCore *NWidgetStacked::GetWidgetFr @@
  * @return true iff the shown plane changed.
  */
 bool NWidgetStacked::SetDisplayedPlane(int plane)
+{
 	if (this->shown_plane == plane) return false;
 	this->shown_plane = plane;
 	/* In case widget IDs are repeated, make sure Window::GetWidget works on displayed widgets. */
 	if (static_cast<size_t>(this->shown_plane) < this->children.size()) this->children[shown_plane]->FillWidgetLookup(*this->widget_lookup);
 	return true;
+}
 NWidgetPIPContainer::NWidgetPIPContainer(WidgetType tp, NWidContainerFlags flags) : NWidgetContainer(tp)
+{
 	this->flags = flags;

src/widget_type.h

➞

Show inline comments

@@ @@ -494,12 +494,14 @@ public: @@
 	NWidgetCore *GetWidgetFromPos(int x, int y) override;
 	bool SetDisplayedPlane(int plane);
 	int shown_plane; ///< Plane being displayed (for #NWID_SELECTION only).
 	const WidgetID index; ///< If non-negative, index in the #Window::widget_lookup.
 private:
 	WidgetLookup *widget_lookup; ///< Window's widget lookup, updated in SetDisplayedPlane().
 };
 /** Nested widget container flags, */
 enum NWidContainerFlags {
 	NCB_EQUALSIZE = 0, ///< Containers should keep all their (resizing) children equally large.
 	NCB_BIGFIRST  = 1, ///< Allocate space to biggest resize first.

0 comments (0 inline, 0 general)