[Koha-bugs] [Bug 11430] Search history: Delete selected lines

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11430

--- Comment #54 from Jonathan Druart <jonathan.druart at biblibre.com> ---
(In reply to Owen Leonard from comment #53)
> Created attachment 29592 [details] [review]
> Note: This patch contains whitespace changes. Please diff accordingly.

Hi Owen,
What is the problem with the space alignment? There already are 2-space
indentation in templates (and actually I have been using 2 spaces for at least
2 years).
I had the discussion with Kyle and he uses 2 spaces too.
I am not sure we had this king of requirement in the coding guidelines for
template.
Since we regularly have a big amount of level in template, I think 2 spaces
will avoid long lines.

-- 
You are receiving this mail because:
You are watching all bug changes.