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 Github Task Lists #86

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Added Github Task Lists #86

wants to merge 1 commit into from

Conversation

pl86
Copy link

@pl86 pl86 commented Sep 25, 2015

  • replace "- [ ]" and "- [x]" to checkboxes
  • live edit current document on click checkbox

zrzut ekranu 2015-09-25 o 13 16 36

@prateekjadhwani
Copy link

+1 for @pl86
I guess you will just need to wait for the PR to be approved. :)

renderer.listitem = function(text) {
if (/^\[ \] \s*/.test(text)) {
text = '<input type="checkbox" class="gfm-checkbox"> ' + text.substr(4);
} else if (/^\[x\] \s*/.test(text)) {
Copy link
Owner

Choose a reason for hiding this comment

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

This should be case insensitive, so [x] or [X] can be used to check an item.

@gruehle
Copy link
Owner

gruehle commented Sep 26, 2015

@pl86 thanks for the pull request!

@pl86
Copy link
Author

pl86 commented Sep 27, 2015

@gruehle thanks for your comments. You are right, this extension should only preview Markdown files. I just found it super easy to create something what I think may be useful.

@prateekjadhwani
Copy link

Waiting for the newer version with this change. :)

@ezequias
Copy link

Nothing new in March 2017.

@gruehle
Copy link
Owner

gruehle commented Mar 14, 2017

@prateekjadhwani @ezequias - I would gladly merge a pull request that added read-only support for task lists. I just don't have time to do the work myself.

@prateekjadhwani
Copy link

I dont think @pl86 has been active on github recently to fix the merge conflicts. And that reminds me. Is anyone still using Brackets?

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.

4 participants