Bug 342162 - User can remove tiles during game over replay animation
Summary: User can remove tiles during game over replay animation
Status: RESOLVED FIXED
Alias: None
Product: kmahjongg
Classification: Applications
Component: general (show other bugs)
Version: 0.8
Platform: Compiled Sources Linux
: NOR minor
Target Milestone: ---
Assignee: Unassigned bugs mailing-list
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-12-23 16:39 UTC by Jan-Peter Nilsson
Modified: 2016-01-04 00:38 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Start a new game if clicking on the game over replay (851 bytes, patch)
2014-12-23 23:50 UTC, Jan-Peter Nilsson
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jan-Peter Nilsson 2014-12-23 16:39:45 UTC
The user can remove tiles during the game over replay animation.
Manually removing tiles during the game over replay animation interferes with the animation so that tiles are left behind, manually removing the tiles left behind result in removed tiles being shown as a large negative number.

Reproducible: Always

Steps to Reproduce:
1. Finish a game
2. During the game over animated replay of the game select a matching pair of tiles to remove them

Actual Results:  
It is possible to select and remove matching tiles.

Expected Results:  
The user should not be allowed to select or remove tiles during the replay.
I would expect one of two things to happen when clicking the game board during the replay:
1. Nothing
2. The replay is aborted and a new game is created

Of those I prefer 2 as it is consistent with the behaviour when clicking on the game board when a demo is playing.
Comment 1 Jan-Peter Nilsson 2014-12-23 23:50:01 UTC
Created attachment 90107 [details]
Start a new game if clicking on the game over replay
Comment 2 Frederik Schwarzer 2016-01-04 00:38:10 UTC
This fix is now in master (the animation aborts on click).

Please test the master version and reopen the report in case yu think the issue is not fixed properly.