diff mbox

[KVM-AUTOTEST,5/8] stepeditor.py: get rid of some shortcuts

Message ID 1244433717-3391-9-git-send-email-lmr@redhat.com (mailing list archive)
State New, archived
Headers show

Commit Message

Lucas Meneghel Rodrigues June 8, 2009, 4:01 a.m. UTC 
Disable a few keyboard shortcuts that were initially assumed to be useful,
but apparently override the default functionality of the involved keys
(e.g. 'home', 'end', 'delete') regardless of the widget that has the keyboard
focus.

Also make some indentation changes to the UI description.

Signed-off-by: Michael Goldish <mgoldish@redhat.com>
---
 client/tests/kvm/stepeditor.py |   63 +++++++++++++++++++---------------------
 1 files changed, 30 insertions(+), 33 deletions(-)

Comments

Lucas Meneghel Rodrigues June 8, 2009, 3:19 p.m. UTC | #1 
On Mon, 2009-06-08 at 01:01 -0300, Lucas Meneghel Rodrigues wrote:
> Disable a few keyboard shortcuts that were initially assumed to be useful,
> but apparently override the default functionality of the involved keys
> (e.g. 'home', 'end', 'delete') regardless of the widget that has the keyboard
> focus.
> 
> Also make some indentation changes to the UI description.
> 
> Signed-off-by: Michael Goldish <mgoldish@redhat.com>
> ---
>  client/tests/kvm/stepeditor.py |   63 +++++++++++++++++++---------------------
>  1 files changed, 30 insertions(+), 33 deletions(-)
> 
> diff --git a/client/tests/kvm/stepeditor.py b/client/tests/kvm/stepeditor.py
> index f2ef1aa..9669200 100755
> --- a/client/tests/kvm/stepeditor.py
> +++ b/client/tests/kvm/stepeditor.py
> @@ -858,27 +858,27 @@ class StepMakerWindow:
>  class StepEditor(StepMakerWindow):
>      ui = '''<ui>
>      <menubar name="MenuBar">
> -      <menu action="File">
> -        <menuitem action="Open"/>
> -        <separator/>
> -        <menuitem action="Quit"/>
> -      </menu>
> -      <menu action="Edit">
> -        <menuitem action="CopyStep"/>
> -        <menuitem action="DeleteStep"/>
> -      </menu>
> -      <menu action="Insert">
> -        <menuitem action="InsertNewBefore"/>
> -        <menuitem action="InsertNewAfter"/>
> -        <separator/>
> -        <menuitem action="InsertStepsBefore"/>
> -        <menuitem action="InsertStepsAfter"/>
> -      </menu>
> -      <menu action="Tools">
> -        <menuitem action="CleanUp"/>
> -      </menu>
> +        <menu action="File">
> +            <menuitem action="Open"/>
> +            <separator/>
> +            <menuitem action="Quit"/>
> +        </menu>
> +        <menu action="Edit">
> +            <menuitem action="CopyStep"/>
> +            <menuitem action="DeleteStep"/>
> +        </menu>
> +        <menu action="Insert">
> +            <menuitem action="InsertNewBefore"/>
> +            <menuitem action="InsertNewAfter"/>
> +            <separator/>
> +            <menuitem action="InsertStepsBefore"/>
> +            <menuitem action="InsertStepsAfter"/>
> +        </menu>
> +        <menu action="Tools">
> +            <menuitem action="CleanUp"/>
> +        </menu>
>      </menubar>
> -    </ui>'''
> +</ui>'''
>  
>      # Constructor
>  
> @@ -896,7 +896,7 @@ class StepEditor(StepMakerWindow):
>          self.window.add_accel_group(accelgroup)
>  
>          # Create an ActionGroup
> -        actiongroup = gtk.ActionGroup('UIManagerExample')
> +        actiongroup = gtk.ActionGroup('StepEditor')
>  
>          # Create actions
>          actiongroup.add_actions([
> @@ -904,22 +904,22 @@ class StepEditor(StepMakerWindow):
>               self.quit),
>              ('Open', gtk.STOCK_OPEN, '_Open', None, 'Open steps file',
>               self.open_steps_file),
> -            ('CopyStep', gtk.STOCK_COPY, '_Copy current step...', None,
> +            ('CopyStep', gtk.STOCK_COPY, '_Copy current step...', "",
>               'Copy current step to user specified position', self.copy_step),
> -            ('DeleteStep', gtk.STOCK_DELETE, '_Delete current step', None,
> +            ('DeleteStep', gtk.STOCK_DELETE, '_Delete current step', "",
>               'Delete current step', self.event_remove_clicked),
> -            ('InsertNewBefore', gtk.STOCK_ADD, '_New step before current', None,
> +            ('InsertNewBefore', gtk.STOCK_ADD, '_New step before current', "",
>               'Insert new step before current step', self.insert_before),
> -            ('InsertNewAfter', gtk.STOCK_ADD, 'N_ew step after current', None,
> +            ('InsertNewAfter', gtk.STOCK_ADD, 'N_ew step after current', "",
>               'Insert new step after current step', self.insert_after),
>              ('InsertStepsBefore', gtk.STOCK_ADD, '_Steps before current...',
> -             None, 'Insert steps (from file) before current step',
> +             "", 'Insert steps (from file) before current step',
>               self.insert_steps_before),
> -            ('InsertStepsAfter', gtk.STOCK_ADD, 'Steps _after current...',
> -             None, 'Insert steps (from file) after current step',
> +            ('InsertStepsAfter', gtk.STOCK_ADD, 'Steps _after current...', "",
> +             'Insert steps (from file) after current step',
>               self.insert_steps_after),
> -            ('CleanUp', gtk.STOCK_DELETE, '_Clean up data directory',
> -             None, 'Move unused PPM files to a backup directory', self.cleanup),
> +            ('CleanUp', gtk.STOCK_DELETE, '_Clean up data directory', "",
> +             'Move unused PPM files to a backup directory', self.cleanup),
>              ('File', None, '_File'),
>              ('Edit', None, '_Edit'),
>              ('Insert', None, '_Insert'),
> @@ -939,9 +939,6 @@ class StepEditor(StepMakerWindow):
>  
>          create_shortcut("Next", self.event_next_clicked, "Page_Down")
>          create_shortcut("Previous", self.event_prev_clicked, "Page_Up")
> -        create_shortcut("First", self.event_first_clicked, "Home")
> -        create_shortcut("Last", self.event_last_clicked, "End")
> -        create_shortcut("Delete", self.event_remove_clicked, "Delete")
>  
>          # Add the actiongroup to the uimanager
>          uimanager.insert_action_group(actiongroup, 0)
diff mbox

Patch

diff --git a/client/tests/kvm/stepeditor.py b/client/tests/kvm/stepeditor.py
index f2ef1aa..9669200 100755
--- a/client/tests/kvm/stepeditor.py
+++ b/client/tests/kvm/stepeditor.py
@@ -858,27 +858,27 @@  class StepMakerWindow:
 class StepEditor(StepMakerWindow):
     ui = '''<ui>
     <menubar name="MenuBar">
