GNU bug report logs - #76704
30.1; Indentation of braces on separate line in js-ts-mode

Previous Next

Package: emacs;

Reported by: Fabian Brosda <f.brosda <at> gmx.de>

Date: Mon, 3 Mar 2025 04:33:03 UTC

Severity: minor

Found in version 30.1

Done: Eli Zaretskii <eliz <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


Message #52 received at 76704 <at> debbugs.gnu.org (full text, mbox):

From: Vincenzo Pupillo <v.pupillo <at> gmail.com>
To: juri <at> linkov.net, Eli Zaretskii <eliz <at> gnu.org>,
 Fabian Brosda <f.brosda <at> gmx.de>, Daniel Colascione <dancol <at> dancol.org>
Cc: 76704 <at> debbugs.gnu.org
Subject: Re: bug#76704: 30.1;
 Indentation of braces on separate line in js-ts-mode
Date: Mon, 07 Apr 2025 11:39:25 +0200
Ciao Fabian,

In data lunedì 7 aprile 2025 09:33:26 Ora legale dell’Europa centrale, Fabian 
Brosda ha scritto:
> Hi Vincenzo,
> 
> thanks for the new patch.  The results look very good and I couldn't
> find any example, which gives unexpected indentation in js-ts-mode now.
> 
> Regarding the use of js-switch-indent-offset for js-mode, the results
> are still different, depending on the brace being on the same line or
> not.  Should I open a separate bug report for this?

whith js-switch-indent-offset = 2 and js-indent-offset = 4, with js-mode the 
result is:

function switch_test(a)
{
    switch (a) {
      case 1:
          x;
          break;
      case 2:
          y;
          break;
      default:
          z;
          break;
    }

    switch (a)
    {
        case 1:
        x;
        break;
        case 2:
        y;
        break;
        default:
        z;
        break;
    }
}

js-ts-mode the result is:

function switch_test(a)
{
    switch (a) {
      case 1:
          x;
          break;
      case 2:
          y;
          break;
      default:
          z;
          break;
    }

    switch (a)
    {
      case 1:
          x;
          break;
      case 2:
          y;
          break;
      default:
          z;
          break;
    }
}

For me the js-ts mode is correct, but I am not an expert. 
@Daniel WDYT?


Vincenzo

> 
> Again thanks for your time and the prompt updates.
> 
> Fabian







This bug report was last modified 90 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.