GNU bug report logs - #59660
29.0.50; typescript-ts-mode consistently fontifies method-names incorrectly

Previous Next

Package: emacs;

Reported by: jostein <at> kjonigsen.net

Date: Mon, 28 Nov 2022 15:05:03 UTC

Severity: normal

Found in version 29.0.50

Fixed in version 29.1

Done: Yuan Fu <casouri <at> gmail.com>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Jostein Kjønigsen <jostein <at> secure.kjonigsen.net>
To: Eli Zaretskii <eliz <at> gnu.org>, Yuan Fu <casouri <at> gmail.com>
Cc: 59660 <at> debbugs.gnu.org, jostein <at> kjonigsen.net, theo <at> thornhill.no
Subject: bug#59660: 29.0.50; typescript-ts-mode consistently fontifies method-names incorrectly
Date: Tue, 29 Nov 2022 14:58:59 +0100
Cc: 59660 <at> debbugs.gnu.org, theo <at> thornhill.no
>> From: Yuan Fu <casouri <at> gmail.com>
>> Date: Mon, 28 Nov 2022 15:06:30 -0800
>>
>> Maybe you can remove the :override flag for the property_identifier rule, so it doesn’t shadow the function rule? Would that work better?
I plan to test that as quickly as I time permits. If we can make both 
behaviours work, that would obviously be the best.
> This could be a personal preference, perhaps?  It isn't clear to me that
> only one of the two is definitely correct.  So maybe we need a user option
> for which one overrides which?

I honestly think this is reasonably clear.

"Properties" with getters and setters are at this point in time almost 
entirely unused in EcmaScript/TypeScript space. They are optional and 
have complicated syntax, and don't play well with JSON serialization 
(and thus can't be used in REST APIs).

You however cannot write a program without defining or invoking 
methods/functions. So not having method-fontification working correctly 
will literally impact **all** TypeScript-users.

--
Jostein






This bug report was last modified 2 years and 197 days ago.

Previous Next


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