Skip to content

Conversation

@pfitzseb
Copy link
Member

@pfitzseb pfitzseb commented Nov 4, 2020

Fixes #1742.

"command": "language-julia.executeFile",
"title": "Julia: Execute File"
"title": "Julia: Execute File",
"icon": "$(play)"
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unrelated fix to make sure we display an icon above the editor to run a file instead of the command's title.

Copy link
Member

@aviatesk aviatesk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left few minor comments, but the code looks good to me in general.

Just to make sure, the current command definitions allows us to invoke them from explorer via GUI and also from keyboard shortcuts ? I would like to be able to invoke them from keyboard shortcut as in Juno.

pfitzseb and others added 3 commits November 4, 2020 15:00
Co-authored-by: Shuhei Kadowaki <40514306+aviatesk@users.noreply.github.com>
Co-authored-by: Shuhei Kadowaki <40514306+aviatesk@users.noreply.github.com>
@pfitzseb
Copy link
Member Author

pfitzseb commented Nov 4, 2020

Just to make sure, the current command definitions allows us to invoke them from explorer via GUI and also from keyboard shortcuts ? I would like to be able to invoke them from keyboard shortcut as in Juno.

Yep. VSCode gives us the right URI in all contexts I've tried.

aviatesk and others added 2 commits November 4, 2020 23:58
@pfitzseb pfitzseb merged commit 5d08ff3 into master Nov 4, 2020
@davidanthoff davidanthoff deleted the sp/cdactivatehere branch February 11, 2021 19:47
@oppo-source oppo-source removed the request for review from a team April 16, 2021 07:35
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.

cd to file/folder command

3 participants