[Kde-games-devel] Review Request 114513: Facility to preview a completed Palapeli jigsaw puzzle

Ian Wadham iandw.au at gmail.com
Sat Dec 28 04:23:43 UTC 2013



> On Dec. 27, 2013, 4:42 p.m., Albert Astals Cid wrote:
> > src/engine/puzzlepreview.cpp, line 75
> > <https://git.reviewboard.kde.org/r/114513/diff/1/?file=225781#file225781line75>
> >
> >     This is wrong, it should be
> >     i18n("%1 - Preview", md.name)

This has been included in my local branch and tested.


> On Dec. 27, 2013, 4:42 p.m., Albert Astals Cid wrote:
> > src/palapeliui.rc, line 29
> > <https://git.reviewboard.kde.org/r/114513/diff/1/?file=225784#file225784line29>
> >
> >     Shouldn't we add this action somewhere in the menus too? I know there's some people that hide toolbars by default so if it's not in the menus they can't enable it.

I plan to have a View menu later on, as part of some work to make handling of large puzzles easier. It should include this option and others: new and existing.


- Ian


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/114513/#review46199
-----------------------------------------------------------


On Dec. 16, 2013, 9:48 p.m., Ian Wadham wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/114513/
> -----------------------------------------------------------
> 
> (Updated Dec. 16, 2013, 9:48 p.m.)
> 
> 
> Review request for KDE Games.
> 
> 
> Repository: palapeli
> 
> 
> Description
> -------
> 
> Facility to preview a completed Palapeli jigsaw puzzle.
> 
> Original by Johannes Loehnert, 2010, modified by Ian Wadham to use current Palapeli interfaces.
> 
> The preview is controlled by a button in the toolbar, which is enabled when a puzzle is being solved. It appears in a floating tool-window and can be shown or hidden at will. The window can be moved, resized, maximized or closed (hidden, but not deleted). It is deleted when the user returns to the puzzle-collection view or terminates Palapeli when in solving mode. The window's title is derived from the name of the puzzle, e.g. "Citrus Fruits - Preview". Its visibility and geometry are saved in Settings between uses. If visibility is on in Settings, the puzzle preview appears immediately a puzzle is requested and can be viewed while the puzzle is loading.
> 
> When the mouse pointer is moved into the preview window, the view is automatically magnified to show an area of at least 3x3 pieces. As the pointer moves around, the view automatically scrolls, to allow the user to home in rapidly on any detail of the puzzle's image.
> 
> 
> Diffs
> -----
> 
>   src/window/mainwindow.cpp e3305fb 
>   src/window/mainwindow.h f94e52c 
>   src/palapeliui.rc d888c0d 
>   src/main.cpp f7fec86 
>   src/palapeli.kcfg b30d6b1 
>   src/engine/puzzlepreview.cpp PRE-CREATION 
>   src/engine/puzzlepreview.h PRE-CREATION 
>   src/CMakeLists.txt c648c91 
> 
> Diff: https://git.reviewboard.kde.org/r/114513/diff/
> 
> 
> Testing
> -------
> 
> Developed and tested on an Apple Macbook OS X platform, using KDE from Macports.
> 
> Tested all the features described above, including title bar buttons. Also tested cases where image and Palapeli metadata are unavailable, in which case the preview contains a text message and a generic title.
> 
> On Apple, the Minimize button is disabled and has not been tested. I am not sure if it will be enabled in Linux, nor what its effect should be.
> 
> 
> Thanks,
> 
> Ian Wadham
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20131228/624721d1/attachment.html>


More information about the kde-games-devel mailing list