From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!.POSTED.blaine.gmane.org!not-for-mail From: Mauro Aranda Newsgroups: gmane.emacs.bugs Subject: bug#11046: 24.0.94; Customize: M-TAB moves cursor over tag; clicking button moves cursor over button Date: Mon, 30 Sep 2019 13:08:50 -0300 Message-ID: References: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="00000000000053488b0593c77488" Injection-Info: blaine.gmane.org; posting-host="blaine.gmane.org:195.159.176.226"; logging-data="55133"; mail-complaints-to="usenet@blaine.gmane.org" To: 11046@debbugs.gnu.org Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Mon Sep 30 18:10:27 2019 Return-path: Envelope-to: geb-bug-gnu-emacs@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 1iEyFn-000EEc-6t for geb-bug-gnu-emacs@m.gmane.org; Mon, 30 Sep 2019 18:10:27 +0200 Original-Received: from localhost ([::1]:54584 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iEyFl-0003To-SK for geb-bug-gnu-emacs@m.gmane.org; Mon, 30 Sep 2019 12:10:25 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:39409) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iEyFS-0003Rn-QH for bug-gnu-emacs@gnu.org; Mon, 30 Sep 2019 12:10:08 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1iEyFQ-0007uu-2Q for bug-gnu-emacs@gnu.org; Mon, 30 Sep 2019 12:10:05 -0400 Original-Received: from debbugs.gnu.org ([209.51.188.43]:51505) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1iEyFP-0007uk-GY for bug-gnu-emacs@gnu.org; Mon, 30 Sep 2019 12:10:03 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1iEyFO-0007Ep-9N for bug-gnu-emacs@gnu.org; Mon, 30 Sep 2019 12:10:03 -0400 X-Loop: help-debbugs@gnu.org In-Reply-To: Resent-From: Mauro Aranda Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Mon, 30 Sep 2019 16:10:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 11046 X-GNU-PR-Package: emacs Original-Received: via spool by 11046-submit@debbugs.gnu.org id=B11046.156985975227714 (code B ref 11046); Mon, 30 Sep 2019 16:10:02 +0000 Original-Received: (at 11046) by debbugs.gnu.org; 30 Sep 2019 16:09:12 +0000 Original-Received: from localhost ([127.0.0.1]:60325 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1iEyEZ-0007Ct-Oy for submit@debbugs.gnu.org; Mon, 30 Sep 2019 12:09:12 -0400 Original-Received: from mail-lf1-f41.google.com ([209.85.167.41]:43452) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1iEyEX-0007CV-6j for 11046@debbugs.gnu.org; Mon, 30 Sep 2019 12:09:09 -0400 Original-Received: by mail-lf1-f41.google.com with SMTP id u3so7476588lfl.10 for <11046@debbugs.gnu.org>; Mon, 30 Sep 2019 09:09:09 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:from:date:message-id:subject:to:cc; bh=IUo2s2ZCG8cxeDjGu2L75Lpk+kmS4ieV3ZA898c1jF4=; b=JDBWVs6VHsuClzjsv2CDkh2zhLVyKil3FaAybycY6vkDu1/B4BQp6Js44v4Xloc/iR BF2XHEctfDC+fuC4qNeB+dbfY7AFg8XTGzoG+uwXn+17V3Ni6EQdRJP6E7VJGkxmtdxC 4Lzs3epCvvFdOTSzOLI6/Q8UaJjesreEdSoRxklEW5G5mYgLcPgovu6Sn8u0rykYXN0g aDtvnWuJ6SmZOw2x6zHwD5IyoLIYGRGhvFpnvdSaMk0Lqv5D9bpWWEUsBQSg6/6ExPND oJNzj6/YKWkF7O+U33DwcZNzU5JVvAuGdhCx/K/LXvkGdLQJcpgCNzLwIPCzbizpdWP4 7pLA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:from:date:message-id:subject:to:cc; bh=IUo2s2ZCG8cxeDjGu2L75Lpk+kmS4ieV3ZA898c1jF4=; b=o4KR1+en09UFPxwXsXtPF2Is5uzFfbMEO8OrwlST/z/9GLHj9dL+HyCKssMjYKpn0T RCjikGeEMOtF+FcaMYLpsx9yGRx0PBFPTSXA5PogrDJk+pPvGKB+F+teOgTveG/uC9Yf kb/rPgryR763/575ISGgaWyK+XgvtYdD9+IFL9ioJ3CjaJDWPgWGFV4z1HEWYoXkK/XQ LJYZruXBh3Ou4oRFWaZhmp2IVqne33TkUNP+LwCPgJXiG6RPOjNXb4AT4uhd24SaJbXn 9kBuOX37HPse8fWvL5HPHVpPeqj1P6mDIEZCksccGf4hrISgexb1+YsDsn+T/XFM58Im 0i/Q== X-Gm-Message-State: APjAAAUWvix8mjd3JFKe4x/1Om1185rDg43iohleC2EfykeoJnTSVfdw mpbUa9AiFreUDrJbjlW3J+uPBe3BZKCS7V9TfzDt72rwzAg= X-Google-Smtp-Source: APXvYqy5gFu80nz7oRqWnT0aDYf4Rqx5YJX7Pjdd1qt0Bw1s0yed4S+C92v6UONphQL4uoa862jjP7jawtDKJAPKY0w= X-Received: by 2002:ac2:4552:: with SMTP id j18mr3377643lfm.120.1569859742937; Mon, 30 Sep 2019 09:09:02 -0700 (PDT) X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.51.188.43 X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Original-Sender: "bug-gnu-emacs" Xref: news.gmane.org gmane.emacs.bugs:167922 Archived-At: --00000000000053488b0593c77488 Content-Type: multipart/alternative; boundary="0000000000005348880593c77486" --0000000000005348880593c77486 Content-Type: text/plain; charset="UTF-8" Eli Zaretskii writes: >> From: "Drew Adams" >> Date: Tue, 20 Mar 2012 09:02:30 -0700 >> >> emacs -Q >> >> M-x customize-option RET site-run-file RET >> >> Click Value Menu button. Choose String in the menu. >> >> 1. First bug: Cursor should be in the edit field. But it is on top of >> the Value Menu button. > > Confirmed. This is because widget-button-click-moves-point is set to t for customize buffers, and also because there is no mechanism implemented for widgets to "pass focus" to others. If it is a desired feature, I can try to come up with something. >> 2. Put cursor at beginning of the edit field. Hit `M-TAB'. >> >> You see this error message: "lookup-words: lookup-words error: No plain >> word-list found at systemdefault locations. Customize >> `ispell-alternate-dictionary' to set yours." >> >> But the cursor is moved back on top of the tag `String:', before the >> `:' (second bug). > > I don't get the error message, but a "*Choices*" buffer (because my > ispell-alternate-dictionary _is_ properly configured), but that buffer > shows hits for "String". Here's a sample: > > (0) /Usr/Dict/Web2:190067:String (1) /Usr/Dict/Web2:190068:Stringboard > (2) /Usr/Dict/Web2:190069:Stringcourse (3) /Usr/Dict/Web2:190070:Stringend > > etc. -- quite un-useful. IOW, it _really_ tries to complete the word > "String", which is not what the user wants. This is because the completion function used for string widgets is ispell-complete-word and that command will look back from point until it finds a word to complete. I can't think of another way of fixing this other than peeking what will ispell-complete-word try to complete, so I attach a patch that does that. > (Shouldn't the type of the widget be "file" instead of "string"?) I think so. --0000000000005348880593c77486 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Eli Zaretskii <eliz@gnu= .org> writes:

>> From: "Drew Adams" <drew.adams@oracle.com>
>&g= t; Date: Tue, 20 Mar 2012 09:02:30 -0700
>>
>> emacs -Q<= br>>>
>> M-x customize-option RET site-run-file RET
>= > =C2=A0
>> Click Value Menu button.=C2=A0 Choose String in the= menu.
>> =C2=A0
>> 1. First bug: Cursor should be in the= edit field.=C2=A0 But it is on top of
>> the Value Menu button.>
> Confirmed.

This is because widget-button-click-moves-= point is set to t for
customize buffers, and also because there is no me= chanism implemented
for widgets to "pass focus" to others.=C2= =A0 If it is a desired feature, I
can try to come up with something.
=
>> 2. Put cursor at beginning of the edit field.=C2=A0 Hit `M-TAB= '.
>> =C2=A0
>> You see this error message: "loo= kup-words: lookup-words error: No plain
>> word-list found at syst= emdefault locations.=C2=A0 Customize
>> `ispell-alternate-dictiona= ry' to set yours."
>> =C2=A0
>> But the cursor i= s moved back on top of the tag `String:', before the
>> `:'= ; (second bug).
>
> I don't get the error message, but a &q= uot;*Choices*" buffer (because my
> ispell-alternate-dictionary = _is_ properly configured), but that buffer
> shows hits for "Str= ing".=C2=A0 Here's a sample:
>
> (0) /Usr/Dict/Web2:19= 0067:String =C2=A0 =C2=A0 =C2=A0 (1) /Usr/Dict/Web2:190068:Stringboard
&= gt; (2) /Usr/Dict/Web2:190069:Stringcourse (3) /Usr/Dict/Web2:190070:String= end
>
> etc. -- quite un-useful.=C2=A0 IOW, it _really_ tries t= o complete the word
> "String", which is not what the user = wants.

