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

languageId is javascript but path icon show javascriptreact #100

Closed
tjx666 opened this issue Nov 28, 2021 · 7 comments · Fixed by #102
Closed

languageId is javascript but path icon show javascriptreact #100

tjx666 opened this issue Nov 28, 2021 · 7 comments · Fixed by #102

Comments

@tjx666
Copy link
Collaborator

tjx666 commented Nov 28, 2021

@ionutvmi
I config the .jsx files the language to javascript, but the path suggestion list icon still javascriptreact.

image

image

@mihai-vlc
Copy link
Owner

The icon is not controlled by the autocomplete extension.
We only configure if the suggestion is a file or a folder.

You may want to use an extension like https://marketplace.visualstudio.com/items?itemName=vscode-icons-team.vscode-icons to customize the icons.

It has more options to fine tune the displayed icons:
https://github.com/vscode-icons/vscode-icons/wiki/FineTuning

@tjx666
Copy link
Collaborator Author

tjx666 commented Dec 2, 2021

I create an issue to vscode microsoft/vscode#138250 (comment)

@tjx666
Copy link
Collaborator Author

tjx666 commented Dec 6, 2021

I had fix it in my fork. just set completionItem.detail to real file path. @ionutvmi

@mihai-vlc
Copy link
Owner

mihai-vlc commented Dec 6, 2021

Feel free to open a PR with the fix.

I can merge it and publish it.
@tjx666

@tjx666
Copy link
Collaborator Author

tjx666 commented Dec 6, 2021

I have change a lot in my fork, with many other optimize and publish my own version. You can have look at my fork https://github.com/tjx666/path-autocomplete/commits/master. You have two choice:

  1. If you would like to merge it, I would stop publish my own version and help maintain your version.
  2. You can just add completion.detail = file.path; to PathAutocompleteProvider.ts at line 49

tjx666 pushed a commit to tjx666/path-autocomplete that referenced this issue Dec 6, 2021
tjx666 pushed a commit to tjx666/path-autocomplete that referenced this issue Dec 6, 2021
@mihai-vlc
Copy link
Owner

hi @tjx666 ,

In general, I am in favor of modernizing the codebase and adding new features to the extension.

I reviewed the changes on the fork and they look ok to me.

I don't have too much time allocated to open source these days but I am also happy to add you as a contributor directly to this repo if you want to maintain these and future changes directly here.

@tjx666
Copy link
Collaborator Author

tjx666 commented Dec 18, 2021

@ionutvmi I have made a pr

mihai-vlc added a commit that referenced this issue Dec 19, 2021
optimize project build & fix #100
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

Successfully merging a pull request may close this issue.

2 participants