@todo not getting rendered on default theme #293

Closed
DenisMir opened this Issue Dec 10, 2014 · 8 comments

Comments

Projects
None yet
4 participants
@DenisMir

Is it normal that the @todo annotation is not getting rendered to the documentation html file in the default theme?

I've got a comment like this:

/**
 * Some example
 *
 * @todo First TODO
 * @todo Second TODO
 */

But I can't find the todos in the rendered output.

@HugoGiraudel

This comment has been minimized.

Show comment
Hide comment
@DenisMir

This comment has been minimized.

Show comment
Hide comment
@DenisMir

DenisMir Dec 10, 2014

I have added the example to the bug repository:

https://github.com/DenisMir/sassdoc-bugs/blob/master/src/todo-bug.scss

I have looked at the same file you included in the last post but reality shows something different. :-)

I have added the example to the bug repository:

https://github.com/DenisMir/sassdoc-bugs/blob/master/src/todo-bug.scss

I have looked at the same file you included in the last post but reality shows something different. :-)

@HugoGiraudel HugoGiraudel added the Bug label Dec 10, 2014

@iamskok

This comment has been minimized.

Show comment
Hide comment
@iamskok

iamskok Dec 20, 2014

I have the same the problem

iamskok commented Dec 20, 2014

I have the same the problem

pascalduez added a commit that referenced this issue Dec 20, 2014

@pascalduez

This comment has been minimized.

Show comment
Hide comment
@pascalduez

pascalduez Dec 20, 2014

Member

Fixed in sassdoc@1.10.9

Thanks for reporting.

Member

pascalduez commented Dec 20, 2014

Fixed in sassdoc@1.10.9

Thanks for reporting.

@pascalduez pascalduez closed this Dec 20, 2014

@iamskok

This comment has been minimized.

Show comment
Hide comment
@iamskok

iamskok Dec 20, 2014

@pascalduez Thanks! Works great 👍

iamskok commented Dec 20, 2014

@pascalduez Thanks! Works great 👍

@pascalduez

This comment has been minimized.

Show comment
Hide comment
@pascalduez

pascalduez Dec 20, 2014

Member

@SassDoc/owners Somehow the default theme is already implementing the renamed annotation. From @todos to @todo introduced in 2.0 (develop).
sassdoc-extras is also already using @todo.

See:
SassDoc/sassdoc#225
SassDoc/sassdoc-theme-default#3
https://github.com/SassDoc/sassdoc-theme-default/blob/master/index.js#L43-L48
https://github.com/SassDoc/sassdoc-extras/blob/master/src/markdown.js#L49-L51

Member

pascalduez commented Dec 20, 2014

@SassDoc/owners Somehow the default theme is already implementing the renamed annotation. From @todos to @todo introduced in 2.0 (develop).
sassdoc-extras is also already using @todo.

See:
SassDoc/sassdoc#225
SassDoc/sassdoc-theme-default#3
https://github.com/SassDoc/sassdoc-theme-default/blob/master/index.js#L43-L48
https://github.com/SassDoc/sassdoc-extras/blob/master/src/markdown.js#L49-L51

@pascalduez pascalduez reopened this Dec 20, 2014

@HugoGiraudel

This comment has been minimized.

Show comment
Hide comment
@HugoGiraudel

HugoGiraudel Dec 20, 2014

Member

How comes the default theme (not RC) is using the new annotation names?

Member

HugoGiraudel commented Dec 20, 2014

How comes the default theme (not RC) is using the new annotation names?

pascalduez added a commit to SassDoc/sassdoc-extras that referenced this issue Dec 21, 2014

pascalduez added a commit to SassDoc/sassdoc-theme-default that referenced this issue Dec 21, 2014

pascalduez added a commit that referenced this issue Dec 21, 2014

@pascalduez pascalduez closed this Dec 21, 2014

@pascalduez

This comment has been minimized.

Show comment
Hide comment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment