On Fri 25 Jun 2010 10:19 -0400, Denis Kobozev wrote:

> diff --git a/web/html/css/archnavbar/archnavbar.css 
> b/web/html/css/archnavbar/archnavbar.css
> new file mode 100644
> index 0000000..fbdffbd
> --- /dev/null
> +++ b/web/html/css/archnavbar/archnavbar.css
> @@ -0,0 +1,41 @@
...
> +/* archdev navbar */
> +div#archdev-navbar { margin: 1.5em 0; }

I think this should be pushed in to line up with the rest of the page.
margin: 1.5em 25px

> diff --git a/web/lib/aur.inc b/web/lib/aur.inc
> index 37f18f8..3c89cb5 100644
> --- a/web/lib/aur.inc
> +++ b/web/lib/aur.inc
> @@ -422,6 +422,14 @@ function uid_from_email($email="")
>       return $row[0];
>  }
>  
> +# check user privileges
> +#
> +function is_tu()
> +{
> +     $type = account_from_sid($_COOKIE['AURSID']);
> +     return ($type == 'Trusted User' || $type == 'Developer');
> +}

The name of this function is misleading. Maybe we could call it
check_user_privileges?

Other than those small things it looks like some awesome work.
Thanks for your submission!

If there are no objections can take care of those changes.

Reply via email to