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.

Sign up for GitHub

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

Jump to bottom

make markdown todo list render into disable checkbox #62

Closed
wants to merge 2 commits into from
Closed

make markdown todo list render into disable checkbox #62

wants to merge 2 commits into from

Conversation

MrEleven
Copy link

static blog don't need interactive form, so make the todolist disable.

MrEleven and others added 2 commits May 22, 2018 10:35
@MrEleven
static blog don't need interactive form
8a1d055 
static blog don't need interactive form, so make the todolist render disable like github.
@MrEleven
support todo disable checkbox test
29ce337
@coveralls
Copy link

coveralls commented May 22, 2018 •
edited
Loading

Coverage Status

Coverage remained the same at 89.091% when pulling 29ce337 on MrEleven:master into 26bc183 on hexojs:master.

@tomap
Copy link
Contributor

tomap commented Sep 10, 2018

It looks like a change/breaking change. It should probably be a minor or major version increment?
Or an option?

tomap
tomap requested changes Nov 5, 2018
View reviewed changes
Copy link
Contributor

@tomap tomap left a comment

Choose a reason for hiding this comment

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

Could you update version to 0.4.0? In package.json

@tomap tomap mentioned this pull request Feb 24, 2019
Merged
@yoshinorin
Copy link
Member

@MrEleven
We released v1.0.0.
This issue probably already been resolved. Thanks :)

@yoshinorin yoshinorin closed this May 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tomap tomap tomap requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@MrEleven @coveralls @tomap @yoshinorin