[PATCH] Add autoindex_css_file option

Wed Feb 23 12:20:23 MSK 2011

On Mon, Feb 21, 2011 at 02:00:04PM +0300, Maxim Dounin wrote:
> Hello!
> 
> On Mon, Feb 21, 2011 at 01:20:27AM +0100, Clément Bœsch wrote:
> 
> > On Mon, Feb 21, 2011 at 03:08:18AM +0300, Maxim Dounin wrote:
> > > Hello!
> > > 
> > > On Sun, Feb 20, 2011 at 09:47:37PM +0100, Clément Bœsch wrote:
> > > 
> > > [...]
> > > 
> > > >  static u_char title[] =
> > > > -"<html>" CRLF
> > > > +"<!DOCTYPE html>" CRLF
> > > >  "<head><title>Index of "
> > > >  ;
> > > 
> > > The result isn't html at all.
> > > 
> > 
> > Well it's HTML5…
> 
> No it isn't.  You dropped html element.
> 

My bad, it didn't look mandatory for the w3c validator: "<!doctype
html><head><title></title></head></html>" seems to be the minimum, but
I'll definitely update the patch.

> Please note that there is also other issues with you patches, you 
> may want spend some more time on them before posting again.
> 

Could you be a bit more explicit? What are the issues?

Regards,

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://nginx.org/pipermail/nginx-devel/attachments/20110223/5c69e668/attachment.pgp>