unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: "Simen Heggestøyl" <simenheg@gmail.com>
To: Tom Tromey <tom@tromey.com>, Stefan Monnier <monnier@iro.umontreal.ca>
Cc: 13425@debbugs.gnu.org
Subject: bug#13425: close this bug?
Date: Sun, 29 Jan 2017 15:01:55 +0100	[thread overview]
Message-ID: <1485698515.4344.0@smtp.gmail.com> (raw)
In-Reply-To: <87efzqhh93.fsf@tromey.com>

I agree that the SMIE-based indenter is good, and I'd like to keep it. 
The
mentioned problem of aligning lines after a comma is indeed the only 
problem
I've had with it.

On Thu, Jan 26, 2017 at 12:06 AM, Tom Tromey <tom@tromey.com> wrote:
> I did spend a bit of time trying to get it to line up continued
> properties under the ":", like:
> 
>     font: 15px "Helvetica Neue",
> 	  Helvetica,
> 	  Arial,
> 	  "Nimbus Sans L",
> 	  sans-serif;
> 
> ... but all I really accomplished was realizing that I don't 
> understand
> SMIE.

I did that too, but I had to reach the same conclusion...

If we aren't able to fix this with problem the SMIE-based indenter, and
Evgeni's indenter handles it better, I think we should consider it.

But as said, I think it would be ideal to fix and keep the indenter we 
have.

Stefan, do you have time to look into it, or do you have some pointers 
to how
we could fix this problem with the SMIE-based indenter?

-- Simen






  reply	other threads:[~2017-01-29 14:01 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-13  7:43 bug#13425: new css-mode indenter E Sabof
2013-01-15  5:00 ` Stefan Monnier
2013-01-15 19:32   ` E Sabof
2013-01-17  0:17     ` E Sabof
2017-01-25 23:06 ` bug#13425: close this bug? Tom Tromey
2017-01-29 14:01   ` Simen Heggestøyl [this message]
2017-01-29 17:16   ` Stefan Monnier
2017-01-29 17:38     ` Simen Heggestøyl
2017-01-29 18:15       ` Stefan Monnier
2017-02-02 19:12         ` Simen Heggestøyl
2017-02-03  0:02           ` Stefan Monnier
2017-02-04 19:31             ` Simen Heggestøyl
2017-01-30 22:17     ` Tom Tromey
2017-01-30 22:28       ` Stefan Monnier

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1485698515.4344.0@smtp.gmail.com \
    --to=simenheg@gmail.com \
    --cc=13425@debbugs.gnu.org \
    --cc=monnier@iro.umontreal.ca \
    --cc=tom@tromey.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

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).