Re: Sidebar and Dynamic News

On Tue, Aug 30, 2011 at 17:32, Stewart Lord <stewey@ambitious.ca> wrote: > > On 2011-08-25, at 5:15 AM, Paul Dragoonis wrote: > > On Thu, Aug 25, 2011 at 1:14 PM, Paul Dragoonis <dragoonis@gmail.com> wrote: >> >> I'm back, I've fixed my editor to show whitespace and tab/spaces. I >> converted all my added code to spaces for indentation instead of tabs. >> I've taken Hannes' feedback into consideration and fixed up the view files >> so instead of 5 there is only 1, which I do like more now. >> My patch is attached, if anyone has no problems with it i'm happy to >> commit and move onto something else. > > Forgot the attachment, duh! - here it is now. > >> >> Cheers, >> Paul. > > Hannes did you get a chance to review this new version? Well, I completely disagree with the whole print_view() concept. The onlything it does is add extra step and confusion when debugging. But since its his implementation its eventually his call. I thought this was committed already though. -Hannes