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

Added the "Quick Export" Command #463

Closed
wants to merge 0 commits into from

Conversation

rtfmkiesel
Copy link
Contributor

Added the quick export command, which enables a faster export. (no filename selection and faster type selection via window.showQuickPick)

I mainly did this, because I was annoyed to always go into the settings to change the export type from PDF to HTML and vice versa.

image

image

Maybe addresses #311

Obviously, it would also be possible to add quick export commands for every single type, which would make it one more step faster.

@rtfmkiesel
Copy link
Contributor Author

Any chance of this getting merged? @yhatt

@yhatt
Copy link
Member

yhatt commented Jan 16, 2025

I think it requires re-considering the whole of commands to make clear for users.

  • Export slide deck (Quick export)
    • Execute exporting, that it was known as "Quick export" in original issue and this PR.
    • Create the output next to the current file.
      • If the target file was not saved, this command has the same meaning as "Export slide deck as..."
    • The format is using specified in markdown.marp.exportType preference. The step to choice the format may not need.
  • Export slide deck to specified format...
    • Execute quick export, but ask the file format before.
  • Export slide deck as...
    • Execute exporting that it has current behavior.
    • Always open save dialog, and ask where to export.
    • User can choice the format in the save dialog.

@rtfmkiesel
Copy link
Contributor Author

Going to start from scratch, taking your requirements into consideration.

@rtfmkiesel rtfmkiesel deleted the quickexport branch February 3, 2025 08:18
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