Mailing List Archive: MythTV: Dev

[PATCH] increase mythweb contrast

Index | Next | Previous | View Threaded

rtsai1111 at comcast

May 4, 2005, 6:01 PM

Post #1 of 8 (988 views)
Permalink

[PATCH] increase mythweb contrast

In the "JobQueue" status area in mythweb, the colored text is very
difficult to read against the dark blue background. Would people
object to putting in brighter shades of green and red, instead of the
current shades of dark red and dark green?

I've attached patches for both mythweb and the HTTP-generating bits of
mythbackend.

--Rob

Attachments:

mythtv.diff (0.95 KB)

mythweb.diff (0.69 KB)

cpinkham at bc2va

May 4, 2005, 8:13 PM

Post #2 of 8 (986 views)
Permalink

Re: [PATCH] increase mythweb contrast [In reply to]

> In the "JobQueue" status area in mythweb, the colored text is very
> difficult to read against the dark blue background. Would people
> object to putting in brighter shades of green and red, instead of the
> current shades of dark red and dark green?

I don't mind the colors being brightened a little, I just picked
all-green, all-red, etc. when I added the JobQueue status to the
backend status page.

Haven't actually applied this to see the colors, but won't the 'white'
make the finished status invisible on the mythbackend status page?

> + << " .jobfinished { color: white; }\r\n"

Isn't the background of the backend status page pretty close to white?

--
Chris

rtsai1111 at comcast

May 4, 2005, 9:13 PM

Post #3 of 8 (958 views)
Permalink

Re: [PATCH] increase mythweb contrast [In reply to]

On Wed, May 04, 2005 at 11:13:17PM -0400, Chris Pinkham wrote:
> > In the "JobQueue" status area in mythweb, the colored text is very
> > difficult to read against the dark blue background. Would people
> > object to putting in brighter shades of green and red, instead of the
> > current shades of dark red and dark green?
>
> I don't mind the colors being brightened a little, I just picked
> all-green, all-red, etc. when I added the JobQueue status to the
> backend status page.
>
> Haven't actually applied this to see the colors, but won't the 'white'
> make the finished status invisible on the mythbackend status page?
>
> > + << " .jobfinished { color: white; }\r\n"
>
> Isn't the background of the backend status page pretty close to white?

Are we talking about the same thing?

I'm talking about:

http://my-mythbox/mythweb/status.php

which is a dark blue background, against which it is difficult to see
the current shades of red/green.

--Rob

Attachments:

signature.asc (0.18 KB)

cpinkham at bc2va

May 4, 2005, 9:23 PM

Post #4 of 8 (989 views)
Permalink

Re: [PATCH] increase mythweb contrast [In reply to]

> > Haven't actually applied this to see the colors, but won't the 'white'
> > make the finished status invisible on the mythbackend status page?
> >
> > > + << " .jobfinished { color: white; }\r\n"
> >
> > Isn't the background of the backend status page pretty close to white?
>
> Are we talking about the same thing?

Yeah, mostly. The httpstatus.cpp file that you modified isn't there
for MythWeb, even though it is used by it. It is there so that you
can point a web browser to the following:

http://master-backend-ip:6544/

That's the backend status webpage. The background for that page is
white, so MythWeb must be wrapping the content if it makes that
information appear on a blue background.

--
Chris

lists at forevermore

May 4, 2005, 9:58 PM

Post #5 of 8 (983 views)
Permalink

Re: [PATCH] increase mythweb contrast [In reply to]

> That's the backend status webpage. The background for that page is
> white, so MythWeb must be wrapping the content if it makes that
> information appear on a blue background.

You are correct. I strip out the css info from that page and rewrite it
so that it matches the rest of mythweb.

Unfortunately, I never have any jobs in my queue list, so I never got
around to adjusting their colors for mythweb.

Rob, can you send me a screenshot of what your color patch looks like?

-Chris
_______________________________________________
mythtv-dev mailing list
mythtv-dev [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev

rtsai1111 at comcast

May 5, 2005, 6:20 AM

Post #6 of 8 (964 views)
Permalink

Re: [PATCH] increase mythweb contrast [In reply to]

On Thu, May 05, 2005 at 12:23:48AM -0400, Chris Pinkham wrote:
> > > Haven't actually applied this to see the colors, but won't the
> > > 'white' make the finished status invisible on the mythbackend
> > > status page?
> > >
> > > > + << " .jobfinished { color: white; }\r\n"
> > >
> > > Isn't the background of the backend status page pretty close to
> > > white?
> >
> > Are we talking about the same thing?
>
> Yeah, mostly. The httpstatus.cpp file that you modified isn't there
> for MythWeb, even though it is used by it. It is there so that you
> can point a web browser to the following:
>
> http://master-backend-ip:6544/
>
> That's the backend status webpage. The background for that page is
> white, so MythWeb must be wrapping the content if it makes that
> information appear on a blue background.

Nah, I'm busted. I had patched my mythweb, then just did a "find|xargs
grep" looking for any other instances of CSS colorings I had missed,
but was apparently too overzealous. I forgot about the :6544 page. So
yeah, I really only care about the mythweb page.

I'll work on getting a screenshot (nothing in queue right now).

--Rob

Attachments:

signature.asc (0.18 KB)

rtsai1111 at comcast

May 5, 2005, 10:44 AM

Post #7 of 8 (941 views)
Permalink

Re: [PATCH] increase mythweb contrast [In reply to]

On Wed, May 04, 2005 at 09:58:08PM -0700, Chris Petersen wrote:
> >That's the backend status webpage. The background for that page is
> >white, so MythWeb must be wrapping the content if it makes that
> >information appear on a blue background.
>
> You are correct. I strip out the css info from that page and
> rewrite it so that it matches the rest of mythweb.
>
> Unfortunately, I never have any jobs in my queue list, so I never
> got around to adjusting their colors for mythweb.
>
> Rob, can you send me a screenshot of what your color patch looks
> like?

Not an actual screen shot, but I extracted the relevant bits of HTML
so you can see a comparison of the text contrast. Will that be good
enough?

As for my patch, please ignore my changes to httpstatus.cpp, and only
consider the ones for status.css; as Chris (Pinkham) said, the
background for the httpstatus.cpp (port 6544) page is white, so the
text should remain dark.

I've reattached a patch (same one as last time), containing only
status.css changes.

Thanks,
--Rob

Attachments:

mythweb.html (1.11 KB)

mythweb.diff (0.69 KB)

signature.asc (0.18 KB)

lists at forevermore

May 13, 2005, 6:38 PM

Post #8 of 8 (936 views)
Permalink

Re: [PATCH] increase mythweb contrast [In reply to]

applied, finally...

-Chris
_______________________________________________
mythtv-dev mailing list
mythtv-dev [at] mythtv
http://mythtv.org/cgi-bin/mailman/listinfo/mythtv-dev

Index | Next | Previous | View Threaded

Interested in having your list archived? Contact Gossamer Threads