[prev in list] [next in list] [prev in thread] [next in thread] 

List:       busybox
Subject:    Re: [PATCH 4/5] httpd_indexcgi.c: Remove new lines in HTML
From:       Xabier Oneca  --  xOneca <xoneca () gmail ! com>
Date:       2020-07-31 22:16:02
Message-ID: CACkgH70N7SrjONLap90eAWbxEexu4Hp3dtySLVKk=n2efybHiw () mail ! gmail ! com
[Download RAW message or body]

Hello Sergey,

El vie., 31 jul. 2020 a las 21:52, Sergey Ponomarev
(<stokito@gmail.com>) escribió:
> 
> Yes, it looks worse but you can see the resulting DOM tree instead \
> (elements/inspector). TBH as a web developer I almost never looked into raw source \
> and always watch the DOM tree. 
> On Fri, Jul 31, 2020 at 19:29 Denys Vlasenko <vda.linux@googlemail.com> wrote:
> > 
> > On Fri, Jul 10, 2020 at 9:50 PM Sergey Ponomarev <stokito@gmail.com> wrote:
> > > 
> > > Nobody sees them, source code is here, resulted DOM can be seen in web \
> > > inspector
> > 
> > I do look at it in "View page source", and it looks much worse when
> > all newlines are removed.

I personally prefer well formatted HTML. But, this being the output of
an embedded/small application, and it already lacks some indenting, I
think it's ok if the output is as compact as it can be. If I need (for
some reason) to see something in the source code, I can use the DOM
inspector, or copy to an editor and add newlines to visualize it.

But one drawback I see changing the source now is if someone is
grepping it. The changes may break someone's hack.

Just my opinion. Cheers,

Xabier Oneca_,,_
_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox


[prev in list] [next in list] [prev in thread] [next in thread]