Update of patch #5387 (project phpgroupware):
Status: Ready For Test => In Progress
Assigned to: ceb => Caeies
_______________________________________________________
Follow-up Comment #1:
This looks good.
A few suggestions
Move the diff CSS to filemanage/templates/base/css/base.css and just use
classes, there is no need to allow it to only be applied to td elements
Fix up the phpdocs for new SVN classes
Split the svn parser off into a new class. If it is 3rd party code put it
into phpgwapi/inc/svnparser and just include it in the top of the svn_dav
class
Feel free to commit it
_______________________________________________________
Reply to this item at:
<http://savannah.gnu.org/patch/?5387>
_______________________________________________
Message sent via/by Savannah
http://savannah.gnu.org/
_______________________________________________
phpGroupWare-tracker mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/phpgroupware-tracker
