Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bash syntax problem in test expressions #51

Open
glhez opened this issue May 25, 2020 · 1 comment
Open

Bash syntax problem in test expressions #51

glhez opened this issue May 25, 2020 · 1 comment

Comments

@glhez
Copy link

glhez commented May 25, 2020

Hello,

With Syntax Highlighter 0.4.4. I have the following colouration which I think is wrong:

When the extension is enabled: we can see that the string is no longer coloured with the string color: it makes it harder to read (which is not what the extension is trying to do :)):
synhighlite_on

Without the extension, this is correct:
synhighlite_off

The extension behave well for this kind of expression:

if [[ -n "${ZETA_BASH_GITEX_CONF}" && ( "${options[refresh]}" == 'yes' || ! -f "${ZETA_BASH_GITEX_CONF}" ) ]];

The quoted 'yes' make it works.

Another problem is with number (I'm using FiraCode, the != is displayed as unicode ≠ but it is only a ligature, should not be the problem here):

image

The first 0 is colored blue, the second is colored gray (or something like that) while I would expect them to use the same color..

I also have a crash after extended use (>30 minutes I think?) but as I played with the Enable/Disable, I no longer have the crash trace... And this is a separate problem anyway.

VSCodium information:

  • Version: 1.45.1 (user setup)
  • Commit: 5763d909d5f12fe19f215cbfdd29a91c0fa9208a
  • Date: 2020-05-15T10:23:54.161Z
  • Electron: 7.2.4
  • Chrome: 78.0.3904.130
  • Node.js: 12.8.1
  • V8: 7.8.279.23-electron.0
  • OS: Windows_NT x64 10.0.18363
@AnrDaemon
Copy link

Another to the list

image

The "$(dirname is colored like it is a single token. Which is not.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants