You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I've taken your code and refactored it to do a couple of things (not sure if I broke anything though) and it's fairly major so I'm not sure if I should do a fork + pull request. Here are the changes I have made:
Renamed all filenames and PHP classes to debugbar (one word) and not have debug-toolbar or toolbar variants
Made it more HMVC (cascading) so the main code is now in a subdir called "kohana" and the base class "Debugbar" is blank (extends Kohana_Debugbar)
Created a new controller Kohana_Controller_Debugbar (also a blank class Controller_Debugbar) that follows the same convention as the userguide in terms of pulling up "media" files (js, css and img) - they are created with Route:set() in init.php
Moved all the css, js and img directories under a subfolder called media and using route to serve them up (so it's more portable) and renamed/moved css and js files to media/css/debugbar.css, media/js/debugbar.js - no need to copy images to DOCROOT (same as userguide)
Made everything output right before the </body> tag - used to have CSS output inline right before the tag which muddled up the View Source (scroll way down to see your code) -- now the JS & CSS are NOT inline and are sent to the output right before the debugbar HTML
TODO
Figure out how to enable this without touching index.php -- possibly somewhere in the after() of your base controller
CSS and JS cleanup to make everything be "debugbar" vs a mix of debug-toolbar, toolbar, etc.
What do you think?
Thanks,
Chris
The text was updated successfully, but these errors were encountered:
Hi,
I've taken your code and refactored it to do a couple of things (not sure if I broke anything though) and it's fairly major so I'm not sure if I should do a fork + pull request. Here are the changes I have made:
TODO
What do you think?
Thanks,
Chris
The text was updated successfully, but these errors were encountered: