bug#23238: [PATCH] Fix alignement rule for CSS

bug#23238: [PATCH] Fix alignement rule for CSS

[Etienne Prud'homme]

[-- Attachment #1: Type: text/plain, Size: 317 bytes --]

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 asign@gnu.org and I still got no response. We
shall consider this work as Copyright paperwork exempt since it's only
one line.

[-- Attachment #2: 0001-Fix-alignement-rule-for-CSS.patch --]
[-- Type: text/x-patch, Size: 863 bytes --]

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

bug#23238: [PATCH] Fix alignement rule for CSS

[Simen Heggestøyl]

[-- Attachment #1: Type: text/plain, Size: 304 bytes --]

On Thu, Apr 7, 2016 at 7:30 PM, Etienne Prud'homme 
<e.e.f.prudhomme@gmail.com> 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

[-- Attachment #2: Type: text/html, Size: 437 bytes --]