emacs-bug-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#71724: closed (30.0.50; [PATCH] php-ts-mode: Colorize CSS property v


From: GNU bug Tracking System
Subject: bug#71724: closed (30.0.50; [PATCH] php-ts-mode: Colorize CSS property value like `css--fontify-region')
Date: Sun, 23 Jun 2024 05:07:01 +0000

Your message dated Sun, 23 Jun 2024 08:06:10 +0300
with message-id <86iky0fdtp.fsf@gnu.org>
and subject line Re: bug#71724: 30.0.50; [PATCH] php-ts-mode: Colorize CSS 
property value like `css--fontify-region'
has caused the debbugs.gnu.org bug report #71724,
regarding 30.0.50; [PATCH] php-ts-mode: Colorize CSS property value like 
`css--fontify-region'
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
71724: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=71724
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems
--- Begin Message --- Subject: 30.0.50; [PATCH] php-ts-mode: Colorize CSS property value like `css--fontify-region' Date: Sat, 22 Jun 2024 23:36:13 +0200
Hi, 
this patch adds a new feature to colorize a property when it's value is a text 
that represents a CSS color. It will be fontified such that its background is 
the color itself, like in 'cs-ts-mode'.
A new custom var 'php-ts-mode-css-fontify-colors' can be used to toggle this 
behavior.

Thanks.
Vincenzo

Attachment: 0001-Colorize-CSS-property-value-like-css-fontify-region.patch
Description: Text Data


--- End Message ---
--- Begin Message --- Subject: Re: bug#71724: 30.0.50; [PATCH] php-ts-mode: Colorize CSS property value like `css--fontify-region' Date: Sun, 23 Jun 2024 08:06:10 +0300
> From: Vincenzo Pupillo <v.pupillo@gmail.com>
> Date: Sat, 22 Jun 2024 23:36:13 +0200
> 
> this patch adds a new feature to colorize a property when it's value is a 
> text 
> that represents a CSS color. It will be fontified such that its background is 
> the color itself, like in 'cs-ts-mode'.
> A new custom var 'php-ts-mode-css-fontify-colors' can be used to toggle this 
> behavior.

Thanks, installed, and closing the bug.

Please in the future observe our conventions for the commit log
messages:

> Subject: [PATCH] Colorize CSS property value like `css--fontify-region'
> 
> If the value of a property is text representing a CSS color, it will be
> fontified such that its background is the color itself.
> 'php-ts-mode-css-fontify-colors' can be used to disable this behaviour.

These lines are too long.  They will be indented by a TAB in the
generated ChangeLog file, so they should preferably be no longer than
65 characters.

> * lisp/progmodes/php-ts-mode.el (php-ts-mode-css-fontify-colors):
>   New custom var.
 ^^^
Please don't leave this whitespace at the beginning of lines.

> * lisp/progmodes/php-ts-mode.el (php-ts-mode): Use the new function.

This line is also too long.

I usually correct any log messages that need it, but if you observe
these conventions better, it will make my job easier.

Thanks.


--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]