Colours leaking from RPROMPT

Colours leaking from RPROMPT

[David Banks]

Hi list,

I adapted a right-hand-side prompt for showing the run time of the
last command.  Code looks like this:

    function preexec() {
        timer=${timer:-$(date "+%s.%N")}
    }

    function precmd() {
        if [ -n "$timer" ]; then
            new_timer=$(date "+%s.%N")
            timer_accurate=$(printf "scale=2; %s - %s\n" "$new_timer"
"$timer" | bc)
            timer_show=$(printf "%.2f" "$timer_accurate")
            RPROMPT="%F{cyan}${timer_show}s %{$reset_color%}"
            unset timer
        fi
    }


I see some strange colour behaviour when using completions, though.
For instance, when I complete something with more than one possible
match, the completion menu is all coloured cyan.  And once I accept
one of the options from the completion menu, the unique substring that
was completed remains coloured cyan in the command string.  There are
also a couple of other odd behaviours that are not so easy to trigger.

The value of PS1 is '%n@%{%}%m%{%} %(!.#.$) '.

What would be the cause of this?  As far as I can see, the colour
should always be reset after the prompt string is printed, so it
should never show outside the prompt.

Thanks!
Dave

Re: Colours leaking from RPROMPT

[Bart Schaefer]

On Mar 19, 10:50am, David Banks wrote:
}
}             RPROMPT="%F{cyan}${timer_show}s %{$reset_color%}"

Are you sure $reset_color is defined?  You need to autoload and run
the "colors" function to initialize $reset_color.

However, you shouldn't need $reset_color at all ... you should use:

    RPROMPT="%F{cyan}${timer_show}s %f"


You might also want to look at the zsh/datetime module and $EPOCHREALTIME
instead of doing all that "date" and "bc" stuff.

Re: Colours leaking from RPROMPT

[David Banks]

On 19 March 2014 22:34, Bart Schaefer <schaefer@brasslantern.com> wrote:
> On Mar 19, 10:50am, David Banks wrote:
> }
> }             RPROMPT="%F{cyan}${timer_show}s %{$reset_color%}"
>
> Are you sure $reset_color is defined?  You need to autoload and run
> the "colors" function to initialize $reset_color.

printf "%s" "$reset_color" | wc -c
=> 5

> However, you shouldn't need $reset_color at all ... you should use:
>
>     RPROMPT="%F{cyan}${timer_show}s %f"

That fixed it, thanks!

> You might also want to look at the zsh/datetime module and $EPOCHREALTIME
> instead of doing all that "date" and "bc" stuff.

Great, I will check these out.

Cheers!
Dave