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

[Jens-Michael Hoffmann]

> On 2010-04-24 07:59:56, Dennis Nienh=C3=BCser wrote:
> > /trunk/KDE/kdeedu/marble/src/lib/DownloadRegionDialog.cpp, line 118
> > <http://reviewboard.kde.org/r/3797/diff/1/?file=3D24424#file24424line11=
8>
> >
> >     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.

Fixed, looks much better now.


> On 2010-04-24 07:59:56, Dennis Nienh=C3=BCser wrote:
> > /trunk/KDE/kdeedu/marble/src/lib/DownloadRegionDialog.cpp, line 91
> > <http://reviewboard.kde.org/r/3797/diff/1/?file=3D24424#file24424line91>
> >
> >     "Visible region"? It may not be clear where "Current" refers to.

Fixed


- jmho


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


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 reg=
ion dialog
> 2) UI hangs for some time while the download queue is being filled
> =

> =

> Diffs
> -----
> =

>   /trunk/KDE/kdeedu/marble/src/lib/AbstractScanlineTextureMapper.h 111823=
5 =

>   /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 =3D> works minu=
s above issues
> - one test with specified lat/lon box =3D> seems to work
> =

> =

> Thanks,
> =

> jmho
> =

>

-------------------------------------------------------------