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

[icon themes] seti theme should associate also to non build-in language modes (elm, caml, nunjucks..) #47432

Closed
ShalokShalom opened this issue Apr 8, 2018 · 20 comments
Assignees
Labels
feature-request Request for new features or functionality themes Color theme issues verification-needed Verification of issue is requested verified Verification succeeded
Milestone

Comments

@ShalokShalom
Copy link

ShalokShalom commented Apr 8, 2018

  • VSCode Version: 1.22
  • OS Version: KaOS

Steps to Reproduce:

  1. Set files.defaultLanguage": "elm",
  2. Open new file
  3. Check if the logo of the language appears left of its file name in the tab bar.
  4. Repeat this for a couple of other languages

The issue happens randomly.
When it applies fine, does this count for all created files.
The issue appears, when you change the setting.
So, the issue is obvious, that it fails to apply randomly.
In ALL cases get the correct language displayed in the status bar.

Does this issue occur when all extensions are disabled?: Yes/No

Yes

@roblourens
Copy link
Member

Do you have some extension installed that contributes Elm? Does it appear in the language mode picker?

@roblourens roblourens added the info-needed Issue requires more information from poster label Apr 9, 2018
@ShalokShalom
Copy link
Author

ShalokShalom commented Apr 9, 2018

Yes, the extension is installed and it appears in the status bar, if thats it, what you call language mode picker.

@roblourens
Copy link
Member

Which extension?

@ShalokShalom
Copy link
Author

@roblourens
Copy link
Member

Works for me. When you open the new file, what language mode does it use instead?

@ShalokShalom
Copy link
Author

Untitled document, so a plain text document, I guess.

@roblourens
Copy link
Member

Does it say "Plain text" in the status bar or something else?

@roblourens
Copy link
Member

image

The popup that you get when you click that is what I mean by language mode picker.

@ShalokShalom
Copy link
Author

ShalokShalom commented Apr 9, 2018

Yeah, I thought so. In the status bar, as I mean.

It shows Elm.

And I just reproduced the issue with ocaml:

ocaml in the settings, ocaml in the status bar and I get a plain text document.

This worked fine yesterday, so it appears seemingly randomly and is independent of the language.

Hnn..

@ShalokShalom ShalokShalom changed the title Open new files by default in .elm doesnt work Open new files in does fail sometimes Apr 9, 2018
@roblourens
Copy link
Member

So to confirm, it shows the correct language in the status bar, but it doesn't use the correct syntax highlighting?

@ShalokShalom ShalokShalom changed the title Open new files in does fail sometimes Open new files in specific formats does fail sometimes Apr 9, 2018
@ShalokShalom
Copy link
Author

Yes, correct.

@ShalokShalom
Copy link
Author

And it does not show the correct format in the tab bar. 🙂

@roblourens
Copy link
Member

What do you mean "the correct format in the tab bar"?

The syntax highlighting works for me. When you save the file, does it become highlighted correctly?

@ShalokShalom
Copy link
Author

ShalokShalom commented Apr 9, 2018

No.

And when you look into the tab bar where the open files are listed, can you see the specific logo of the language, when it loads fine. 🙂 This is not the case, when it fails to load.

You see then a generic text file logo, which is the same for all languages and provides no syntax highlighting.

As said, this is sometimes the case and sometimes does it load correctly.
I can reproduce it by simply changing the setting for files.default.language:

When it loads correctly, count this for so many files as I decide to open.
When the error case happens, is this also the case for all the created files then.

So, in my point of view, the issue is behind the setting:
It seems to apply sometimes, and sometimes not.

In order to reproduce it on your side:

Change the setting of files.default.language to different languages.
Open new files with Ctrl+N
Look at the logo in the tab bar or try syntax highlighting.

@roblourens
Copy link
Member

If it doesn't get highlighted when you save it, then this isn't specific to opening a new file, you're just missing Elm highlighting entirely right?

Can you include a screenshot of it in this state so we're clear on what the problem is?

@ShalokShalom
Copy link
Author

No, because it works always, when I create a specific .elm file.

I reworked the comment above.

@ShalokShalom
Copy link
Author

ShalokShalom commented Apr 9, 2018

As you can see, is the elm extension in the first tab working. In a newly opened file with the files.default.language setting does it sometimes apply and sometimes not. This is independent from the file type. I already had ocaml files created by files.default.language, while it fails in this video. It is clearly a mechanism behind the setting itself, imho. And it seems to fail randomly. So what can cause this?

https://vimeo.com/263954339#42889

EDIT: Hnn, currently does it look like ocaml and elm never apply and the rest do always.
Very spooky.

And, at least now, does it seem like the syntax highlighting is supported, even when the logo does not appear.

@aeschli
Copy link
Contributor

aeschli commented Apr 10, 2018

I assume you use the Seti icon theme.

The reason is the way our Seti icon theme associates the icons. The elm icon is contributed to all files with file extension '.elm', for for the language mode elm.

That can be improved, but IMO it's not most urgent. I hope you agree.

@aeschli aeschli changed the title Open new files in specific formats does fail sometimes [icon themes] seti theme should associate also elm and caml to language modes. Apr 10, 2018
@aeschli aeschli added themes Color theme issues and removed info-needed Issue requires more information from poster labels Apr 10, 2018
@aeschli aeschli added this to the Backlog milestone Apr 10, 2018
@aeschli aeschli added the feature-request Request for new features or functionality label Apr 10, 2018
@ShalokShalom
Copy link
Author

Oh I see. The OCaml mode also showed sometimes the logo and sometimes not, when created by the files.default.language setting.

@aeschli aeschli changed the title [icon themes] seti theme should associate also elm and caml to language modes. [icon themes] seti theme should associate also to non build-in language modes (elm, caml, nunjucks..) May 14, 2018
@aeschli aeschli modified the milestones: Backlog, August 2018 Aug 10, 2018
@aeschli aeschli added the verification-needed Verification of issue is requested label Aug 10, 2018
@aeschli
Copy link
Contributor

aeschli commented Aug 10, 2018

To verify:

  • set seti theme
  • install a language extension, e.g. elm
  • create an untitled file, set the language to elm
  • verify that the icon is the elm icon

aeschli added a commit that referenced this issue Aug 10, 2018
@roblourens roblourens added the verified Verification succeeded label Aug 29, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Sep 24, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality themes Color theme issues verification-needed Verification of issue is requested verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

3 participants