Bug 388263 - Pause, Hint do not work after continuing a lost game
Summary: Pause, Hint do not work after continuing a lost game
Status: RESOLVED FIXED
Alias: None
Product: kmahjongg
Classification: Applications
Component: general (show other bugs)
Version: 0.9
Platform: Slackware Linux
: NOR normal
Target Milestone: ---
Assignee: Unassigned bugs mailing-list
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-12-27 00:22 UTC by sombragris
Modified: 2018-05-13 11:47 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description sombragris 2017-12-27 00:22:17 UTC
Upon starting a game, it seems to run OK. However, after undoing a movement (using Ctrl-Z), both fuctions of Pause and Hint turn greyed out and unavailable.

The expected behavior is that both functions should remain available to the user even after undoing a movement.

Thanks,

Eduardo
Comment 1 sombragris 2017-12-27 00:40:31 UTC
I would like to add that this behavior was not present in previous versions so it could be seen as a regression.
Comment 2 Burkhard Lück 2017-12-27 14:10:08 UTC
Using kmahjongg master build from sources and neon devedition git unstable Pause and Hint are still available after undoing a move. Seems to be an isue with your distribution.
Comment 3 sombragris 2017-12-27 20:04:37 UTC
Burkhard is right. I edited the description to update the bug case.

Undo is *not* the trigger for the bug.

The bug appears when one plays and suddenly there are no moves left. In that case, one is presented with a choice of a new game, restart, or cancel. One way of keep playing the game is by pressing Cancel and then undoing one or more movements until the game could be won once more.

Well, when there are no moves lest, and upon selecting Cancel, the game loses its ability to give hints and pause.

Thanks,

Eduardo
Comment 4 Albert Astals Cid 2017-12-27 22:10:45 UTC
Proposed fix at https://phabricator.kde.org/D9525
Comment 5 Albert Astals Cid 2018-01-04 17:50:38 UTC
Landed for KDE Applications 17.12.1
Comment 6 sombragris 2018-01-04 19:22:16 UTC
Thank you Albert!!