Firebug lite has a new version with lots of useful features. It can also be started from a bookmarklet from getfirebug.com with no code in the development environment. See http://getfirebug.com/lite.html for more details.
It would be good to simplify and update firebug module to use these new features. Some features, such as analysis of scripts, don't work unless firebug.js is on the same URI as the development server, so this drupal module still serves a purpose by making it easy to install firebug lite and enabling this feature.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 351967.patch | 3.01 KB | Bevan |
| #3 | firebug-lite-compressed.js_.txt | 58.96 KB | Bevan |
| #1 | firebug.zip | 26.5 KB | Bevan |
Comments
Comment #1
Bevan commentedI rewrote the module, deleted unnecessary cruft and buggy code, simplified and updated for latest version of firebug lite. Can I please be given commit rights so that I can commit and maintain this as DRUPAL-6--2-0?
Comment #2
Anonymous (not verified) commentedplease submit a patch instead of a zip.
Comment #3
Bevan commentedI zipped it since this is a rewrite of the module. In addition to application of the attached patch, firebug/ and firebug_controls.js have been removed. firebug-lite-compressed.js has been added. I propose this becomes 6--2-0.
The main loss with this rewrite is the firebug control. This is now part of firebug core. However I couldn't find how to minimize it by default. It also doesn't support conditional activation anymore (AFAICT), so it initiates even if firebug is open in the web browser. I would suggest submitting a patch to the firebug lite team to re-add this feature. The gains offset these losses dramatically.
Is there a standard practice or policy about whether compressed or uncompressed third-party javascript files should be used in contrib modules?
Comment #4
Anonymous (not verified) commentedno policy i believe. compressed should be fine.
cant review this till later.