This is because the completion function used for string widge= ts is
ispell-complete-word and that command will look back from point un= til it
finds a word to complete.
I can't think of another way of = fixing this other than peeking what will
ispell-complete-word try to com= plete, so I attach a patch that does
that.

> (Shouldn't th= e type of the widget be "file" instead of "string"?)
I think so.
--0000000000005348880593c77486-- --00000000000053488b0593c77488 Content-Type: text/x-patch; charset="US-ASCII"; name="0001-Only-complete-words-inside-of-the-string-widget.patch" Content-Disposition: attachment; filename="0001-Only-complete-words-inside-of-the-string-widget.patch" Content-Transfer-Encoding: base64 Content-ID: X-Attachment-Id: f_k16lwyub0 RnJvbSBjZTkwZjY0NDJjYTQwZjAzMGQzZGU0NDZjZDlmNTdjNTM4MDRkN2ZlIE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBNYXVybyBBcmFuZGEgPG1hdXJvb2FyYW5kYUBnbWFpbC5jb20+ CkRhdGU6IFdlZCwgMjUgU2VwIDIwMTkgMTk6NDA6MzUgLTAzMDAKU3ViamVjdDogW1BBVENIXSBP bmx5IGNvbXBsZXRlIHdvcmRzIGluc2lkZSBvZiB0aGUgc3RyaW5nIHdpZGdldAoKKiBsaXNwL3dp ZC1lZGl0LmVsICgnc3RyaW5nIHdpZGdldCk6IFBlZWsgdGhlIHdvcmQgdGhhdAppc3BlbGwtY29t cGxldGUtd29yZCB3aWxsIHRyeSB0byBjb21wbGV0ZSwgYW5kIG9ubHkgb2ZmZXIgY29tcGxldGlv bnMKd2hlbiB0aGUgd29yZCBpcyBpbnNpZGUgb2YgdGhlIGZpZWxkLiAgKEJ1ZyMxMTA0NikKLS0t CiBsaXNwL3dpZC1lZGl0LmVsIHwgNyArKysrKystCiAxIGZpbGUgY2hhbmdlZCwgNiBpbnNlcnRp b25zKCspLCAxIGRlbGV0aW9uKC0pCgpkaWZmIC0tZ2l0IGEvbGlzcC93aWQtZWRpdC5lbCBiL2xp c3Avd2lkLWVkaXQuZWwKaW5kZXggOTE2ZDQxYS4uOWE2MzRiNyAxMDA2NDQKLS0tIGEvbGlzcC93 aWQtZWRpdC5lbAorKysgYi9saXNwL3dpZC1lZGl0LmVsCkBAIC0zMDczLDcgKzMwNzMsMTIgQEAg J3N0cmluZwogICAiQSBzdHJpbmcuIgogICA6dGFnICJTdHJpbmciCiAgIDpmb3JtYXQgIiV7JXQl fTogJXYiCi0gIDpjb21wbGV0ZS1mdW5jdGlvbiAnaXNwZWxsLWNvbXBsZXRlLXdvcmQKKyAgOmNv bXBsZXRlIChsYW1iZGEgKHdpZGdldCkKKyAgICAgICAgICAgICAgKGV2YWwtYW5kLWNvbXBpbGUg KHJlcXVpcmUgJ2lzcGVsbCkpIDsgRm9yIGBpc3BlbGwtZ2V0LXdvcmQnLgorICAgICAgICAgICAg ICAobGV0ICgoc3RhcnQgKHNhdmUtZXhjdXJzaW9uIChudGggMSAoaXNwZWxsLWdldC13b3JkIG5p bCkpKSkpCisgICAgICAgICAgICAgICAgKGlmICg8IHN0YXJ0ICh3aWRnZXQtZmllbGQtc3RhcnQg d2lkZ2V0KSkKKyAgICAgICAgICAgICAgICAgICAgKG1lc3NhZ2UgIk5vIHdvcmQgdG8gY29tcGxl dGUgaW5zaWRlIGZpZWxkIikKKyAgICAgICAgICAgICAgICAgIChpc3BlbGwtY29tcGxldGUtd29y ZCkpKSkKICAgOnByb21wdC1oaXN0b3J5ICd3aWRnZXQtc3RyaW5nLXByb21wdC12YWx1ZS1oaXN0 b3J5KQogCiAoZGVmaW5lLXdpZGdldCAncmVnZXhwICdzdHJpbmcKLS0gCjIuNy40Cgo= --00000000000053488b0593c77488--