[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#64814: 29.1; typescript-ts-mode indentation broken since 235561a
From: |
Theodor Thornhill |
Subject: |
bug#64814: 29.1; typescript-ts-mode indentation broken since 235561a |
Date: |
Mon, 24 Jul 2023 16:24:11 +0200 |
Eli Zaretskii <eliz@gnu.org> writes:
>> From: Jimmy Yuen Ho Wong <wyuenho@gmail.com>
>> Date: Sun, 23 Jul 2023 22:23:04 +0100
>>
>>
>> Reproduction:
>>
>> 1. emacs -q
>> 2. C-x C-f ~/test.ts
>> 3. M-x typescript-ts-mode
>> 4. Insert
>>
>> ```typescript
>> if (true) {
>> console.log('hello');
>> }
>> ```
>>
>> 5. Move point to the beginning of the second line
>> 6. TAB
>>
>> Expectation:
>>
>> I expect the indentation behavior to be the same as js-ts-mode, in which
>> the if statement body will be indented as I type RET after {, or on TAB
>> on the second line, but commit 235561a seems to have broken this behavior.
>
> Thanks.
>
> Theo and Vincenzo, could you please look into this?
Fixed in 4e977136d3148c3baa0e03568c7a2c7dc28eea65. Funnily enough, this
wasn't indentation related, but font-locking related. Seems
initialization of the major mode didn't work properly because the compat
check for font lock didn't account for typescript as a language. The fix
was simple enough.
Thanks,
Theo