Hello!

On Wed, Sep 19, 2018 at 10:41:23AM -0500, Nova wrote:

> TZAG,
>
> I've run into a compatability issue between Nginx error/special response
> pages and customized font color and background color in
> user stylesheets. I use white on black as my default color scheme, and
> since the Nginx special pages specify bgcolor but not color
> (font color), this leads to a "blank" white page (with a hr on it). The
> attached patch presented for review removes Nginx bgcolor
> on special pages, since this is not necessary to render them as intended
> without interfering with accessibility settings.
>
> Best,
>
> Nova

> # HG changeset patch
> # User Nova DasSarma <nova@novalinium.com>
> # Date 1537367207 18000
> # Wed Sep 19 09:26:47 2018 -0500
> # Node ID f359ea07f1d61c284fb54706c881120c3e4628c0
> # Parent 87d2ea860f380dc8418c97c0163412f53c2d008e
> HTTP Special Responses: removed bgcolor attribute on body.
>
> bgcolor overrides compatibility settings in browsers and leads to undesirable behavior when the default font color is set to white in the browser, since font-color is not also overridden.

[...]

Thanks for the patch.

I've extended it on the autoindex module where the same problem
occurs, and committed it with some minor changes to commit log:

http://hg.nginx.org/nginx/rev/170922952477

--
Maxim Dounin
http://mdounin.ru/
_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel