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

Use json view for jres and refresh file explorer on new generated files #82

Merged
merged 2 commits into from
Feb 22, 2023

Conversation

jwunderl
Copy link
Member

@jwunderl jwunderl commented Feb 22, 2023

set .jres as an alias for .json and .blocks as an alias for .html so they're formattable / vscode can treat appropriately. (xml is more accurate for .blocks, but then you have to install a custom formatter as there's no default / etc). I suspect this might actually help out in #70 as liveshare should 'learn' how to handle .jres as just json, but this has been hard to test for me as I somehow got my vscode install to identify .jres / .blocks as these types no matter what I do (I've looked through all the settings files I can find and culled the file assocations but it keeps popping up somehow) -- either way this is how python extension does it / little harm in adding it https://github.com/microsoft/vscode-python/blob/main/package.json#L1517-L1520

One more thing this gets us; anyone who downloads a makecode project with a .blocks or .jres should get us recommended when clicking on 'Search marketplace for extension' I think?

image

Also, refresh file explorer after the asset editor gives new files (e.g. a tilemap.g.ts is added to the project); currently they don't show up till something else triggers a refresh which was a little confusing.

@jwunderl jwunderl requested a review from riknoll February 22, 2023 20:48
@jwunderl jwunderl merged commit 9453558 into main Feb 22, 2023
@jwunderl jwunderl deleted the useJsonViewForJres branch February 22, 2023 23:59
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 this pull request may close these issues.

2 participants