Bug 300188

Summary:	Editing a bookmark changes its order in the tree
Product:	[Applications] marble	Reporter:	Dennis Nienhüser <nienhueser>
Component:	general	Assignee:	marble-bugs
Status:	RESOLVED FIXED
Severity:	minor
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:	http://commits.kde.org/marble/f5029d2d85c7ae146f3d042af2aa05c3c3027dd7	Version Fixed/Implemented In:
Sentry Crash Report:

Description Dennis Nienhüser 2012-05-17 09:58:46 UTC

In master (to be KDE/4.9) when editing a bookmark it is always removed from the tree and re-added. Therefore it appears at the end of a folder even when only changing its name.
In KDE/4.8 this is implemented in a nicer way such that the original bookmark is adjusted.


Reproducible: Always

Steps to Reproduce:
1. Note the order of a bookmark in the tree (folder)
2. Edit the bookmark (change the name)
3. Note the order of the same bookmark (now at the end of the containing folder)



Expected Results:  
Bookmarks should only change their position in the tree when needed (i.e. when the parent container is changed)

Comment 1 Thibaut Gridel 2012-06-10 09:05:35 UTC

Git commit f5029d2d85c7ae146f3d042af2aa05c3c3027dd7 by Thibaut Gridel.
Committed on 09/06/2012 at 18:47.
Pushed by tgridel into branch 'master'.

BookmarkManager: don't remove and add a bookmark that got edited

M  +5    -0    src/lib/BookmarkManager.cpp
M  +2    -0    src/lib/BookmarkManager.h
M  +13   -13   src/lib/BookmarkManagerDialog.cpp

http://commits.kde.org/marble/f5029d2d85c7ae146f3d042af2aa05c3c3027dd7