search

irrationalistic / atom-tasks

Handle tasks/todo files in atom
MIT License
154 stars 22 forks source link

status bar count fix #83

Closed paulroub closed 6 years ago

paulroub commented 7 years ago

Make done/cancelled token names consistent, fixing #82

irrationalistic commented 6 years ago

Thanks for doing this! Could you double check that the syntax highlighting still works when you have markdown within the task item? That was one of the reasons I kept changing these around :)

paulroub commented 6 years ago

As far as I can tell, the highlighting is unchanged when I revert to the release.

irrationalistic commented 6 years ago

I pulled this in locally and it does check out, but i think we actually should re-visit the code. Changing the classnames like this is a bit of a bandaid (primarily because i didn't really understand how to pull in the markdown syntax!) I have a fix locally that properly enables markdown which i'll put up as a separate pull request for you to take a look :)

irrationalistic commented 6 years ago

Closing this in favor of #85