From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mailout02.t-online.de (mailout02.t-online.de [194.25.134.17]) by sourceware.org (Postfix) with ESMTPS id 541833858C52 for ; Thu, 2 Feb 2023 17:48:09 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 541833858C52 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=t-online.de Authentication-Results: sourceware.org; spf=none smtp.mailfrom=t-online.de Received: from fwd72.dcpf.telekom.de (fwd72.aul.t-online.de [10.223.144.98]) by mailout02.t-online.de (Postfix) with SMTP id 2530541F9; Thu, 2 Feb 2023 18:48:08 +0100 (CET) Received: from [192.168.2.101] ([87.187.38.45]) by fwd72.t-online.de with (TLSv1.3:TLS_AES_256_GCM_SHA384 encrypted) esmtp id 1pNdgp-3ee9GS0; Thu, 2 Feb 2023 18:48:03 +0100 Subject: Re: [PATCH setup] Add tooltip for Ctrl+I/R/U accelerator keys To: Jon Turney , "cygwin-apps@cygwin.com" References: <3f2e6d0f-100d-bf00-4f7f-839bc64b26fc@t-online.de> From: Christian Franke Message-ID: <6a681c15-4ad8-6e61-f676-a5d4c6509a98@t-online.de> Date: Thu, 2 Feb 2023 18:48:03 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:68.0) Gecko/20100101 SeaMonkey/2.53.14 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-TOI-EXPURGATEID: 150726::1675360083-80AEDD99-C7ED363B/0/0 CLEAN NORMAL X-TOI-MSGID: d130c3b2-baad-42ba-9d99-006d6d38f9fb X-Spam-Status: No, score=2.1 required=5.0 tests=BAYES_00,FREEMAIL_FROM,KAM_DMARC_STATUS,KAM_LAZY_DOMAIN_SECURITY,NICE_REPLY_A,RCVD_IN_BARRACUDACENTRAL,RCVD_IN_DNSWL_LOW,RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,SPF_NONE,TXREP autolearn=no autolearn_force=no version=3.4.6 X-Spam-Level: ** X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: Jon Turney wrote: > On 22/11/2022 16:00, Christian Franke wrote: >> This is a first try to make these keys more obvious as requested on >> the Cygwin ML. >> >> A more complex approach would be to mention only the keys which >> actually would change the current package state. > > Thanks for this patch. > > So, I appreciate what this is trying to do, but I have great > difficultly convincing myself that this is the right approach. > I'm also not sure. An alternative would possibly be some help below the listbox, centered in the same line as "Hide obsolete packages". > We don't need to resort to a tooltip for any other shortcut, because > there is an established visual design language to indicate those.  I > guess we just need to find what's consistent with pre-existing ways of > indicating this kind of behaviour in other applications. > Cygwin setup is somewhat uncommon as there is no default menu bar with drop-down menus. Accelerator keys are traditionally mentioned there.