[Patch] Improved Build Station GUI [r19159]

[Vaulter]

Here patch against 1.0.2 + some additions
Thanks for such good stuff!

1.0.2-station-build.png (216.15 KiB) Viewed 1951 times

[maquinista]

Thanks. It looks nice, but I see a problem: There are a empty space in the buttons, and tall sprites are cut. If the sprites are drawn 25 pixels down, most of them would fit in the space.

[Ammler]

Thanks. It looks nice, but I see a problem: There are a empty space in the buttons, and tall sprites are cut. If the sprites are drawn 25 pixels down, most of them would fit in the space.

Be aware that the patch from Vaulter is just a backport to 1.0.2, the trunk patch from sbr doesn't have this glitch:

[ChillCore]

Hello scoz,

Attached is the patch (by sbr) updated to current trunk.

I have added the needed lines and code changes to make the patch work in current trunk.
Also I have unified the breaks in "case ...:" with the recent trunk change and I have added a few accolades that were missing in trunk (see line 366 in the patch for an example)
Also I have added 1 todo which is not a really a todo but a to check if I did it correctly.
Otherwise no changes have been made and functionality is the same as before.

ps:
Vaulter, changing the possible number of tracks to build or max lenght in one go is something for a separate patch.

[Alberth]

.... and I have added a few accolades that were missing in trunk (see line 366 in the patch for an example)

Just to clear this up, those curly brackets are not missing.

The aim in trunk is not to add curly brackets everywhere, but only when needed, ie when a local variable is defined inside a case.

[ChillCore]

The aim in trunk is not to add curly brackets everywhere, but only when needed, ie when a local variable is defined inside a case.

I see. Thank you for the clarification Alberth.
Attached patch removes the unneeded curly braces I added to trunk outside the patch.
I have not yet looked at the rest of the code to see if some more could/should be removed.

[Terkhen]

After discussing this feature for a while, we have two suggestions to make.

The first one is somewhat complicated. Waypoints are basically the same thing than stations, and as a result their building window could also benefit from some of the features introduced by this patch. Because of this, the build waypoint window should be merged into the build station window as additional prerrequisite patches, and after that it would be trivial for both waypoints and station to use the new features. In the following screenshot, the suggested build waypoint window would only use the right part of the current build station window.



Codewise, both windows could share the same code but use different NWidgetPart arrays during creation (as done for the vehicle details window). Using NWID_SELECTION is a possibility too.

About the station name, we think that it should be shown more clearly. A possibility could be displaying the name of the currently selected station under the matrix. This also has the advantage of being in the part of the window that waypoints would need to use.

Before reaching these conclusions, I updated the three parts and fixed a few issues with 02 (the most important being that OpenTTD would crash if the station class dropdown was down and a station NewGRF was removed). IMO these three patches are ready for trunk inclusion, but I have not commited them just in case they need to be reworked because of the waypoint window unification. They can be found here.

[sbr]

An update against r21062:

• Show station name under the station class matrix.
• Move station type matrix code inside a separate widget.
• Use this widget to display waypoints in their window.
• Make the waypoint window support NewGRF reloading, resizable and able to restore its previous size.

[Kogut]

Nice!

[Lord Aro]

i love itwhen some random newbie (no offense) coder appears with an updated patch...
Nice work!
Hope this makes trunk soon...

[Dante123]

would be great to have this default in Ottd (and for the NewObjects too )

[glx]

An update against r21062:

• Show station name under the station class matrix.
• Move station type matrix code inside a separate widget.
• Use this widget to display waypoints in their window.
• Make the waypoint window support NewGRF reloading, resizable and able to restore its previous size.

You forgot to svn add the new file, so it's not in the diff.

[sbr]

I should have better check the svn diff. Attached is, hopefully, the correct patch file (the previous post is also updated).

There is still a lot of things to clean up to properly merge the BuildWaypoint and BuildRailStation windows. That patch present a way to do it even if far from perfect. Thanks for reviewing it.

[Dante123]

any chance this patch will ever make it to trunk ? seems to me (as just a plain user) far better than the current one in very way.
but dont know about any other clashes in code that i dont notice as just that plain user.

[Alberth]

any chance this patch will ever make it to trunk ? seems to me (as just a plain user) far better than the current one in very way.

Yeah, selecting a picture is way nicer.

The matrix widget file is still not added (station-build-gui_v7.0-r21062.patch).
In addition, I wonder about airports.
Having them for other buildable things may be nice too, but I don't know whether there are any graphics for them (at least industries don't iirc).

[Terkhen]

I think that it could also be used for newobjects.

[Dante123]

I think that it could also be used for newobjects.

for that it would be extremely usefull.
for example new graphics need 1 line of text for each view in in the current way. think of a newgrf that will add sound barrier graphics as new-objects, you end up with huge lists of text where you cant find back the graphics you want i guess.

[Alberth]

I downloaded a few other patch versions, and found the widgets/station_type_matrix.cpp file. It appears to be very special purpose for station previewing.
The widget needs to be generalized first before being able to use it in an other context.

[Timmaexx]

Some parts of this Patch are now in trunk!
Thank you for your Ideas!

[Alberth]

Some parts of this Patch are now in trunk!
Thank you for your Ideas!

Although I build the patch from scratch using the new NWidgetMatrix widget, I indeed used a lot of the ideas expressed here.

Thank you for your efforts.