Hi David,

On Fri, Apr 24, 2009 at 2:21 PM, David Goldfein
<[email protected]> wrote:
> Reworked ViewLog.pl and the template.  Eliminated html errors, cleaned up
> the template, change mime type to text/csv and other minor changes.

I have pushed this patch.  Since this is your first commit accepted
into the main tree, let me welcome you to the Koha committer cabal!

A couple comments about this patch:

* my XHTML validator says that there's an extra </div> - could you
chase that down?
* as I mentioned before, it would be nice if the form remembered the
modules that were selected and and displayed the mutliselect control
accordingly.

Regards,

Galen
-- 
Galen Charlton
VP, Research & Development, LibLime
[email protected]
p: 1-888-564-2457 x709
skype: gmcharlt
_______________________________________________
Koha-patches mailing list
[email protected]
http://lists.koha.org/mailman/listinfo/koha-patches

Reply via email to