From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!.POSTED.blaine.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.devel Subject: Re: BIKESHED: completion faces Date: Fri, 08 Nov 2019 17:33:55 +0200 Message-ID: <83pni2bcws.fsf@gnu.org> References: <87zhhaxalt.fsf@gmail.com> <83bltpgffr.fsf@gnu.org> <83tv7gg9oz.fsf@gnu.org> <83r22kg8pa.fsf@gnu.org> <20191106205133.njij3ve7qqy7yh3q@Ergus> <83ftizg4nr.fsf@gnu.org> <8336ezg2vm.fsf@gnu.org> <83wocbelu1.fsf@gnu.org> <83imnvegcp.fsf@gnu.org> <83ftizeelw.fsf@gnu.org> <83bltne6oq.fsf@gnu.org> <87lfsrqs63.fsf@gmail.com> <838soqeuzr.fsf@gnu.org> <87ftiyn07t.fsf@gmail.com> <8336eycvpf.fsf@gnu.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Injection-Info: blaine.gmane.org; posting-host="blaine.gmane.org:195.159.176.226"; logging-data="139224"; mail-complaints-to="usenet@blaine.gmane.org" Cc: spacibba@aol.com, dgutov@yandex.ru, monnier@iro.umontreal.ca, emacs-devel@gnu.org To: =?utf-8?B?Sm/Do28gVMOhdm9yYQ==?= Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Fri Nov 08 16:38:51 2019 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([209.51.188.17]) by blaine.gmane.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.89) (envelope-from ) id 1iT6Lb-000a28-45 for ged-emacs-devel@m.gmane.org; Fri, 08 Nov 2019 16:38:51 +0100 Original-Received: from localhost ([::1]:56400 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iT6La-0001an-0B for ged-emacs-devel@m.gmane.org; Fri, 08 Nov 2019 10:38:50 -0500 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:52026) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iT6H3-0004qG-Vk for emacs-devel@gnu.org; Fri, 08 Nov 2019 10:34:11 -0500 Original-Received: from fencepost.gnu.org ([2001:470:142:3::e]:49453) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1iT6H2-0004D3-U0; Fri, 08 Nov 2019 10:34:08 -0500 Original-Received: from [176.228.60.248] (port=2179 helo=home-c4e4a596f7) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1iT6H1-0008Tr-Cj; Fri, 08 Nov 2019 10:34:08 -0500 In-reply-to: (message from =?utf-8?B?Sm/Do28gVMOhdm9yYQ==?= on Fri, 8 Nov 2019 15:09:31 +0000) X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Original-Sender: "Emacs-devel" Xref: news.gmane.org gmane.emacs.devel:241985 Archived-At: > From: João Távora > Date: Fri, 8 Nov 2019 15:09:31 +0000 > Cc: Ergus , emacs-devel , > Stefan Monnier , Dmitry Gutov > > > > >> fabrobazor > > > >> ^ ^ ^ > > > >> `---`---`----- I want these bold by default in the future > > > >> > > > >> no change to the 'basic/prefix' completion. > > > > > > > > I understand that you want to highlight both f, o, o, and r, but the > > > > latter with a different face. If my understanding is incorrect, > > > > > > It is incorrect indeed: 'r' is should have the same face as 'a' or 'b'. > > > > Did I say that _only_ r will be highlighted with that face? > > I don' think so. I think you can read what you said above. > > I confirmed your sentence "if my understanding is incorrect". Because > I _don't_ intend flex to "highlight" r at all, or 'a' or any other character > except 'f', 'o' and ' o'. So these characters ('a','b','r' and 'z' in this > example) should, for now, have the same face, 'default'. We can think > about the value of highlighting other neighboring characters later. > > > I'm okay with highlighting a and b as well in this example, provided > > that typing "faoo" will include "fabrobazor" in the results. > > It will indeed include that in the results. But in that your > example only 4 characters should be highlighted: f, > a (the first one),o, and o . So no 'b'. 'b' comes from what you originally wrote. I admit I don't understand why 'b' won't be highlighted: wouldn't typing "fobo" narrow the list of candidates shown by "foo"? If so, 'b' should be highlighted like 'a' and 'r', no? > > That would mean this face is used inconsistently: in basic and other > > similar styles it highlights the character(s) that narrow(s) the > > selection of candidates, while in flex they will highlight the > > characters already typed by the user. Is that correct? > > Yes it is, but "this face" would now be called "completion-emphasis" > , not "completion-narrower" or "completion-already-typed" or > "completion-first-difference". That's why the renames are > important. I don't think that names that obfuscate the use of the face is a good idea. I understood the reason for renaming to be that "first difference" and "common part" don't necessarily make sense for flex and other similar styles, but the solution should be to make the names more general, not obfuscated. "First" and "second" or "primary" and "secondary" are too opaque to be useful, IMO. > > If so, I object to this inconsistency. > > OK. It's good to finally clear that up. But when I talked about this topic > ealier in the thread you replied this. > > https://lists.gnu.org/archive/html/emacs-devel/2019-11/msg00176.html Yes, I've changed my mind after understanding better what you propose.