Bug 259655 - 'Remove all bookmarks' does not ask for a confirmation
Summary: 'Remove all bookmarks' does not ask for a confirmation
Status: RESOLVED FIXED
Alias: None
Product: marble
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: 1.0 (KDE 4.6)
Assignee: marble-bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-12-12 22:14 UTC by Michael G. Hansen
Modified: 2010-12-15 23:21 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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