[Marble-devel] Review Request: Add "Download Region" feature

[Dennis Nienhüser]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3797/#review5198
-----------------------------------------------------------

Ship it!


Very nice :-)


/trunk/KDE/kdeedu/marble/src/lib/DownloadRegionDialog.cpp
<http://reviewboard.kde.org/r/3797/#comment4696>

    "Visible region"? It may not be clear where "Current" refers to.



/trunk/KDE/kdeedu/marble/src/lib/DownloadRegionDialog.cpp
<http://reviewboard.kde.org/r/3797/#comment4697>

    Maybe add "to download"?



/trunk/KDE/kdeedu/marble/src/lib/DownloadRegionDialog.cpp
<http://reviewboard.kde.org/r/3797/#comment4698>

    Using a QDialogButtonBox would get the button order (Ok/Cancel left/right) correct here. I think it still let's you pass your own button.


- Dennis


On 2010-04-24 04:35:27, jmho wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/3797/
> -----------------------------------------------------------
> 
> (Updated 2010-04-24 04:35:27)
> 
> 
> Review request for marble.
> 
> 
> Summary
> -------
> 
> This patch adds the missing bits (mostly UI related) for "Download region".
> The .ui are based on those kindly provided by Torsten Rahn.
> 
> Remaining issues:
> 1) the tile level range widget is not shown correctly in the download region dialog
> 2) UI hangs for some time while the download queue is being filled
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdeedu/marble/src/lib/AbstractScanlineTextureMapper.h 1118235 
>   /trunk/KDE/kdeedu/marble/src/lib/CMakeLists.txt 1118235 
>   /trunk/KDE/kdeedu/marble/src/lib/DownloadRegionDialog.h PRE-CREATION 
>   /trunk/KDE/kdeedu/marble/src/lib/DownloadRegionDialog.cpp PRE-CREATION 
>   /trunk/KDE/kdeedu/marble/src/lib/LatLonBoxWidget.h PRE-CREATION 
>   /trunk/KDE/kdeedu/marble/src/lib/LatLonBoxWidget.cpp PRE-CREATION 
>   /trunk/KDE/kdeedu/marble/src/lib/LatLonBoxWidget.ui PRE-CREATION 
>   /trunk/KDE/kdeedu/marble/src/lib/MarbleModel.h 1118235 
>   /trunk/KDE/kdeedu/marble/src/lib/MarbleModel.cpp 1118235 
>   /trunk/KDE/kdeedu/marble/src/lib/TileCoordsPyramid.h PRE-CREATION 
>   /trunk/KDE/kdeedu/marble/src/lib/TileCoordsPyramid.cpp PRE-CREATION 
>   /trunk/KDE/kdeedu/marble/src/lib/TileLevelRangeWidget.h PRE-CREATION 
>   /trunk/KDE/kdeedu/marble/src/lib/TileLevelRangeWidget.cpp PRE-CREATION 
>   /trunk/KDE/kdeedu/marble/src/lib/TileLevelRangeWidget.ui PRE-CREATION 
>   /trunk/KDE/kdeedu/marble/src/marble_part.h 1118235 
>   /trunk/KDE/kdeedu/marble/src/marble_part.cpp 1118235 
>   /trunk/KDE/kdeedu/marble/src/marble_part.rc 1118235 
> 
> Diff: http://reviewboard.kde.org/r/3797/diff
> 
> 
> Testing
> -------
> 
> - several tests with "current region" as selection method => works minus above issues
> - one test with specified lat/lon box => seems to work
> 
> 
> Thanks,
> 
> jmho
> 
>