<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://svn.reviewboard.kde.org/r/6411/">http://svn.reviewboard.kde.org/r/6411/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://svn.reviewboard.kde.orgrb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for marble.</div>
<div>By Dennis Nienhüser.</div>
<p style="color: grey;"><i>Updated Jan. 24, 2011, 8:15 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Changes to integrate it nicely on the N900.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This patch makes the bookmark manager nearly complete: You can delete folders, rename them, create new ones, delete bookmarks and edit bookmarks (including moving to other folders). It also adds support for importing .kml files. The only missing thing -- which I plan to make a junior job -- is drag & drop support. (Internal move to change the order of folders and bookmarks inside folders. Maybe also D&D between views to support dragging bookmarks quickly to other folders.)
Squashed patch of a local git branch:
- Support importing bookmarks from .kml files. When conflicts arise (imported bookmark already exists), the user is asked whether to skip the new or replace the existing bookmark.
- NewFolderInfoDialog is a misleading name. Rename it NewBookmarkFolderDialog.
- Rename NameDialog to NewBookmarkFolderDialog, the former is too generic. Fix broken license header in NewBookmarkFolderDialog.h. Make it possible to use the dialog without changing existing bookmarks directly. Use a ButtonBox in the .ui to get a consistent interface.
- New bookmark folders can be created within the bookmark manager.
- Deletion of individual folders and bookmarks. Use the first folder as default selection.
- Use the existing dialogs to edit bookmark folders and bookmarks in the bookmark manager.
- Rename BookmarkInfoDialog to EditBookmarkDialog which describes its purpose better.
</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="https://bugs.kde.org/show_bug.cgi?id=261124">261124</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>trunk/KDE/kdeedu/marble/src/lib/BookmarkInfoDialog.ui <span style="color: grey">(1216524)</span></li>
<li>trunk/KDE/kdeedu/marble/src/QtMainWindow.h <span style="color: grey">(1216524)</span></li>
<li>trunk/KDE/kdeedu/marble/src/QtMainWindow.cpp <span style="color: grey">(1216524)</span></li>
<li>trunk/KDE/kdeedu/marble/src/lib/BookmarkInfoDialog.h <span style="color: grey">(1216524)</span></li>
<li>trunk/KDE/kdeedu/marble/src/lib/BookmarkInfoDialog.cpp <span style="color: grey">(1216524)</span></li>
<li>trunk/KDE/kdeedu/marble/src/lib/BookmarkManager.cpp <span style="color: grey">(1216524)</span></li>
<li>trunk/KDE/kdeedu/marble/src/lib/BookmarkManagerDialog.h <span style="color: grey">(1216524)</span></li>
<li>trunk/KDE/kdeedu/marble/src/lib/BookmarkManagerDialog.cpp <span style="color: grey">(1216524)</span></li>
<li>trunk/KDE/kdeedu/marble/src/lib/BookmarkManagerDialog.ui <span style="color: grey">(1216524)</span></li>
<li>trunk/KDE/kdeedu/marble/src/lib/CMakeLists.txt <span style="color: grey">(1216524)</span></li>
<li>trunk/KDE/kdeedu/marble/src/lib/EditBookmarkDialog.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>trunk/KDE/kdeedu/marble/src/lib/EditBookmarkDialog.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>trunk/KDE/kdeedu/marble/src/lib/EditBookmarkDialog.ui <span style="color: grey">(PRE-CREATION)</span></li>
<li>trunk/KDE/kdeedu/marble/src/lib/MarbleWidgetPopupMenu.cpp <span style="color: grey">(1216524)</span></li>
<li>trunk/KDE/kdeedu/marble/src/lib/NameDialog.ui <span style="color: grey">(1216524)</span></li>
<li>trunk/KDE/kdeedu/marble/src/lib/NewBookmarkFolderDialog.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>trunk/KDE/kdeedu/marble/src/lib/NewBookmarkFolderDialog.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>trunk/KDE/kdeedu/marble/src/lib/NewBookmarkFolderDialog.ui <span style="color: grey">(PRE-CREATION)</span></li>
<li>trunk/KDE/kdeedu/marble/src/lib/NewFolderInfoDialog.h <span style="color: grey">(1216524)</span></li>
<li>trunk/KDE/kdeedu/marble/src/lib/NewFolderInfoDialog.cpp <span style="color: grey">(1216524)</span></li>
<li>trunk/KDE/kdeedu/marble/src/marble_part.h <span style="color: grey">(1216524)</span></li>
<li>trunk/KDE/kdeedu/marble/src/marble_part.cpp <span style="color: grey">(1216524)</span></li>
</ul>
<p><a href="http://svn.reviewboard.kde.org/r/6411/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Screenshots </h1>
<div>
<a href="http://svn.reviewboard.kde.org/r/6411/s/607/"><img src="http://svn.reviewboard.kde.org/media/uploaded/images/2011/01/24/marble-maemo-bookmark-manager_400x100.png" style="border: 1px black solid;" alt="Bookmark Manager on the N900" /></a>
</div>
</td>
</tr>
</table>
</div>
</body>
</html>