Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 1 Sep 2005 23:07:28 +0300
From:      Giorgos Keramidas <keramida@freebsd.org>
To:        Daniel Gerzo <danger@rulez.sk>
Cc:        www@freebsd.org
Subject:   Re: query-pr-summary.cgi rework
Message-ID:  <20050901200727.GA40448@orion.daedalusnetworks.priv>
In-Reply-To: <1029392505.20050901194124@rulez.sk>
References:  <1029392505.20050901194124@rulez.sk>

next in thread | previous in thread | raw e-mail | index | archive | help
On 2005-09-01 19:41, Daniel Gerzo <danger@rulez.sk> wrote:
> Hello www,
>
>   I would like to announce my work to you and get some feedback. If
>   you feel that the patch is okay and doesn't need any more changes,
>   I would like to send PR and get it commited :-)
>
>   Changes to the query-pr-summary.cgi script include:
>
>   o) rewritten code so it's more readable now, replaced some " with '.
>   o) make the html output html valid.
>   o) the minor feature I've added is that the PRs in the html output
>      are now splitted according to the each PR's state and each state
>      has it's own color, so it's easier now to determinate the PR's
>      state on the web page. This has been done by adding a few css
>      classes, so if colors aren't meeting our needs, it is pretty
>      easy to modify them.
>
>   This work was already tested by remko@ and it seems to work. The
>   affects of my work are visible at for example:
>
>   http://people.freebsd.org/~remko/danger-test/query-pr-summary2.cgi?category=conf

This looks nice.  I like the use of the existing stylesheet for
coloring.  If Remko wants to commit it, there's no objection from me.




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20050901200727.GA40448>