[Top][All Lists]

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

[debbugs-tracker] bug#19815: closed (25.0.50; [PATCH] css-mode: function

From: GNU bug Tracking System
Subject: [debbugs-tracker] bug#19815: closed (25.0.50; [PATCH] css-mode: function call indentation)
Date: Mon, 09 Feb 2015 17:28:02 +0000

Your message dated Mon, 09 Feb 2015 12:27:45 -0500
with message-id <address@hidden>
and subject line Re: bug#19815: 25.0.50; [PATCH] css-mode: function call 
has caused the debbugs.gnu.org bug report #19815,
regarding 25.0.50; [PATCH] css-mode: function call indentation
to be marked as done.

(If you believe you have received this mail in error, please contact

19815: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=19815
GNU Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: 25.0.50; [PATCH] css-mode: function call indentation Date: Sun, 08 Feb 2015 13:19:44 +0100
Sometimes it is convenient to break CSS function calls up in multiple
lines. The current CSS indentation engine does not handle this well
when the parameters don't begin at the same line as the opening
paren. For instance:

div {
    background: linear-gradient(
                        to right, red, orange, yellow,
                        green, blue, indigo, violet

div {
    background: linear-gradient(

I would instead expect the following indentation:

div {
    background: linear-gradient(
        to right, red, orange, yellow,
        green, blue, indigo, violet

div {
    background: linear-gradient(

The following patch accomplishes this:

diff --git a/lisp/textmodes/css-mode.el b/lisp/textmodes/css-mode.el
index f6a3ca6..ab911ce 100644
--- a/lisp/textmodes/css-mode.el
+++ b/lisp/textmodes/css-mode.el
@@ -331,8 +331,9 @@
     (`(:elem . basic) css-indent-offset)
     (`(:elem . arg) 0)
     (`(:list-intro . ,(or `";" `"")) t) ;"" stands for BOB (bug#15467).
-    (`(:before . "{") (if (smie-rule-hanging-p)
-                         (smie-rule-parent 0)))))
+    (`(:before . ,(or "{" "("))
+     (if (smie-rule-hanging-p)
+         (smie-rule-parent 0)))))
 (define-derived-mode css-mode fundamental-mode "CSS"

In GNU Emacs (x86_64-unknown-linux-gnu, GTK+ Version 3.14.5)
 of 2015-02-08 on x240
Repository revision: dcec731abfc110f719f0890fd0769e099393af7d
Windowing system distributor `The X.Org Foundation', version 11.0.11602901
System Description: Debian GNU/Linux 8.0 (jessie)

Configured using:
 `configure --without-makeinfo'

Configured features:

Important settings:
  value of $LANG: en_US.UTF-8
  locale-coding-system: utf-8-unix

Major mode: Lisp Interaction

Minor modes in effect:
  tooltip-mode: t
  global-eldoc-mode: t
  electric-indent-mode: t
  mouse-wheel-mode: t
  tool-bar-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  blink-cursor-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t
  line-number-mode: t

Recent messages:
For information about GNU Emacs and the GNU system, type C-h C-a.

Load-path shadows:
None found.

(shadow sort gnus-util mail-extr emacsbug message dired format-spec
rfc822 mml easymenu mml-sec mm-decode mm-bodies mm-encode mail-parse
rfc2231 mailabbrev gmm-utils mailheader sendmail rfc2047 rfc2045
ietf-drums mm-util help-fns mail-prsvr mail-utils time-date tooltip
eldoc electric uniquify ediff-hook vc-hooks lisp-float-type mwheel x-win
x-dnd tool-bar dnd fontset image regexp-opt fringe tabulated-list
newcomment elisp-mode lisp-mode prog-mode register page menu-bar
rfn-eshadow timer select scroll-bar mouse jit-lock font-lock syntax
facemenu font-core frame cham georgian utf-8-lang misc-lang vietnamese
tibetan thai tai-viet lao korean japanese hebrew greek romanian slovak
czech european ethiopic indian cyrillic chinese case-table epa-hook
jka-cmpr-hook help simple abbrev minibuffer cl-preloaded nadvice
loaddefs button faces cus-face macroexp files text-properties overlay
sha1 md5 base64 format env code-pages mule custom widget
hashtable-print-readable backquote make-network-process dbusbind
gfilenotify dynamic-setting system-font-setting font-render-setting
move-toolbar gtk x-toolkit x multi-tty emacs)

Memory information:
((conses 16 74953 5948)
 (symbols 48 17996 0)
 (miscs 40 43 124)
 (strings 32 10952 5168)
 (string-bytes 1 296625)
 (vectors 16 9370)
 (vector-slots 8 386714 15587)
 (floats 8 70 63)
 (intervals 56 175 9)
 (buffers 976 11)
 (heap 1024 23572 1045))

--- End Message ---
--- Begin Message --- Subject: Re: bug#19815: 25.0.50; [PATCH] css-mode: function call indentation Date: Mon, 09 Feb 2015 12:27:45 -0500 User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.0.50 (gnu/linux)
> -    (`(:before . "{") (if (smie-rule-hanging-p)
> -                         (smie-rule-parent 0)))))
> +    (`(:before . ,(or "{" "("))
> +     (if (smie-rule-hanging-p)
> +         (smie-rule-parent 0)))))

Thanks, installed,


--- End Message ---

reply via email to

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