search

Velron / doxygen-bootstrapped

Customize doxygen output to use the twitter bootstrap framework.
Apache License 2.0
401 stars 72 forks source link

Broken with doxygen 1.8.12 #20

Open xaqq opened 7 years ago

xaqq commented 7 years ago

Hello,

I believe this theme is broken when using doxygen 1.8.12. I've simply cloned the project and ran doxygen from example-site.

The generation seems fine, but going on the page gives a javascript error related to the search, and that prevent some other script to run. The result is that the whole page header ("main page", "related page", "namespace", etc) is not present.

Zeioth commented 7 years ago

Probably your doxygen is not recognizing doxy-boot.js. Have followed the instructions?

xaqq commented 7 years ago

Well I checked the instructions, but unless I'm mistaken there is nothing to do for the example project because the doxyfile is already correctly configured.

I just git clone, went to example-site folder and ran doxygen.

dvj commented 7 years ago

I'm experiencing the same issue as @xaqq on doxygen 1.8.12

mvcutter commented 7 years ago

I am also seeing the same issue, I verified that the generated output is finding doxy-boot.js

jowens commented 7 years ago

I'm having problems with my local installation with doxygen 1.8.13. Would love to see this updated to work with newer doxygens!

Velron commented 7 years ago

Looks like builds for doxygen 1.8.12+ are broken for this project right now.

The problem has to do with this line item in their changelog:

This is going to require some work to fix and am open to help on this. The commit that broke it can be found here:

https://github.com/doxygen/doxygen/commit/8480d35beef57ed08139b58972bfb83a3b37422c

At a first look it would seem that a new way of handling the nav is needed as well as a way to deal with smartmenus as its added after the custom html header and before the custom html footer.

cellcortex commented 7 years ago

I hacked together a first attempt on fixing this for newer doxygens. Still work in progress but at least the menus show and work. https://github.com/cellcortex/doxygen-bootstrapped

peperunas commented 7 years ago

I'm also having this problem. :-(

edgarriba commented 7 years ago

Any update for that?

cellcortex commented 7 years ago

I won't be putting more work into my fork above. @Velron do you want a pull request of that? I guess it will break support for older Doxygens and I don't plan to add support for both.

jowens commented 7 years ago

@Velron @cellcortex I'd certainly prefer support-for-more-recent-doxygen rather than support-for-less-recent-doxygen ...

Velron commented 7 years ago

@cellcortex Yes please, I made a feature branch for you to submit a pull request to so we don't break master. Thanks for the work getting this started! I'll need to think about it a little more but I don't see an easy way to support both so I'm leaning towards supporting 1.1.12+ and bumping the version to 2.0 once its working but I'm open to suggestions.

dskvr commented 7 years ago

Has this made progress?

jayjaybillings commented 7 years ago

The branch works great for me with only minor issues. Thanks!