Subject: bug#23238: closed (Re: bug#23238: [PATCH] Fix alignement rule for CSS)

From: [email-masked] (GNU bug Tracking System)
Date: Sun, 10 Apr 2016 21:36:02 +0000

Your bug report

#23238: [PATCH] Fix alignement rule for CSS

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
23238: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=23238
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: bug#23238: [PATCH] Fix alignement rule for CSS
From: Simen Heggestøyl <[email-masked]>
To: Etienne Prud'homme <[email-masked]>
Cc: [email-masked]
Date: Sun, 10 Apr 2016 23:35:15 +0200
List-Id: <debbugs-submit.debbugs.gnu.org>

On Thu, Apr 7, 2016 at 7:30 PM, Etienne Prud'homme 
<[email-masked]> wrote:
> The attached patch fixes a bug in `align-rules-list` for
> `css-declaration`. The Regex rule didn't recognize properties with 
> more
> than one word like `background-color`.

Thanks, installed in master.

-- Simen

Subject: [PATCH] Fix alignement rule for CSS
From: Etienne Prud'homme <[email-masked]>
To: [email-masked]
Date: Thu, 7 Apr 2016 13:30:31 -0400
List-Id: <debbugs-submit.debbugs.gnu.org>

The attached patch fixes a bug in `align-rules-list` for
`css-declaration`. The Regex rule didn't recognize properties with more
than one word like `background-color`.

I've sent an email to [email-masked] and I still got no response. We
shall consider this work as Copyright paperwork exempt since it's only
one line.

>From 0b50ef758085624a94eead9826eaf42e688e3f76 Mon Sep 17 00:00:00 2001
From: Etienne <etienne@ThinkPax>
Date: Wed, 6 Apr 2016 17:05:41 -0400
Subject: [PATCH] Fix alignement rule for CSS

* lisp/align.el (align-rules-list): Support CSS properties that have
  multiple words.

Copyright-paperwork-exempt: yes
---
 lisp/align.el | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lisp/align.el b/lisp/align.el
index c3389dc..41519a5 100644
--- a/lisp/align.el
+++ b/lisp/align.el
@@ -577,7 +577,7 @@ The possible settings for `align-region-separate' are:
 		     (eq '- current-prefix-arg)))))
 
     (css-declaration
-     (regexp . "^\\s-*\\w+:\\(\\s-*\\).*;")
+     (regexp . "^\\s-*\\(?:\\w-?\\)+:\\(\\s-*\\).*;")
      (group . (1))
      (modes . '(css-mode html-mode))))
   "A list describing all of the available alignment rules.
-- 
2.8.1