The new statistics slider does not update if the window is left open

Bug #887728 reported by Hans Joachim Desserud
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
widelands
Fix Released
Medium
Unassigned

Bug Description

(Split out from bug 536543)
1. Start a new game.
2. Open Statistics->General Statistics.
3. Leave the window open and increase the game speed to 50x and leave it for a while

Expected result:
New options should be added to the slider as time goes, when they make sense.

Actual result:
While the window is open, the slider does not update. However, if you close the window and reopen it, any new options that make sense will have been added.

It was commented on bug 536543 that this was a known issue, and would probably be hard to fix with the current code.

Widelands r6090.

Tags: statistics ui

Related branches

Revision history for this message
Joachim Breitner (nomeata) wrote :

Just a remark for later: It should actually be slightly easier now; a call to layout() in the DiscreteSlider class should take care of any rearranging. The only problem left is to communicate the change from the WUI_Plot class to the associated slider. But a simple pointer could work, assuming (or enforcing) that at most one slider is generated for a given Plot instance.

Revision history for this message
Joachim Breitner (nomeata) wrote :

Ok, I think I got it, see the related merge request.

Revision history for this message
SirVer (sirver) wrote :

r6091

Changed in widelands:
status: New → Fix Committed
Changed in widelands:
milestone: none → build17-rc1
Revision history for this message
LAZA (laza74) wrote :

I hope, i understood the bug reporter correct - then this bug is not fixed in bzr6335!

- start game
- open statistics (Warenstatistik)
- play

There are now two possibilities to set the time:
15 min - game

Closing the statistics and reopen it, i got after 50 min those possibilities:
15 min - 30 min - game

Can somebody please confirm?!?

Revision history for this message
SirVer (sirver) wrote :

Released in build17-rc1.

Changed in widelands:
status: Fix Committed → Fix Released
Revision history for this message
Hans Joachim Desserud (hjd) wrote :

Regarding #4, since the slider was added, it now shows only a few options intially then add more as they become useful. (Compared to the old buttons which always had the same set of buttons always available) As an example, seeing the stats for the latest 8 hours doesn't do much good if your game has only lasted half an hour. On the other hand, once you have played 8 hours, it would be useful to add that as an option, so that happens automatically.

The original bug here was that if you had the statistics window open as time went by, the new options wouldn't be added unless you closed the window and reopened it again. So new options adding themselves over time is expected behaviour, regardless whether the statistics window is open or not.

>Closing the statistics and reopen it, i got after 50 min those possibilities:
>15 min - 30 min - game

So after 50 mins of in-game time, those three options should be available whether or not the statistics window has been open the whole time or not. If this is not the case, please reply/elaborate.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.