From: Bart Schaefer <schaefer@brasslantern.com>
To: Marlon Richert <marlon.richert@gmail.com>
Cc: Zsh hackers list <zsh-workers@zsh.org>
Subject: Re: Rewrite of zsh-newuser-install
Date: Tue, 9 Feb 2021 18:30:17 -0800 [thread overview]
Message-ID: <CAH+w=7bXpF=FGys0aEZ9Kz3Cj5FWFCY0WfE=4yWsHLzK+RF=Sw@mail.gmail.com> (raw)
In-Reply-To: <CAHLkEDsq0MxTMKVuc++VE81SCX8QLvya0kL5hyvN9jRCCXWFmw@mail.gmail.com>
On Tue, Feb 9, 2021 at 1:42 PM Marlon Richert <marlon.richert@gmail.com> wrote:
>
> https://gitlab.com/marlonrichert/zsh-sensible/-/commit/3d867257b7dcdb7826bc07f44756ba30bf539fda?w=1
> # %F{x}: set bacKground color to x; %k: default bacKground color
Typo, should be %K{x}
> # $(( [#10] x )): cast x to int; COLUMNS: current line width of terminal
Is there any advantage to computing this with PROMPT_SUBST versus, for
example, reassigning PS1 with a new width in precmd? Anyway if you're
already allowing the dynamic parts to consume 40% of the terminal
width I'm not sure it's worth the effort of making them dependent on
that. Just pick a width for %n@%M and a number of trailing components
for %~.
> add-zle-hook-widget line-finish :zshrc:timer
See previous comments in other messages, but if keeping this I'd put
it in preexec instead.
> PS4=$': -> %(?,%F{g},%F{b}%K{r})%?%f%k\t%e: %F{g}%1N %f%2x:%I ;
> %(1_,%S%_%s ,)%f%k'
The :...; trick doesn't work if you have an ">" in PS4, it becomes a
redirection upon copy-paste. I don't think you need the semicolon if
you have a newline.
> # m:{[:lower:]-}={[:upper:]_} does zsh-name -> ZSH_NAME
Doesn't work for completing parameter names, e.g. $xdg-TAB just beeps.
Is that an issue?
> typeset -T LS_COLORS ls_colors
Per previous discussion, I'd protect this with a test that LS_COLORS
is set. Unless you've setopt WARN_CREATE_GLOBAL, the later reference
to "$ls_colors[@]" will be harmless.
Speaking of that, heh, there should be a warning comment (?) or
something that all of this expects to load in zsh native mode,
otherwise those naked subscripts etc. will break.
> () {
> local zcachedir=${ZDOTDIR:-${XDG_CACHE_HOME:-$HOME/.cache}/zsh}
I wonder if it would be less potentially puzzling to use "function {"
here instead of empty parens?
> setopt AUTO_PUSHD # Go back to previous dirs with `cd (+|-)[<num>]` or `~(+|-)[<num>]`.
That description doesn't seem quite right to me. The behavior of cd
doesn't depend on this option; it depends on using the directory
stack, but you can do that without AUTO_PUSHD.
I think we've commented regarding everything else. Thanks again.
next prev parent reply other threads:[~2021-02-10 2:30 UTC|newest]
Thread overview: 117+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-02-06 20:03 Marlon Richert
[not found] ` <0102017778f35f33-a962e4d3-83e9-4d3b-a0d7-45701bb40b11-000000@eu-west-1.amazonses.com>
2021-02-06 20:19 ` Marlon Richert
2021-02-06 20:33 ` Lawrence Velázquez
2021-02-06 21:54 ` Bart Schaefer
2021-02-07 13:41 ` Marlon Richert
2021-02-07 13:51 ` Roman Perepelitsa
2021-02-07 17:10 ` Marlon Richert
2021-02-07 17:28 ` Marlon Richert
2021-02-07 20:20 ` Bart Schaefer
2021-02-07 21:06 ` dana
2021-02-07 21:15 ` Marlon Richert
2021-02-08 21:57 ` Marlon Richert
2021-02-08 23:33 ` Lawrence Velázquez
2021-02-09 1:42 ` Bart Schaefer
2021-02-09 2:00 ` Bart Schaefer
2021-02-09 8:18 ` Marlon Richert
2021-02-09 23:09 ` Bart Schaefer
2021-02-09 8:17 ` Marlon Richert
2021-02-09 8:29 ` Roman Perepelitsa
2021-02-09 23:16 ` Bart Schaefer
2021-02-12 0:09 ` Bart Schaefer
2021-02-12 5:58 ` Marlon Richert
2021-02-09 4:51 ` dana
2021-02-09 6:00 ` Bart Schaefer
2021-02-09 7:30 ` dana
2021-02-09 7:34 ` dana
2021-02-09 9:55 ` Marlon Richert
2021-02-09 10:01 ` Roman Perepelitsa
2021-02-09 10:04 ` Marlon Richert
2021-02-09 10:56 ` dana
2021-02-09 11:14 ` Roman Perepelitsa
2021-02-09 11:39 ` Marlon Richert
2021-02-09 17:21 ` dana
2021-02-09 21:01 ` Marlon Richert
2021-02-09 21:41 ` Marlon Richert
2021-02-09 23:15 ` dana
2021-02-10 0:02 ` Bart Schaefer
2021-02-10 7:02 ` Marlon Richert
2021-02-10 6:57 ` Marlon Richert
2021-02-12 0:10 ` Bart Schaefer
2021-02-12 5:59 ` Roman Perepelitsa
2021-02-13 0:23 ` dana
2021-02-10 2:30 ` Bart Schaefer [this message]
2021-02-10 7:44 ` Marlon Richert
2021-02-10 20:27 ` Marlon Richert
2021-02-11 8:30 ` Bart Schaefer
2021-02-11 21:11 ` Marlon Richert
2021-02-11 22:57 ` Bart Schaefer
2021-02-12 5:49 ` Marlon Richert
2021-02-12 5:47 ` Marlon Richert
2021-02-12 23:43 ` Oliver Kiddle
2021-02-13 1:11 ` Bart Schaefer
2021-02-12 23:26 ` Oliver Kiddle
2021-02-13 0:15 ` Marlon Richert
2021-02-13 1:33 ` Bart Schaefer
2021-02-13 1:36 ` Oliver Kiddle
2021-02-13 2:53 ` Bart Schaefer
2021-02-13 10:26 ` Marlon Richert
2021-02-13 22:53 ` Marlon Richert
2021-02-14 0:34 ` Bart Schaefer
2021-02-14 8:12 ` Marlon Richert
2021-02-13 22:56 ` Bart Schaefer
2021-02-14 8:01 ` Marlon Richert
2021-02-19 21:38 ` Marlon Richert
2021-02-20 0:30 ` dana
2021-02-20 8:18 ` Marlon Richert
2021-02-20 18:57 ` Bart Schaefer
2021-02-21 19:24 ` Marlon Richert
2021-02-24 22:15 ` dana
2021-02-25 8:05 ` Daniel Shahaf
2021-02-25 16:58 ` dana
2021-02-26 22:31 ` Marlon Richert
2021-02-27 13:21 ` Daniel Shahaf
2021-02-27 13:46 ` Daniel Shahaf
2021-03-19 22:12 ` Marlon Richert
2021-03-24 13:45 ` gi1242+zsh
2021-03-24 14:16 ` Paul
2021-03-24 17:44 ` Bart Schaefer
2021-03-24 20:38 ` Marlon Richert
2021-03-25 3:36 ` Paul
2021-04-05 18:16 ` Daniel Shahaf
2021-04-05 18:52 ` Sorting of <-> (was Re: Rewrite of zsh-newuser-install) Bart Schaefer
2021-04-05 21:31 ` Rewrite of zsh-newuser-install gammafunction
2021-04-07 14:45 ` Marlon
2021-04-09 16:49 ` Marlon
2021-04-09 23:14 ` Daniel Shahaf
2021-04-10 1:17 ` Oliver Kiddle
2021-04-07 18:17 ` Mikael Magnusson
2021-04-07 18:56 ` Daniel Shahaf
2021-03-24 10:00 ` Marlon Richert
2021-03-25 1:15 ` Daniel Shahaf
2021-04-05 14:00 ` Marlon Richert
2021-04-05 18:36 ` Daniel Shahaf
2021-04-05 19:22 ` Daniel Shahaf
2021-02-22 3:54 ` Paul
2021-02-22 8:14 ` Marlon Richert
2021-02-22 16:31 ` Bug in compdescribe with matcher 'b:-=+' Bart Schaefer
2021-06-14 8:28 ` Marlon Richert
2021-08-12 12:03 ` Marlon Richert
2021-08-12 16:15 ` Bart Schaefer
2021-02-19 21:34 ` Rewrite of zsh-newuser-install Marlon Richert
2021-02-13 1:28 ` Bart Schaefer
2021-02-13 1:34 ` Bart Schaefer
2021-04-22 13:57 ` Marlon Richert
2021-02-09 23:05 ` Bart Schaefer
2021-02-09 9:44 ` Marlon Richert
2021-02-09 18:13 ` Greg Klanderman
2021-02-08 9:21 ` Peter Stephenson
2021-02-08 6:35 ` Daniel Shahaf
2021-02-08 8:44 ` Marlon Richert
2021-02-08 8:46 ` Marlon Richert
2021-02-08 10:32 ` Daniel Shahaf
2021-02-08 17:44 ` Marlon Richert
2021-02-08 20:47 ` Bart Schaefer
2021-02-09 21:44 ` Eric Cook
2021-02-09 22:34 ` Bart Schaefer
2021-02-07 20:22 ` Bart Schaefer
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='CAH+w=7bXpF=FGys0aEZ9Kz3Cj5FWFCY0WfE=4yWsHLzK+RF=Sw@mail.gmail.com' \
--to=schaefer@brasslantern.com \
--cc=marlon.richert@gmail.com \
--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).