Bug 228504

Summary:	'Apply' button not needed in the 'Edit Reminders' dialogue
Product:	[Applications] korganizer	Reporter:	Sabine Faure <sabine>
Component:	general	Assignee:	kdepim bugs <kdepim-bugs>
Status:	VERIFIED FIXED
Severity:	normal	CC:	montel
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed In:

Description Sabine Faure 2010-02-25 19:01:03 UTC

Version:            (using Devel)
Compiler:          gcc 4.3.2 
OS:                Linux
Installed from:    Compiled sources

- Launch KOrg
- Click on the 'New Event' button
- Enter title, location, date and time for the event
- Click on the 'Advanced...' button
- set up a reminder
- click on 'Apply' and you can see that in the 'New Event' dialogue, next to the reminder icon the following message has been added: '1 advanced reminder configured'.

However you cannot type any more information (ex description, add attachement...) into the 'New Event' dialogue until you click on the 'Ok' button to close the 'Edit Reminders' dialogue.

So I am wondering how useful it is here to have an 'Apply' button. I would just remove it from the 'Edit Reminders' dialogue.

Trunk, Svn Rev 1095977

Comment 1 Laurent Montel 2010-02-26 09:52:05 UTC

SVN commit 1096205 by mlaurent:

fix bug #228504 it's a modal dialogbox
=> apply is not necessary here
CCBUG: 228504


 M  +1 -1      koeditoralarms.cpp  


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

Comment 2 Laurent Montel 2010-02-26 09:52:29 UTC

Fixed in svn #1096205.

Comment 3 Laurent Montel 2010-02-26 09:56:11 UTC

SVN commit 1096207 by mlaurent:

Backport fix bug #228504
MERGE: e4/e5/ 3.5 ?
CCBUG: 228504


 M  +1 -1      koeditoralarms.cpp  


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

Comment 4 Sabine Faure 2010-02-26 21:50:36 UTC

It is corrected now.

The 'Apply' button has been removed.

Trunk, Svn Rev 1096391