fix-todo-list-checkbox-issue

[leollon]

After page rendered, visitor still can click on the checkbox. So I'm trying to fix it.

[coveralls]

Coverage remained the same at 87.234% when pulling 5a9a478 on wkzcn:fix-todo-list-checkbox-issue into ffeca47 on hexojs:Fix-checklist-issue.

[NoahDragon]

Just wonder why need to disable the check box? Even user ticked it, after refresh, it will be set back to the original.

[leollon]

ok,It means that there is no need to do that? so I gonna close this PR.

[NoahDragon]

We may need it, but just feel a little bit odd if the check box is gray. We can keep this open and let others discuss this.

[coveralls]

Coverage remained the same at 87.234% when pulling 5a9a478 on wkzcn:fix-todo-list-checkbox-issue into ffeca47 on hexojs:Fix-checklist-issue.

[leollon]

ok.I think that if thecheckbox is gray, it means that this task has been done.

[NoahDragon]

@wkzcn maybe we could add a setting to support such behavior, or users modify the css class to disable it. Either way, more make sense to me.

[leollon]

@NoahDragon I can't agree more. Is it more convenient for user to add a setting to support such behavior?

[MrEleven]

I think disable checkbox is better, we can't post the form data in a static blog.
it just show the infomation todo item finished or unfinished, we can't change it by ticked the checkbox.

[yoshinorin]

@leollon
We released v1.0.0.
This issue probably already been resolved. Please check it. Thanks :)