unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Yuan Fu <casouri@gmail.com>
To: Randy Taylor <dev@rjt.dev>
Cc: jostein@kjonigsen.net, 59833@debbugs.gnu.org,
	"Theodor Thornhill" <theo@thornhill.no>,
	59833-done@debbugs.gnu.org,
	"Jostein Kjønigsen" <jostein@secure.kjonigsen.net>
Subject: bug#59833: 29.0.60; json-ts-mode fontifies everything as string
Date: Mon, 5 Dec 2022 12:22:09 -0800	[thread overview]
Message-ID: <A7C67181-10A0-4712-8C3F-9827FC3B1EE2@gmail.com> (raw)
In-Reply-To: <aQA0pNlSA_soXEuWsIIv3uvEPFIuV5NkzgT2or7HVT6U45nqIfgoJSBuAkaduAFa1RKx5_TOdJEtkwnFmc47r_IH6Ll2MFKf8erRdnT4_Ss=@rjt.dev>



> On Dec 5, 2022, at 12:19 PM, Randy Taylor <dev@rjt.dev> wrote:
> 
> On Monday, December 5th, 2022 at 15:14, Theodor Thornhill <theo@thornhill.no> wrote:
>> 
>> 
>> On 5 December 2022 20:57:55 CET, Yuan Fu casouri@gmail.com wrote:
>> 
>>>> On Dec 5, 2022, at 7:37 AM, Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors bug-gnu-emacs@gnu.org wrote:
>>>> 
>>>> On 5 December 2022 16:35:12 CET, Randy Taylor dev@rjt.dev wrote:
>>>> 
>>>>> On Monday, December 5th, 2022 at 10:03, Theodor Thornhill theo@thornhill.no wrote:
>>>>> 
>>>>>> Yeah js-json-mode is a lot simpler, and is imo too simple. The json-mode
>>>>>> is not infact default in emacs, but there are two, one in melpa and one
>>>>>> in elpa, IIRC.
>>>>>> 
>>>>>> Anyway, last patch for now - are you both satisfied with this one?
>>>>>> 
>>>>>> Theo
>>>>> 
>>>>> It's perfect! Sorry for all the trouble :D.
>>>> 
>>>> No problem :)
>>>> 
>>>> Yuan, will you commit this one?
>>> 
>>> Yeah. IIUC :override t is not necessary for pair, right? If so, I’ll apply the patch and remove the :override, is that OK?
>>> 
>>> Yuan
>> 
>> 
>> Please test it first, but sure! We want it to not be string face at least :)
>> 
>> Theo
> 
> It's necessary, since string comes first and pair needs to override it (just tested).

Ah, yes, of course, sorry for my ignorance. Applied!

Yuan




  reply	other threads:[~2022-12-05 20:22 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-12-05 10:40 bug#59833: 29.0.60; json-ts-mode fontifies everything as string Jostein Kjønigsen
2022-12-05 11:24 ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-05 11:35   ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-05 11:53     ` Jostein Kjønigsen
2022-12-05 12:06       ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-05 13:52   ` Randy Taylor
2022-12-05 13:58     ` Randy Taylor
2022-12-05 14:12       ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-05 14:42         ` Randy Taylor
2022-12-05 14:52           ` Jostein Kjønigsen
2022-12-05 15:00             ` Eli Zaretskii
2022-12-05 15:01             ` Randy Taylor
2022-12-05 15:03             ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-05 15:35               ` Randy Taylor
2022-12-05 15:37                 ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-05 19:57                   ` Yuan Fu
2022-12-05 20:14                     ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors
2022-12-05 20:19                       ` Randy Taylor
2022-12-05 20:22                         ` Yuan Fu [this message]
2022-12-05 20:38                           ` Theodor Thornhill via Bug reports for GNU Emacs, the Swiss army knife of text editors

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=A7C67181-10A0-4712-8C3F-9827FC3B1EE2@gmail.com \
    --to=casouri@gmail.com \
    --cc=59833-done@debbugs.gnu.org \
    --cc=59833@debbugs.gnu.org \
    --cc=dev@rjt.dev \
    --cc=jostein@kjonigsen.net \
    --cc=jostein@secure.kjonigsen.net \
    --cc=theo@thornhill.no \
    /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).