Andrew Dupont

That's bizarre, since I was the one schmuck using this grammar for months before we added it to core and this never happened to me despite the fact that I've...

> Just wanted to note that I run into this frequently when typing Japanese into .mdx files. Yeah, that's the consensus, @xansec. I hope to replace the current grammar with...

Sorry it took me a while to read this one. I'll take a look when I'm able.

@Tamaranch [reported](https://github.com/pulsar-edit/pulsar/issues/872#issuecomment-1899391556) that the `while` keyword is unhighlighted. ~~This is on my radar and will be fixed very soon.~~ (Fixed; see below.)

The `while` keyword is now highlighted; [this commit](https://github.com/pulsar-edit/pulsar/pull/859/commits/3cd7557504fb3da35a021b12447700b24ca799f8) contains two other small fixes. Subscribe to #859 to be notified when this fix (and many others) land.

I expect some of these to be bugs in `tree-sitter-bash`, sadly, but I won't know for sure until I try some of these tonight.

> I also think there is a bug with command substitution (here with One Dark theme): A few bugs here! One is that the `$(` is not scoped with a...

> After this, the rest of the code is no longer highlighted: ``` var="${var/'a'*/b}" ``` @Tamaranch, how would you describe what that line of code is doing? I know a...

Meanwhile, everything else you mentioned is addressed [in this commit](https://github.com/pulsar-edit/pulsar/pull/859/commits/5e84a7cda414e7b6fd6aa3db42e16ac6d538ab08). I'll open a ticket against `tree-sitter-bash` when I hear back from you about the parameter expansion thing.

> > Meanwhile, everything else you mentioned is addressed [in this commit](https://github.com/pulsar-edit/pulsar/pull/859/commits/5e84a7cda414e7b6fd6aa3db42e16ac6d538ab08). > > I can confirm it's fixed, but I'm not sure if it's best to use the same...