Bug 259655

Summary:	'Remove all bookmarks' does not ask for a confirmation
Product:	[Applications] marble	Reporter:	Michael G. Hansen <mike>
Component:	general	Assignee:	marble-bugs
Status:	RESOLVED FIXED
Severity:	normal	CC:	nienhueser
Priority:	NOR
Version:	unspecified
Target Milestone:	1.0 (KDE 4.6)
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed In:

Description Michael G. Hansen 2010-12-12 22:14:34 UTC

Version:           unspecified (using Devel) 
OS:                Linux

IMHO the 'Remove all bookmarks' menu action should ask for a confirmation before erasing all bookmarks. It is very prominently placed, and can in my opinion be pressed accidentally, causing a loss of all bookmarks.

Michael

Reproducible: Didn't try




OS: Linux (x86_64) release 2.6.32-trunk-amd64
Compiler: gcc-amd64

Comment 1 Dennis Nienhüser 2010-12-12 23:44:00 UTC

Indeed, good point. I hoped to have a bookmark manager ready for 4.6, but we didn't find time/people to work on it.

It's easy to fix, but will add a new string, so I'll have to ask translators first for permission.

Comment 2 Dennis Nienhüser 2010-12-15 23:21:55 UTC

SVN commit 1206794 by nienhueser:

Ask for confirmation before removing all bookmarks.
Contains one new string (same string in marble_qt.po and marble.po), acked in http://lists.kde.org/?t=129236639300002&r=1&w=2, thanks.
BUG: 259655
RB: 6122
CCMAIL: kde-i18n-doc@kde.org

 M  +5 -0      QtMainWindow.cpp  
 M  +4 -0      marble_part.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1206794