[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    kdemultimedia/kmix
From:       Aaron J. Seigo <aseigo () kde ! org>
Date:       2003-12-12 0:03:44
[Download RAW message or body]

CVS commit by aseigo: 

Usability: remove the superfluous save/restore actions, as discussed on
kde-core-devel. seeing as i was really the only one playing devils advocate
and Frans' audio engineer experience trumps my, uh, lack of audio engineer
experience, i'm committing this change as an improvement to usability in
the spirit of KISS

original patch by Frans Englich <frans.englich@telia.com>

CCMAIL: Frans Englich <frans.englich@telia.com>


  M +2 -5      kmix.cpp   1.168
  M +0 -3      kmixui.rc   1.11


--- kdemultimedia/kmix/kmix.cpp  #1.167:1.168
@@ -107,8 +107,4 @@ KMixWindow::initActions()
 {
         // file menu
-        (void)new KAction( i18n("&Restore Default Volumes"), 0, this, SLOT(loadVolumes()),
-                                                         actionCollection(), "file_load_volume" );
-        (void)new KAction( i18n("&Save Current Volumes as Default"), 0, this, SLOT(saveVolumes()),
-                                                         actionCollection(), "file_save_volume" );
         KStdAction::quit( this, SLOT(quit()), actionCollection());
 
@@ -498,4 +494,5 @@ void
 KMixWindow::quit()
 {
+        saveVolumes();
         kapp->quit();
 }

--- kdemultimedia/kmix/kmixui.rc  #1.10:1.11
@@ -6,7 +6,4 @@
   <Action name="file_new_tab"/>
   <Action name="file_close_tab"/>
-  <Separator/>
-  <Action name="file_load_volume"/>
-  <Action name="file_save_volume"/>
  </Menu>
  <Menu name="help" append="about_merge"><text>&amp;Help</text>


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic