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

markdown.copyFiles.destination doesn't apply to .mov files? #239016

Closed
roblourens opened this issue Jan 28, 2025 · 3 comments · Fixed by #239046
Closed

markdown.copyFiles.destination doesn't apply to .mov files? #239016

roblourens opened this issue Jan 28, 2025 · 3 comments · Fixed by #239046
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders markdown Markdown support issues verified Verification succeeded
Milestone

Comments

@roblourens
Copy link
Member

I pasted a png and a mov file from my Desktop. I have

"markdown.copyFiles.destination": {
    "**/*": "${fileName}.${unixTime}.md"
  },

It's confusing which files this applies to, the setting doesn't say specifically. The doc says "images" mostly but also "media files" in one place. https://code.visualstudio.com/docs/languages/markdown#_inserting-images-and-links-to-files. And I just expected that if it works on images, it would work on videos?

Image
@mjbvz
Copy link
Collaborator

mjbvz commented Jan 28, 2025

Yes needs a doc update and small fix. It should work for any media file (image, video, audio) but importantly we previously didn't consider .mov as media

@mjbvz mjbvz added this to the January 2025 milestone Jan 28, 2025
@mjbvz mjbvz added the bug Issue identified by VS Code Team member as probable bug label Jan 28, 2025
@mjbvz mjbvz modified the milestones: January 2025, February 2025 Jan 28, 2025
@mjbvz
Copy link
Collaborator

mjbvz commented Jan 28, 2025

Pushing to feb since not a regression and I want to try a more complete fix

@Jorge-Martin777
Copy link

"markdown.copyFiles.destination": {
"**/*": "${fileName}.${unixTime}.md"
`},```

@mjbvz mjbvz closed this as completed in f9213d7 Feb 4, 2025
@vs-code-engineering vs-code-engineering bot added unreleased Patch has not yet been released in VS Code Insiders insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Feb 4, 2025
@roblourens roblourens added the verified Verification succeeded label Feb 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders markdown Markdown support issues verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants