> On May 25, 2015, 11:49 a.m., Dennis Nienhüser wrote: > > src/lib/marble/geodata/data/GeoDataCoordinates.cpp, line 27 > > <https://git.reviewboard.kde.org/r/123579/diff/4/?file=368082#file368082line27> > > > > still needed?
Yes, the implementation of all the methods declared at GeoDataCoordinatesPrivate.h were in GeoDataCoordinates.cpp, so I included UTM methods definitions there too. - Alejandro ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123579/#review80806 ----------------------------------------------------------- On May 11, 2015, 12:10 a.m., Alejandro García Montoro wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/123579/ > ----------------------------------------------------------- > > (Updated May 11, 2015, 12:10 a.m.) > > > Review request for Marble. > > > Repository: marble > > > Description > ------- > > Two different tasks have been accomplished concerning UTM coordinate system: > > 1. The UTM grid rendering has been fixed. Now the grid and labels are drawn > as expected (see > http://upload.wikimedia.org/wikipedia/commons/e/ed/Utm-zones.jpg for > comparison) > 2. Some UTM functions have been added in order to properly calculate and > display the coordinates on the status bar. Those functions have been wrapped > in the new GeoDataUTM namespace inside GeoDataCoordinates.cpp If you prefer > to see the functions integrated in the GeoDataCoordinates class, please, let > me know. > > **Still to be fixed**: The UTM string showed in the status bar is too long > for the text field in which it is shown. I have not been able to change its > size, as I have not found the relevant .ui file; where in the code should I > check to change that? > > **Note**: The lat/lon <-> UTM conversion algorithm has been adapted from this > JavaScript code > http://home.hiwaay.net/~taylorc/toolbox/geography/geoutm.html. The author > says in the page "The JavaScript source code in this document may be copied > and reused without restriction", and I have linked the website in the code. > However, as I am not sure about all the licenses and authoring stuff, I would > like you to review it. > > > Diffs > ----- > > src/apps/marble-kde/marble_part.cpp > 283ad87a39f91e01c391a4dd8f4c9ddbeef41481 > src/apps/marble-qt/QtMainWindow.cpp > c4280c62b4534b7d10387b6895d142920f957bb7 > src/lib/marble/geodata/data/GeoDataCoordinates.h > 6efabd047f8f3aff63381facb78cb371c0740bf4 > src/lib/marble/geodata/data/GeoDataCoordinates.cpp > fe74a2e42db921b152a29ef04246ceaaa16c2325 > src/lib/marble/geodata/data/GeoDataCoordinates_p.h > 7b84e860c11937602b66faef2cee6b9cc403a132 > src/plugins/render/graticule/GraticulePlugin.cpp > a5e6d14a921c66d55cf01e39b16564dedcf6532b > tests/TestGeoDataCoordinates.cpp a38b090104503f021fb98eed5f3f4394c10d5f8f > > Diff: https://git.reviewboard.kde.org/r/123579/diff/ > > > Testing > ------- > > Changes tested in master branch @ 1st May. > Grid and coordinates work well (except for the displaying issue explained > above) and are consistent with the UTM coordinate system. > > > File Attachments > ---------------- > > UTM grid exceptions around Norway > > https://git.reviewboard.kde.org/media/uploaded/files/2015/05/01/505df756-87f2-47bb-a6c1-d5b52629547a__NorwayScreenshot.png > > > Thanks, > > Alejandro García Montoro > >
_______________________________________________ Marble-devel mailing list Marble-devel@kde.org https://mail.kde.org/mailman/listinfo/marble-devel