unofficial mirror of help-gnu-emacs@gnu.org
 help / color / mirror / Atom feed
* Colourising a label prints it twice
@ 2024-07-12 23:09 Heime
  2024-07-12 23:26 ` Stephen Berman
  0 siblings, 1 reply; 4+ messages in thread
From: Heime @ 2024-07-12 23:09 UTC (permalink / raw)
  To: Heime via Users list for the GNU Emacs text editor

I have the following function to colourise the label, but I get OFF printed twice. 

(defun label-select (label)
  "Color the part of LABEL after [-]."

  (if (string-match "\\[\\-\\] \\(.*\\)" label)

      (let ( (bt (match-string 0 label))
             (lb (match-string 1 label)) )
        (concat bt (propertize lb 'face '(:foreground "red"))))

    label))

    (insert (format " %s \n" (label-select "[-] OFF ")))
    (insert (format " %s \n" (label-select "[-]  OFF ")))
    (insert (format " %s \n" (label-select " [-]   OFF ")))



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Colourising a label prints it twice
  2024-07-12 23:09 Colourising a label prints it twice Heime
@ 2024-07-12 23:26 ` Stephen Berman
  2024-07-12 23:55   ` Heime
  0 siblings, 1 reply; 4+ messages in thread
From: Stephen Berman @ 2024-07-12 23:26 UTC (permalink / raw)
  To: Heime; +Cc: Heime via Users list for the GNU Emacs text editor

On Fri, 12 Jul 2024 23:09:28 +0000 Heime <heimeborgia@protonmail.com> wrote:

> I have the following function to colourise the label, but I get OFF printed twice.
>
> (defun label-select (label)
>   "Color the part of LABEL after [-]."
>
>   (if (string-match "\\[\\-\\] \\(.*\\)" label)
>
>       (let ( (bt (match-string 0 label))
>              (lb (match-string 1 label)) )
>         (concat bt (propertize lb 'face '(:foreground "red"))))
>
>     label))
>
>     (insert (format " %s \n" (label-select "[-] OFF ")))
>     (insert (format " %s \n" (label-select "[-]  OFF ")))
>     (insert (format " %s \n" (label-select " [-]   OFF ")))

`bt' is the entire string passed by LABEL, but IIUC it should just be
the string "[-] ", so put that in a group:

(defun label-select (label)
  "Color the part of LABEL after [-]."

  (if (string-match "\\(\\[\\-\\] \\)\\(.*\\)" label)

      (let ( (bt (match-string 1 label))
             (lb (match-string 2 label)) )
        (concat bt (propertize lb 'face '(:foreground "red"))))

    label))

Steve Berman



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Colourising a label prints it twice
  2024-07-12 23:26 ` Stephen Berman
@ 2024-07-12 23:55   ` Heime
  2024-07-13  9:05     ` Stephen Berman
  0 siblings, 1 reply; 4+ messages in thread
From: Heime @ 2024-07-12 23:55 UTC (permalink / raw)
  To: Stephen Berman; +Cc: Heime via Users list for the GNU Emacs text editor

On Saturday, July 13th, 2024 at 11:26 AM, Stephen Berman <stephen.berman@gmx.net> wrote:

> On Fri, 12 Jul 2024 23:09:28 +0000 Heime heimeborgia@protonmail.com wrote:
> 
> > I have the following function to colourise the label, but I get OFF printed twice.
> > 
> > (defun label-select (label)
> > "Color the part of LABEL after [-]."
> > 
> > (if (string-match "\\[\\-\\] \\(.*\\)" label)
> > 
> > (let ( (bt (match-string 0 label))
> > (lb (match-string 1 label)) )
> > (concat bt (propertize lb 'face '(:foreground "red"))))
> > 
> > label))
> > 
> > (insert (format " %s \n" (label-select "[-] OFF ")))
> > (insert (format " %s \n" (label-select "[-] OFF ")))
> > (insert (format " %s \n" (label-select " [-] OFF ")))
> 
> 
> `bt' is the entire string passed by LABEL, but IIUC it should just be
> the string "[-] ", so put that in a group:
> 
> (defun label-select (label)
> "Color the part of LABEL after [-]."
> 
> (if (string-match "\\(\\[\\-\\] \\)\\(.*\\)" label)
> 
> (let ( (bt (match-string 1 label))
> (lb (match-string 2 label)) )
> (concat bt (propertize lb 'face '(:foreground "red"))))
> 
> label))
> 
> Steve Berman

Regarding grouping, I want to replicate the original string with respect 
to any whitespaces.  Does the grouping outline respect the original whitespace ? 




^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Colourising a label prints it twice
  2024-07-12 23:55   ` Heime
@ 2024-07-13  9:05     ` Stephen Berman
  0 siblings, 0 replies; 4+ messages in thread
From: Stephen Berman @ 2024-07-13  9:05 UTC (permalink / raw)
  To: Heime; +Cc: Heime via Users list for the GNU Emacs text editor

On Fri, 12 Jul 2024 23:55:49 +0000 Heime <heimeborgia@protonmail.com> wrote:

> On Saturday, July 13th, 2024 at 11:26 AM, Stephen Berman
> <stephen.berman@gmx.net> wrote:
>
>> On Fri, 12 Jul 2024 23:09:28 +0000 Heime heimeborgia@protonmail.com wrote:
>>
>> > I have the following function to colourise the label, but I get OFF
>> > printed twice.
>> >
>> > (defun label-select (label)
>> > "Color the part of LABEL after [-]."
>> >
>> > (if (string-match "\\[\\-\\] \\(.*\\)" label)
>> >
>> > (let ( (bt (match-string 0 label))
>> > (lb (match-string 1 label)) )
>> > (concat bt (propertize lb 'face '(:foreground "red"))))
>> >
>> > label))
>> >
>> > (insert (format " %s \n" (label-select "[-] OFF ")))
>> > (insert (format " %s \n" (label-select "[-] OFF ")))
>> > (insert (format " %s \n" (label-select " [-] OFF ")))
>>
>>
>> `bt' is the entire string passed by LABEL, but IIUC it should just be
>> the string "[-] ", so put that in a group:
>>
>> (defun label-select (label)
>> "Color the part of LABEL after [-]."
>>
>> (if (string-match "\\(\\[\\-\\] \\)\\(.*\\)" label)
>>
>> (let ( (bt (match-string 1 label))
>> (lb (match-string 2 label)) )
>> (concat bt (propertize lb 'face '(:foreground "red"))))
>>
>> label))
>>
>> Steve Berman
>
> Regarding grouping, I want to replicate the original string with respect
> to any whitespaces.  Does the grouping outline respect the original whitespace ?

The regexp I used is fairly specific to the examples you gave (though I
overlooked that your third example has a space before "[-]"); to allow
arbitrary whitespace in the first group before and after "[-]" you can
use "\\s-*":

(defun label-select (label)
  "Color the part of LABEL after [-]."

  (if (string-match "\\(\\s-*\\[\\-\\]\\s-*\\)\\(.*\\)" label)

      (let ( (bt (match-string 1 label))
             (lb (match-string 2 label)) )
        (concat bt (propertize lb 'face '(:foreground "red"))))

    label))

Steve Berman



^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2024-07-13  9:05 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-07-12 23:09 Colourising a label prints it twice Heime
2024-07-12 23:26 ` Stephen Berman
2024-07-12 23:55   ` Heime
2024-07-13  9:05     ` Stephen Berman

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 read-only IMAP folder(s) and NNTP newsgroup(s).