Skip to content

Refactoring #6

@chrisgo

Description

@chrisgo

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:

  • 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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions