[PATCH setup] Keyboard accelerator for keep or skip

[PATCH setup] Keyboard accelerator for keep or skip

[Christian Franke]

[-- Attachment #1: Type: text/plain, Size: 146 bytes --]

Use case: Easily prevent update of multiple packages in the "Pending" 
view, in particular useful if "Test" is selected.

-- 
Regards,
Christian


[-- Attachment #2: 0001-Keyboard-accelerator-for-keep-or-skip.patch --]
[-- Type: text/plain, Size: 1521 bytes --]

From 05d9c72f07cf754dc6172a998b2e0991034d363f Mon Sep 17 00:00:00 2001
From: Christian Franke <christian.franke@t-online.de>
Date: Tue, 25 Apr 2023 18:42:18 +0200
Subject: [PATCH] Keyboard accelerator for keep or skip

Ctrl+K selects keep or skip and then moves selection to next row.
---
 PickPackageLine.cc | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/PickPackageLine.cc b/PickPackageLine.cc
index c1e2a15..d380965 100644
--- a/PickPackageLine.cc
+++ b/PickPackageLine.cc
@@ -155,6 +155,7 @@ PickPackageLine::map_key_to_action(WORD vkey, int modkeys, int & col_num,
       col_num = new_col;
       return Action::PopUp;
     case 'I': // Ctrl+I: select install default version and move to next row
+    case 'K': // Ctrl+K: select keep or skip package and move to next row
     case 'R': // Ctrl+R: select reinstall and move to next row
     case 'U': // Ctrl+U: select uninstall and move to next row
       if (modkeys != ModifierKeys::Control)
@@ -163,8 +164,9 @@ PickPackageLine::map_key_to_action(WORD vkey, int modkeys, int & col_num,
       switch (vkey)
         {
         case 'I': action_id = packagemeta::Install_action; break;
+        default:  action_id = packagemeta::NoChange_action; break;
         case 'R': action_id = packagemeta::Reinstall_action; break;
-        default:  action_id = packagemeta::Uninstall_action; break;
+        case 'U': action_id = packagemeta::Uninstall_action; break;
         }
       return Action::Direct | Action::NextRow;
     }
-- 
2.39.0

Re: [PATCH setup] Keyboard accelerator for keep or skip

[Jon Turney]

On 25/04/2023 18:00, Christian Franke via Cygwin-apps wrote:
> Use case: Easily prevent update of multiple packages in the "Pending" 
> view, in particular useful if "Test" is selected.

Sorry for the delay in looking at this.  Obviously needed.

Please apply.

Re: [PATCH setup] Keyboard accelerator for keep or skip

[Christian Franke]

Jon Turney wrote:
> On 25/04/2023 18:00, Christian Franke via Cygwin-apps wrote:
>> Use case: Easily prevent update of multiple packages in the "Pending" 
>> view, in particular useful if "Test" is selected.
>
> Sorry for the delay in looking at this.  Obviously needed.

N.P.


> Please apply.

Done.