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

Extract the MarkDown editor into an extension #6106

Closed
egamma opened this issue May 4, 2016 · 6 comments
Closed

Extract the MarkDown editor into an extension #6106

egamma opened this issue May 4, 2016 · 6 comments
Assignees
Labels
markdown Markdown support issues plan-item VS Code - planned item for upcoming
Milestone

Comments

@egamma
Copy link
Member

egamma commented May 4, 2016

@bpasero please document the features that are still missing

@egamma egamma added the plan-item VS Code - planned item for upcoming label May 4, 2016
@egamma egamma added this to the May 2016 milestone May 4, 2016
@egamma egamma mentioned this issue May 4, 2016
87 tasks
@bpasero bpasero changed the title Extrace the MarkDown editor into an extension Extract the MarkDown editor into an extension May 4, 2016
@bpasero
Copy link
Member

bpasero commented May 5, 2016

@egamma @kieferrm @Tyriar I started a markdown extension last milestone and you can find it here: https://github.com/bpasero/vscode-markdown

It was created using the TM language from [1] and it leverages our virtual document preview support to show a preview. It depends on markdown-it [2] and highlight.js for the preview rendering.

There is functionality missing to make it a replacement of our markdown support:

  • we need to investigate how to support embedded languages in the markdown editor (not for the preview, this is for the source) so that code blocks get colored properly. according to @aeschli TM supports the notion of referencing other grammars from within a grammar. ideally we could reference our installed TM bundles for any code block that is being used in the markdown source
  • we need to change the extension to update the preview based on document edits so that the preview updates while the user is typing
  • we need to finish the CSS styling for the preview so that it is in sync with what we have today and also works in dark and light theme and any other theme from extensions (also depends on Webview: allow me to adjust my content based on theme #5737)
  • we need to bring back the functionality that the preview scroll position stays stable when you make changes or even find a way to scroll the preview in sync with the editor scrolling (that would be a cool feature)
  • we currently do not have support to contribute actions to the context menu of the explorer or to the editor title area, but we have actions there to open a preview from a markdown file in both places and we also have an action to jump back to the source from a markdown preview
  • there is a setting in our markdown preview to configure a custom CSS style to use and this setting needs to be added to the extension too
  • anything labeled markdown [3] is something to revisit once we work on the extension

[1] https://github.com/textmate/markdown.tmbundle/blob/master/Syntaxes/Markdown.tmLanguage
[2] https://github.com/markdown-it/markdown-it
[3] https://github.com/Microsoft/vscode/issues?q=is%3Aopen+is%3Aissue+label%3Amarkdown

@felixfbecker
Copy link
Contributor

@bpasero I love that you are switching from marked to markdown-it, it seems much more active and is pluggable. I would really love to see checklists integrated by default, it is awesome for ToDo-lists - would it work to actually check a checkbox in the preview and it would change the source file? That would be awesome

@tht13
Copy link
Contributor

tht13 commented May 28, 2016

Returning to source in the markdown preview is related to this issue #6941
There is no way to detect a virtual document with focus when a command is triggered.

@kieferrm
Copy link
Member

kieferrm commented May 31, 2016

Current state:

@kieferrm
Copy link
Member

Current state:

@kieferrm kieferrm mentioned this issue Jun 22, 2016
3 tasks
@kieferrm
Copy link
Member

kieferrm commented Jun 22, 2016

Done-Done criteria:

  • Test Plan Item created
  • Keyboard accessible
  • Screen reader accessible
  • Works with the different themes, including the high contrast theme
  • Telemetry events in place
  • Release notes updated

@kieferrm kieferrm closed this as completed Jul 1, 2016
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
markdown Markdown support issues plan-item VS Code - planned item for upcoming
Projects
None yet
Development

No branches or pull requests

6 participants