After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 669624 - Preview next block doesn't work
Preview next block doesn't work
Status: RESOLVED FIXED
Product: quadrapassel
Classification: Applications
Component: general
git master
Other Linux
: Normal normal
: ---
Assigned To: GNOME Games maintainers
GNOME Games maintainers
Depends on:
Blocks:
 
 
Reported: 2012-02-08 01:55 UTC by Jeremy Bicha
Modified: 2013-05-24 01:33 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch disabling preview toggle when difficult block mode is enabled (1.41 KB, patch)
2013-04-21 17:19 UTC, Mario Wenzel
committed Details | Review
Don't pick difficult blocks by default (987 bytes, patch)
2013-05-14 18:33 UTC, Michael Catanzaro
committed Details | Review
Flip the two settings (1.94 KB, patch)
2013-05-14 18:36 UTC, Michael Catanzaro
committed Details | Review
allow previews of difficult blocks (1.15 KB, patch)
2013-05-22 19:50 UTC, Michael Catanzaro
committed Details | Review

Description Jeremy Bicha 2012-02-08 01:55:56 UTC
I have Preview Next Block checked in Quadrapassel's preferences but the black preview area in the top right corner stays black and empty. I believe this worked before the switch to Vala.

quadrapassel 3.3.5 on Ubuntu 12.04 "Alpha" on amd64.
Comment 1 Robert Ancell 2012-02-08 02:12:50 UTC
This is working for me and also working for the packaged version.  Any log messages / can you test on another video driver?
Comment 2 Jeremy Bicha 2012-02-08 05:28:26 UTC
I don't see any log messages when run like normal from the terminal.

My primary computer is using Intel graphics but I also verified this bug on my wife's computer which uses Nouveau.
Comment 3 Bin Li 2012-08-14 10:17:58 UTC
Jeremy,

So do you use the 'difficult block' mode? When I use the 'difficult block',
there are no image of the next block, when I cancel the selection, it show
again.
Comment 4 Mario Wenzel 2013-04-21 17:19:14 UTC
Created attachment 242090 [details] [review]
Patch disabling preview toggle when difficult block mode is enabled

This patch should make it abundantly clear that preview-mode is dependent on difficult block mode being disabled.
(it disables the preview-mode-toggle if difficult-block-mode is enabled)

Either that or we actually do precalculate one block to be shown, which should prove rather difficult.
Comment 5 Michael Catanzaro 2013-05-14 18:12:08 UTC
Review of attachment 242090 [details] [review]:

Well if it'd be too much work to have both options at the same time, then this fix works great.

There is only one problem: if checking a checkbox marks other checkboxes as insensitive, then that box should be above (or to the left of) the affected boxes.  So we just need to flip tho order of these checkboxes in the UI (or have difficult block mode depend on preview mode, instead of the other way around, but I think your way makes more sense).  We should also change the default settings so that whichever option is disabled by default isn't checked by default.  Both of those changes are trivial and can be done separately from this patch, but should be done before this is committed.

I also notice that the black box should also be hidden when this previews are off, but that's a completely separate bug.
Comment 6 Michael Catanzaro 2013-05-14 18:33:20 UTC
Created attachment 244226 [details] [review]
Don't pick difficult blocks by default
Comment 7 Michael Catanzaro 2013-05-14 18:36:01 UTC
Created attachment 244227 [details] [review]
Flip the two settings

To be applied after the original patch
Comment 8 Michael Catanzaro 2013-05-22 19:50:18 UTC
Created attachment 245083 [details] [review]
allow previews of difficult blocks

Mario,

Thanks for the assistance with fixing the preferences dialog to make clear that these two options can't be used at the same time!

While looking at this further, I found that it was quite easy to allow both options to function properly at the same time, so I'm going to push that change instead.
Comment 9 Michael Catanzaro 2013-05-22 19:50:44 UTC
This problem has been fixed in our software repository. The fix will go into the next software release. Thank you for your bug report.
Comment 10 Mario Wenzel 2013-05-22 20:17:24 UTC
But this changes the concept of difficult block mode as the most difficult block isn't picked on the current layout of the board but on the layout of the board before the current piece. So the hard-block-selector can't choose the actual hardest block. So you can prepare beforehand and it becomes like standard mode, doesn't it?
Comment 11 Thomas Andersen 2013-05-22 20:22:01 UTC
Maybe we could just show a big question mark in the preview for the hard-block mode?
Comment 12 Mario Wenzel 2013-05-22 20:33:40 UTC
Either that or disabling preview (both viable):. But I don't think, we should make it easier. A question mark is a nice alternative.
Comment 13 Michael Catanzaro 2013-05-22 21:03:16 UTC
(In reply to comment #10)
> But this changes the concept of difficult block mode as the most difficult
> block isn't picked on the current layout of the board but on the layout of the
> board before the current piece. 

Good point, it's based on the turn prior.

Seems best to just revert my patch and go with the ones we already had prepared. Mario's "hide preview entirely" patch works well enough that we won't need a question mark.
Comment 14 Michael Catanzaro 2013-05-22 21:09:41 UTC
OK, damage undone. We still need to make checking difficult block mode disable the previews.
Comment 15 Michael Catanzaro 2013-05-22 21:23:34 UTC
Well, no, it looks like difficult block mode is set on a per-game basis, so with both of Mario's patches, everything just works.  Mario, if anything's still wrong with either master or gnome-3-8, please let us know.