Bug 228504 - 'Apply' button not needed in the 'Edit Reminders' dialogue
Summary: 'Apply' button not needed in the 'Edit Reminders' dialogue
Status: VERIFIED FIXED
Alias: None
Product: korganizer
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-02-25 19:01 UTC by Sabine Faure
Modified: 2010-02-26 21:50 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:

Attachments
Add an attachment

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