zsh-workers
 help / color / mirror / code / Atom feed
From: Peter Stephenson <pws@csr.com>
To: zsh-workers@sunsite.dk (Zsh hackers list)
Subject: Re: PATCH: random attribute stuff
Date: Tue, 06 May 2008 16:10:07 +0100	[thread overview]
Message-ID: <200805061510.m46FA71C001035@news01.csr.com> (raw)
In-Reply-To: <080506075838.ZM1161@torch.brasslantern.com>

Bart Schaefer wrote:
> On May 6,  3:42pm, Peter Stephenson wrote:
> }
> } Bart Schaefer wrote:
> } > Some people want the entire command line to look one way and the output
> } > from the command to look another way, so they can easily tell what's a
> } > command and what isn't when scrolling backward through buffered output.
> } 
> } Right, but I'm not convinced it actually works.
> 
> It almost certainly *won't* work except in the most simple cases now that
> ZLE is doing its own highlighting effects.  People are going to have to
> decide whether they want that badly enough to use zle_highlight=(none).

I tried it without any highlighting.

> Inevitably someone will complain that ZLE should put back whatever effect
> was there before when it turns on/off its own effects.

That's probably doable in principle by tying the highlighting state to
what came in with the prompt, but (i) I don't have a lot of interest in
doing this since it's quite a complicated way of propagating a hack to
get particular effects that's getting hackier as the highlighting code
develops (ii) as I said, it doesn't seem to work very well anyway.

It ought to be (but might turn out not to be) fairly easy to add a
"default" element to zle_highlight that causes the given attributes to
be used as the default set for the command line, which would probably
work a lot better, and then deprecate any reliance on what the prompt
happened to produce.

-- 
Peter Stephenson <pws@csr.com>                  Software Engineer
CSR PLC, Churchill House, Cambridge Business Park, Cowley Road
Cambridge, CB4 0WZ, UK                          Tel: +44 (0)1223 692070


  reply	other threads:[~2008-05-06 15:11 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-06  9:15 Peter Stephenson
2008-05-06 14:34 ` Bart Schaefer
2008-05-06 14:42   ` Peter Stephenson
2008-05-06 14:58     ` Bart Schaefer
2008-05-06 15:10       ` Peter Stephenson [this message]
2008-05-06 15:38         ` Bart Schaefer
2008-05-06 15:48           ` Peter Stephenson
2008-05-06 18:16             ` Peter Stephenson
2008-05-07 15:44               ` Oliver Kiddle
2008-05-07 15:49               ` Oliver Kiddle
2008-05-07 15:55                 ` Peter Stephenson

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=200805061510.m46FA71C001035@news01.csr.com \
    --to=pws@csr.com \
    --cc=zsh-workers@sunsite.dk \
    /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).