From: Oliver Kiddle <okiddle@yahoo.co.uk>
To: zsh-workers <zsh-workers@zsh.org>
Subject: Re: Regression: broken completion on modification time
Date: Sat, 18 Jun 2016 01:50:21 +0200 [thread overview]
Message-ID: <20306.1466207421@thecus.kiddle.eu> (raw)
In-Reply-To: <160617111101.ZM488@torch.brasslantern.com>
Bart wrote:
> } How would you want to present the exactly match given that it inserts
> } nothing? In the past with the match -- description form, we didn't
> } include it. With the description only form it is less out-of-place.
>
> Isn't there a way to stick a dummy match into the listing just to take
> up space so that only the description will appear?
Without the verbose style, descriptions are
»before«, »exactly« and »since«
You can select »exactly« but it inserts nothing.
With verbose (and the recent patch), they are:
»+ -- before« and »- -- since« exactly is omitted
So if exactly is to be included there as requested by Daniel, it would
need to be something like »'' -- exactly«, » -- exactly« or just
»exactly«.
> I think that in any case where the calendar won't be displayed [almost]
> immediately, an entirely different set of matches should be passed to
> compadd, or at least they should be added in a different order. The
Okay, lets try the following. Looking at _main_complete, checking
$MENUSELECT is perhaps better than $_def_menu_style.
If someone has a setup where they defer menu selection and are bothered
that they now miss out on the calendar, we can add a style or something.
Or is there some way to prevent the completion system from saving the
old_list?
Matches can't be added in a different order while preserving the layout.
Without the layout, the numbers alone are useless regardless of their
order.
Oliver
diff --git a/Completion/Unix/Type/_dates b/Completion/Unix/Type/_dates
index 83397b4..0b9cead 100644
--- a/Completion/Unix/Type/_dates
+++ b/Completion/Unix/Type/_dates
@@ -37,6 +37,7 @@ _tags dates || return 0
_comp_mesg=yes
_description -2V -x dates expl date
compadd "${@:/-X/-x}" "$expl[@]" -
+[[ -z $MENUSELECT && $WIDGET != menu-select ]] && return
[[ -n $PREFIX$SUFFIX ]] && return 0
(( rows )) || return 0
compstate[list]='packed rows'
next prev parent reply other threads:[~2016-06-17 23:56 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-06-12 20:32 Dima Kogan
2016-06-13 0:47 ` Bart Schaefer
2016-06-13 1:37 ` Dima Kogan
2016-06-13 22:00 ` Oliver Kiddle
2016-06-14 16:14 ` Bart Schaefer
2016-06-15 16:32 ` Oliver Kiddle
2016-06-15 23:24 ` Daniel Shahaf
2016-06-17 16:19 ` Oliver Kiddle
2016-06-17 18:11 ` Bart Schaefer
2016-06-17 19:10 ` Nikolay Aleksandrovich Pavlov (ZyX)
2016-06-17 22:02 ` Oliver Kiddle
2016-06-17 22:20 ` Nikolay Aleksandrovich Pavlov (ZyX)
2016-06-17 22:33 ` Bart Schaefer
2016-06-17 22:49 ` Nikolay Aleksandrovich Pavlov (ZyX)
2016-06-17 23:00 ` Bart Schaefer
2016-06-17 23:50 ` Oliver Kiddle [this message]
2016-06-19 16:04 ` Bart Schaefer
2016-06-21 1:41 ` Daniel Shahaf
2016-06-21 14:31 ` Oliver Kiddle
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20306.1466207421@thecus.kiddle.eu \
--to=okiddle@yahoo.co.uk \
--cc=zsh-workers@zsh.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.vuxu.org/mirror/zsh/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).