Velron / doxygen-bootstrapped

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

Styles are now correctly displayed when 'inline_sources = TRUE' #22

Open Zeioth opened 7 years ago

Zeioth commented 7 years ago

The styles were incorrecly displayed in all sections showing source code when said option was enabled. I fixed it replacing the next line in doxy-boot.js

$( document ).ready(function() { /* code */ });

by

window.setTimeout(function() { /* code */ }));

Also I clarified the readme for an easier installation process. Tested and working.

Zeioth commented 7 years ago

Ok, ready to merge.

ghost commented 7 years ago

I tested your commits in my projects' docs, and your main – window.setTimeout – is causing a "flicker" when the style is transitioning from – without Bootstrap – to the new visual – with Bootstrap. Despite this is a valuable contribution, I believe this "flickering" behavior is undesirable, as it is very unconfortable for the final user.

Zeioth commented 7 years ago

You are right. The fix works perfectly on Firefox, but doesn't work at all in Chrome. I'll look for a better solution.