The International Simutrans Forum

Development => Bug Reports => Topic started by: gauthier on April 02, 2017, 08:50:26 AM

Title: r8163 lines list scrollbar slider in depots not working
Post by: gauthier on April 02, 2017, 08:50:26 AM
Simutrans rr8163 (120.2.2) for Linux 32 bits

Open a depot's window.
Click on the line field to show the lines'list
Try to click and drag the slider of the vertical scrollbar -> the slider doesn't move, the list doesn't scroll.
Title: Re: r8163 lines list scrollbar slider in depots not working
Post by: prissi on April 02, 2017, 11:37:59 AM
Seeing these bugs only after releases makes me questioning who actually is using the nightlies any more.

Fixed in r8169 hopefully. But there could be more GUI bugs.
Title: Re: r8163 lines list scrollbar slider in depots not working
Post by: Ters on April 02, 2017, 11:46:37 AM
I've been a bit of a coward myself, staying a few revision behind so that I don't lose lot of playing time to newly introduced bugs.
Title: Re: r8163 lines list scrollbar slider in depots not working
Post by: gauthier on April 02, 2017, 11:58:58 AM
Well ... I don't know how the code is and I don't want to blame any one here, but I feel like the developer(s) who bring modifications to the GUI code does not take the right time to test what he has done. If you are waiting for people to regularly download a nightly and test if everything goes well, without knowing what the developers did between that nithly and the previous one they used, you are going to be deceived.

I use nightlies only to see if a bug I just saw is already fixed or if a reported bug is told being fixed in that nightly.
Title: Re: r8163 lines list scrollbar slider in depots not working
Post by: Ters on April 02, 2017, 12:28:35 PM
It is a fact of life that the developer himself will never test the code good enough. That is why concepts like code reviews, automated testing (on several levels) and beta/nightly/early-access releases have been invented. Unfortunately, Simutrans has no automated test suites, and it is very difficult to make one the way the game is built. Without full time developers, code reviews are difficult as well, and only done for new developers or very big changes (at least occasionally).
Title: Re: r8163 lines list scrollbar slider in depots not working
Post by: prissi on April 02, 2017, 02:36:30 PM
The test games I do had not enough lines to make the scrollbars appear. The problem is, that some internal routines are a hack, and by doing things properly there might be side effects. (For instance, so far the focussing of elements, i.e. using buttons by keyboard was largely broken, and comboboxes were closed in many places just to be sure they close.)

I also highly doubt that automatied test of such things would be less complex than the actual code, and hence require tests of the tests ...
Title: Re: r8163 lines list scrollbar slider in depots not working
Post by: Ters on April 02, 2017, 06:12:20 PM
Quote from: prissi on April 02, 2017, 02:36:30 PM
I also highly doubt that automatied test of such things would be less complex than the actual code, and hence require tests of the tests ...

The thing is, once the tests have been written, they will tell you when thing breaks later on. So complex, yes, but very valuable.
Title: Re: r8163 lines list scrollbar slider in depots not working
Post by: gauthier on April 02, 2017, 08:24:42 PM
Automated tests for GUIs are quite tedious. As a developer who hates GUIs, I won't blame you for not doing that lol  ;D

Anyway, before releasing a version, developers can let themselves some days to freely play the game, it would be enough to see such bugs before the release.
Title: Re: r8163 lines list scrollbar slider in depots not working
Post by: Ters on April 02, 2017, 08:47:12 PM
The thing that probably complicated things here is that there was another bug already out that needed fixing. Since Simutrans lacks stable branches, that means that there is a choice between letting players struggle with a known bug and potentially introducing a new one. If the developers are to playtest for a few days, that could easily postpone the fix for a month.
Title: Re: r8163 lines list scrollbar slider in depots not working
Post by: gauthier on April 03, 2017, 05:12:25 PM
What is needed is a sufficiently stable release once, and then release every version after one or two days of playtesting. The point here is to spot bugs that are currently reported just after the release, to avoid this cycle of release > bugfix release > another bugfix release > etc ... Obtaining this "stable cycle" means letting players with bugs for a short period of time, but only once, that's the price.
Title: Re: r8163 lines list scrollbar slider in depots not working
Post by: Ters on April 03, 2017, 05:54:43 PM
While a day or two of playtesting would have uncovered this bug, it might not have been enough for what started this cycle. Simutrans have a lot of different playing styles. Some players may spend a lot of time in windows and with tools that the developers don't even remember exists.

And I've seen rather serious bugs slip through a far more elaborate testing regime at work the last few weeks. The last one even included an entire major new feature missing from the delivered product, and no one noticing for over a month!
Title: Re: r8163 lines list scrollbar slider in depots not working
Post by: An_dz on April 03, 2017, 08:47:39 PM
We could tell people here at the forum to download the nightly version two days before a release to let them find bugs, then if nobody says anything we release a stable version for everybody that don't follow the forum.

BTW: The XML version needs a bump for taking advantage of the 16 bit colours. Not sure where this is done.
Title: Re: r8163 lines list scrollbar slider in depots not working
Post by: prissi on April 03, 2017, 11:46:19 PM
You can't, you need to increase the entire savegame version.
Title: Re: r8163 lines list scrollbar slider in depots not working
Post by: Dwachs on April 04, 2017, 06:17:08 AM
@An_dz please increase savegame version in simversion.h to meet the needs of the 16bit change.