http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8202
--- Comment #7 from Jared Camins-Esakov <jcam...@cpbibliography.com> --- (In reply to comment #5) > QA comments: > > Hi Jared, > > Your modifications seem good and they are close to mine (for export.pl). > About your patch, I think you can repropose it modifying the indentation of > export.tt (ex. > + <ol><li> > > + </li></ol> > > +[% ELSE %] > > +<div id="exporttype" class="toptabs"> > +<ul> > ) > > I agree with you, I quibble... I would be happy to submit a follow-up fixing indentation. I was very tempted to reindent export.tt, actually, but section 1.3 in the coding guidelines says don't refactor needlessly, and don't spend time changing someone else's style to yours. :) -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/