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

List:       kde-core-devel
Subject:    patch: khelpcenter close/quit
From:       Thomas Zander <zander () planescape ! com>
Date:       2001-07-25 8:48:58
[Download RAW message or body]


It my ethernal fight to let all applications use the styleguide I want to 
ask if anyone has a problem with the following patch:

It basically lets the khelpcenter use the quit action instead of the close
action, this is needed because of the section "quit" and the section "close"
in the following page:
 http://developer.kde.org/documentation/standards/kde/style/basics/quitclose.html


Index: khc_main.cc
===================================================================
RCS file: /home/kde/kdebase/khelpcenter/khc_main.cc,v
retrieving revision 1.42
diff -u -3 -p -r1.42 khc_main.cc
--- khc_main.cc 2001/07/23 14:52:32     1.42
+++ khc_main.cc 2001/07/25 08:41:37
@@ -92,7 +92,7 @@ KHMainWindow::KHMainWindow(const KURL &u
     setGeometry(366, 0, 800, 600);
 
     (*actionCollection()) += *doc->actionCollection();
-    (void)KStdAction::close(this, SLOT(close()), actionCollection());
+    (void)KStdAction::quit(this, SLOT(close()), actionCollection());
 
     back = new KToolBarPopupAction( i18n( "&Back" ), "back", ALT+Key_Left,
                                     this, SLOT( slotBack() ),
Index: khelpcenterui.rc
===================================================================
RCS file: /home/kde/kdebase/khelpcenter/khelpcenterui.rc,v
retrieving revision 1.5
diff -u -3 -p -r1.5 khelpcenterui.rc
--- khelpcenterui.rc    2001/07/23 14:52:32     1.5
+++ khelpcenterui.rc    2001/07/25 08:41:37
@@ -4,7 +4,7 @@
  <Menu name="file" noMerge="1">
     <text>&amp;File</text>
     <Action name="printFrame" group="print" />
-    <Action name="file_close"/>
+    <Action name="file_quit"/>
  </Menu>
  <Menu name="edit" noMerge="1">
   <text>&amp;Edit</text>

-- 
Thomas Zander                                            zander@earthling.net
The only thing worse than failure is the fear of trying something new

[Attachment #3 (application/pgp-signature)]

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

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