-      <menu action="File">
-        <menuitem action="Open"/>
-        <separator/>
-        <menuitem action="Quit"/>
-      </menu>
-      <menu action="Edit">
-        <menuitem action="CopyStep"/>
-        <menuitem action="DeleteStep"/>
-      </menu>
-      <menu action="Insert">
-        <menuitem action="InsertNewBefore"/>
-        <menuitem action="InsertNewAfter"/>
-        <separator/>
-        <menuitem action="InsertStepsBefore"/>
-        <menuitem action="InsertStepsAfter"/>
-      </menu>
-      <menu action="Tools">
-        <menuitem action="CleanUp"/>
-      </menu>
+        <menu action="File">
+            <menuitem action="Open"/>
+            <separator/>
+            <menuitem action="Quit"/>
+        </menu>
+        <menu action="Edit">
+            <menuitem action="CopyStep"/>
+            <menuitem action="DeleteStep"/>
+        </menu>
+        <menu action="Insert">
+            <menuitem action="InsertNewBefore"/>
+            <menuitem action="InsertNewAfter"/>
+            <separator/>
+            <menuitem action="InsertStepsBefore"/>
+            <menuitem action="InsertStepsAfter"/>
+        </menu>
+        <menu action="Tools">
+            <menuitem action="CleanUp"/>
+        </menu>
     </menubar>
-    </ui>'''
+</ui>'''
 
     # Constructor
 
@@ -896,7 +896,7 @@  class StepEditor(StepMakerWindow):
         self.window.add_accel_group(accelgroup)
 
         # Create an ActionGroup
-        actiongroup = gtk.ActionGroup('UIManagerExample')
+        actiongroup = gtk.ActionGroup('StepEditor')
 
         # Create actions
         actiongroup.add_actions([
@@ -904,22 +904,22 @@  class StepEditor(StepMakerWindow):
              self.quit),
             ('Open', gtk.STOCK_OPEN, '_Open', None, 'Open steps file',
              self.open_steps_file),
-            ('CopyStep', gtk.STOCK_COPY, '_Copy current step...', None,
+            ('CopyStep', gtk.STOCK_COPY, '_Copy current step...', "",
              'Copy current step to user specified position', self.copy_step),
-            ('DeleteStep', gtk.STOCK_DELETE, '_Delete current step', None,
+            ('DeleteStep', gtk.STOCK_DELETE, '_Delete current step', "",
              'Delete current step', self.event_remove_clicked),
-            ('InsertNewBefore', gtk.STOCK_ADD, '_New step before current', None,
+            ('InsertNewBefore', gtk.STOCK_ADD, '_New step before current', "",
              'Insert new step before current step', self.insert_before),
-            ('InsertNewAfter', gtk.STOCK_ADD, 'N_ew step after current', None,
+            ('InsertNewAfter', gtk.STOCK_ADD, 'N_ew step after current', "",
              'Insert new step after current step', self.insert_after),
             ('InsertStepsBefore', gtk.STOCK_ADD, '_Steps before current...',
-             None, 'Insert steps (from file) before current step',
+             "", 'Insert steps (from file) before current step',
              self.insert_steps_before),
-            ('InsertStepsAfter', gtk.STOCK_ADD, 'Steps _after current...',
-             None, 'Insert steps (from file) after current step',
+            ('InsertStepsAfter', gtk.STOCK_ADD, 'Steps _after current...', "",
+             'Insert steps (from file) after current step',
              self.insert_steps_after),
-            ('CleanUp', gtk.STOCK_DELETE, '_Clean up data directory',
-             None, 'Move unused PPM files to a backup directory', self.cleanup),
+            ('CleanUp', gtk.STOCK_DELETE, '_Clean up data directory', "",
+             'Move unused PPM files to a backup directory', self.cleanup),
             ('File', None, '_File'),
             ('Edit', None, '_Edit'),
             ('Insert', None, '_Insert'),
@@ -939,9 +939,6 @@  class StepEditor(StepMakerWindow):
 
         create_shortcut("Next", self.event_next_clicked, "Page_Down")
         create_shortcut("Previous", self.event_prev_clicked, "Page_Up")
-        create_shortcut("First", self.event_first_clicked, "Home")
-        create_shortcut("Last", self.event_last_clicked, "End")
-        create_shortcut("Delete", self.event_remove_clicked, "Delete")
 
         # Add the actiongroup to the uimanager
         uimanager.insert_action_group(actiongroup, 0)