[Marble-devel] Review Request: Bookmark Feature : Icons, Coordinate in bookmarks.kml and Edit Bookmark

[1989.gaurav at gmail.com]

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

(Updated 2010-08-15 16:54:29.279911)


Review request for marble.


Changes
-------

Removed all comment which was not related to this patch
Removed all changes of GeoDataTreeModel


Summary
-------

Added Icons in Bookmark menu.
Added GeoDataCoordinate in bookmark.kml file along with lookAt.
Added ExtendedData Tag writer to write in bookmark.kml file.
Adding Edit Bookmark Feature along with some modification in GeoDataTreeModel ( In Process )


Diffs (updated)
-----

  /trunk/KDE/kdeedu/marble/src/QtMainWindow.cpp 1163977 
  /trunk/KDE/kdeedu/marble/src/lib/BookmarkInfoDialog.cpp 1163977 
  /trunk/KDE/kdeedu/marble/src/lib/BookmarkManager.h 1163977 
  /trunk/KDE/kdeedu/marble/src/lib/BookmarkManager.cpp 1163977 
  /trunk/KDE/kdeedu/marble/src/lib/GeoDataTreeModel.cpp 1163977 
  /trunk/KDE/kdeedu/marble/src/lib/MarbleWidget.h 1163977 
  /trunk/KDE/kdeedu/marble/src/lib/MarbleWidget.cpp 1163977 
  /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataExtendedData.h 1163977 
  /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataExtendedData.cpp 1163977 
  /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataPlacemark.h 1163977 
  /trunk/KDE/kdeedu/marble/src/lib/geodata/data/GeoDataPlacemark.cpp 1163977 
  /trunk/KDE/kdeedu/marble/src/lib/geodata/writers/kml/KmlDataTagWriter.h PRE-CREATION 
  /trunk/KDE/kdeedu/marble/src/lib/geodata/writers/kml/KmlDataTagWriter.cpp PRE-CREATION 
  /trunk/KDE/kdeedu/marble/src/lib/geodata/writers/kml/KmlExtendedDataTagWriter.h PRE-CREATION 
  /trunk/KDE/kdeedu/marble/src/lib/geodata/writers/kml/KmlExtendedDataTagWriter.cpp PRE-CREATION 
  /trunk/KDE/kdeedu/marble/src/lib/geodata/writers/kml/KmlPlacemarkTagWriter.cpp 1163977 
  /trunk/KDE/kdeedu/marble/src/marble.qrc 1163977 

Diff: http://reviewboard.kde.org/r/5034/diff


Testing
-------

Running


Thanks,

1989gaurav

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/marble-devel/attachments/20100815/203bf4b2/attachment.htm