Forum List Message List New Topic Print View

Adrian Perez

February 21, 2011 07:40AM

Hello to all,

On Mon, 21 Feb 2011 14:00:04 +0300
Maxim Dounin <mdounin@mdounin.ru> wrote:

> 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.
>
> Please note that there is also other issues with you patches, you
> may want spend some more time on them before posting again.

As an alternative, instead of hijacking the autoindex module, you could
use the Fancy Indexes [1] module, and avoid duplicating effort. The
following is actually possible:

location / {
fancyindex on;
fancyindex_css_href /path/to/your.css;
}

As a bonus, the module outputs listings as tabled, using proper
XHTML 1.0 Strict.

Cheers,

---
[1] http://gitorious.org/ngx-fancyindex
--
-Adrian
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://nginx.org/mailman/listinfo/nginx-devel

Reply Quote

RSS

Subject	Author	Views	Posted
[PATCH] Add autoindex_css_file option	Clément Bœsch	4044	February 20, 2011 12:04PM
Re: [PATCH] Add autoindex_css_file option	Clément Bœsch	953	February 20, 2011 12:10PM
Re: [PATCH] Add autoindex_css_file option	Maxim Dounin	931	February 20, 2011 01:26PM
Re: [PATCH] Add autoindex_css_file option	Clément Bœsch	878	February 20, 2011 03:48PM
Re: [PATCH] Add autoindex_css_file option	Maxim Dounin	911	February 20, 2011 07:10PM
Re: [PATCH] Add autoindex_css_file option	Clément Bœsch	969	February 20, 2011 07:22PM
Re: [PATCH] Add autoindex_css_file option	Maxim Dounin	1072	February 21, 2011 06:02AM
Re: [PATCH] Add autoindex_css_file option	Adrian Perez	982	February 21, 2011 07:40AM
Re: [PATCH] Add autoindex_css_file option	Clément Bœsch	921	February 23, 2011 04:22AM
Re: [PATCH] Add autoindex_css_file option	Clément Bœsch	1187	March 03, 2011 07:58PM

Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 202

Record Number of Users: 8 on April 13, 2023

Record Number of Guests: 421 on December 02, 2018