[Marble-devel] Review Request: Showing/editing the coords in bookmark editor

Friedrich W. H. Kossebau kossebau at kde.org
Wed Sep 14 22:17:09 UTC 2011



> On Sept. 14, 2011, 10:10 p.m., Thibaut Gridel wrote:
> > Please close the review if it got pushed.
> > Even better doing it with REVIEW: ### in the commit message.

Sure, was going to do in the end :)

Was just unsure if it still is possible to comment on it if it is closed, so kept it open for a while.
And this review was actually about a few commits, so unsure if it makes sense to just mark the last one with the REVIEW keyword.

Will test now if commenting still would be possible :)


- Friedrich W. H.


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


On Sept. 13, 2011, 11:09 p.m., Friedrich W. H. Kossebau wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102608/
> -----------------------------------------------------------
> 
> (Updated Sept. 13, 2011, 11:09 p.m.)
> 
> 
> Review request for Marble.
> 
> 
> Summary
> -------
> 
> This patchs adds the possibility to see and also edit also the coords of a bookmark in the bookmark editor.
> 
> Initially used a lineeddit and GeoDataCoordinates::toString, but GeoDataCoordinates::fromString cannot yet handle coords with secs it seems.
> So turned to use a LatLonEdit per coord.
> 
> Found LatLonEdit to be quite incomplete, so rewrote it to get it working for me.
> E.g. LatLonEdit::setDimension(...) was pretty broken.
> Also needed/wanted a double value for the secs editor, to enable more precision.
> No idea if 2 digits is a sane number of digits after the ".".
> 
> If LatLonEdit has to stay ABI compatible I will have to redo it then, as I changed the API
> (removed unused/useless dimensionChanged signal and changed some method signatures).
> 
> 
> Diffs
> -----
> 
>   src/lib/BookmarkManagerDialog.cpp 3f9db62 
>   src/lib/EditBookmarkDialog.h 4157726 
>   src/lib/EditBookmarkDialog.cpp 7e36ddc 
>   src/lib/EditBookmarkDialog.ui 9baf50d 
>   src/lib/LatLonEdit.h 4958805 
>   src/lib/LatLonEdit.cpp 758a167 
>   src/lib/LatLonEdit.ui 26774d3 
> 
> Diff: http://git.reviewboard.kde.org/r/102608/diff
> 
> 
> Testing
> -------
> 
> Added and viewed/edited several coords of geo bookmarks, worked all fine, same values shown after reloading or same places focussed if selected.
> 
> Also tested overflow behaviour of LatLonEdit for quite some border cases.
> 
> LatLonEdit widget still works fine in worldclock plasmoid.
> 
> 
> Thanks,
> 
> Friedrich W. H.
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20110914/eb7ef46b/attachment.html>


More information about the Marble-devel mailing list