[phpesp-dev] Re: phpESP Survey Themes

[Matthew G. <gr...@mu...>]

I've only had time to take a quick look at the patch this weekend, but
it looks darn good!  I'll do some deeper digging next week.

What I've noticed so far:
- "/css" should probably be "/public/css" by default

- Maybe think about moving your CSS additions in phpESP.ini,
  so that they will be grouped together with the other file/http
  paths.

- Maybe include the new files and directories in the patch instead of 
  of having to manually create/copy them/ 

- When you build your patch make sure the phpESP.ini and the other
  places paths are configured(manage.php, handler.php, etc..) are
  unchanged from CVS, except for your new CSS additions of course.
 
This is gonna be cool 8)


On a side note...
Since I sit on Open Projects(http://www.openprojects.net/)
most all the time anyhow, I decided to register #phpesp and sit on
that as well.  If anyone wants to drop by and talk about devel issues,
a problem with phpESP, etc.  I'll be lurking.  



On Sat, Jun 08, 2002 at 05:37:50PM +1000, Kon Angelopoulos wrote:
> Hi Guys,
> 
> 	It's finally here.
> 	Attached you should find a phpThemes.tar.gz file. This file contains a 
> directory called phpESP_addons which further contains the following:
...Snip...
> 
> Kon


-- 
brought to you by, Matthew Gregg...
one of the friendly folks in the IT Lab.
--------------------------------------\
The IT Lab (http://www.itlab.musc.edu) \____________________
Probably the world's premier software development center.
Serving: Programming, Tools, Ice Cream, Seminars