On 20 Apr 2005, at 14:38, Jim Riggs wrote:
Although I like this module a lot, if it is going to be in contrib, it has to behave like a contrib module. That is, it should not patch drupal.css, but use its own css file inside the module's directory. And, it should not have a name that conflicts with a core module. This will cause a bunch of problems and support issues as (non-developer) users stumble across it and try it. If you want to leave it as is (patching core and conflicting with a core module), it should stay in your sandbox.
I suggest making it a true contrib module by changing the name of the cvs directory/module and accessing the css from the module's directory. This way people can use it until it -- hopefully -- makes it into core, replacing archive.
The module is meant for core, and therefore belongs in a sandbox (IMO). Once ready for inclusion, please provide a patch and upload it to the patch queue. In the mean time, everyone is encourage to review/discuss the poroposed changes. -- Dries Buytaert :: http://www.buytaert.net/