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

adding filename to onHandleHTML() plugin event #175

Merged
merged 1 commit into from
Feb 6, 2016

Conversation

skratchdot
Copy link
Contributor

I am working on a plugin, and would like to modify the html based on the file that is being generated.

I've added a simple test case, and confirmed that the fileName is being passed in the ev.data.

This commit allows me to write plugins that do things like removing the navigation on certain pages, or adding 'active' links in the header based on a regex.

It would be nice to pass this value in all plugin events, but I really want it onHandleHTML().

Docs haven't been updated.

Version has been tagged and bumped to 0.4.4.

@skratchdot
Copy link
Contributor Author

@h13i32maru - If you pull this in, can you please run npm publish so I can pull 0.4.4?

h13i32maru added a commit that referenced this pull request Feb 6, 2016
adding filename to onHandleHTML() plugin event
@h13i32maru h13i32maru merged commit 0b887a7 into esdoc:master Feb 6, 2016
@h13i32maru
Copy link
Member

Thanks for this PR! But the version number in package.json is handled by me.

h13i32maru added a commit that referenced this pull request Feb 6, 2016
@h13i32maru
Copy link
Member

I released v0.4.4

@skratchdot
Copy link
Contributor Author

Thanks @h13i32maru . I'm using your tool to build these docs: http://projects.skratchdot.com/stats-collector/

I'm using onHandleHTML() to add content to the header/footer, and add links to the "extends" output:

https://github.com/skratchdot/stats-collector/blob/master/esdoc/plugin.js

but now I can do some other things based on the page a person is on. Thanks again!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants