Bug#680505: desktop-base: Text has too low contrast with new Grub background

2012-07-27 Thread Paul Tagliamonte
tags 680505 + pending
thanks

Hey folks,

Here's the diff of what I pushed to svn:

| Index: debian/changelog
| ===
| --- debian/changelog(revision 298)
| +++ debian/changelog(revision 297)
| @@ -9,8 +9,6 @@
|* Add ${misc:Depends} to the Depends for the binary package. We're using
|  debhelper, so we should behave nicely.
|* Updated GDM  KDM's login background to have branding in the lower-right.
| -  * Inverting the GRUB highlight colors. This helps those who might be hard
| -of sight, or think it's generally hard to see. (Closes: #680505)
|
|   -- Paul Tagliamonte paul...@debian.org  Mon, 09 Jul 2012 20:24:40 -0400
|
| Index: grub/grub_background.sh
| ===
| --- grub/grub_background.sh (revision 298)
| +++ grub/grub_background.sh (revision 297)
| @@ -1,3 +1,3 @@
|  WALLPAPER=/usr/share/images/desktop-base/desktop-grub.png
| -COLOR_NORMAL=white/black
| -COLOR_HIGHLIGHT=black/white
| +COLOR_NORMAL=light-gray/black
| +COLOR_HIGHLIGHT=white/black

It's a bit more classic grub, but I think we're going for more classic this
cycle anyway.

Thanks, everyone!
  Paul

-- 
 .''`.  Paul Tagliamonte paul...@debian.org
: :'  : Proud Debian Developer
`. `'`  4096R / 8F04 9AD8 2C92 066C 7352  D28A 7B58 5B30 807C 2A87
 `- http://people.debian.org/~paultag


signature.asc
Description: Digital signature


Bug#680505: desktop-base: Text has too low contrast with new Grub background

2012-07-10 Thread Fabian Greffrath



The background image looks nice in Grub2, but the text color really has
to be changed to something with better contrast with respect to the
background colors!


I'd vote for magenta as color for the highlighted text. I think it 
also was like this when text-on-background was first introduced in 
GRUB and magenta comes most close to the color of the Debian swirl.


 - Fabian



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#680505: desktop-base: Text has too low contrast with new Grub background

2012-07-10 Thread Fabian Greffrath

Am 10.07.2012 11:21, schrieb Fabian Greffrath:

I'd vote for magenta as color for the highlighted text. I think it
also was like this when text-on-background was first introduced in
GRUB and magenta comes most close to the color of the Debian swirl.


IMHO, this looks just great and provides enough contrast against the 
background image:


--- a/usr/share/desktop-base/grub_background.sh.orig
+++ b/usr/share/desktop-base/grub_background.sh
@@ -1,3 +1,3 @@
 WALLPAPER=/usr/share/images/desktop-base/desktop-grub.png
-COLOR_NORMAL=light-gray/black
-COLOR_HIGHLIGHT=white/black
+COLOR_NORMAL=white/black
+COLOR_HIGHLIGHT=light-magenta/black

ALternatively, one could use black text on a magenta (not necessarily 
light-magenta) background, which would even more distinguish selected 
from non-selected menu entries.


 - Fabian



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#680505: desktop-base: Text has too low contrast with new Grub background

2012-07-10 Thread Paul Tagliamonte
On Tue, Jul 10, 2012 at 11:41:17AM +0200, Fabian Greffrath wrote:
 Am 10.07.2012 11:21, schrieb Fabian Greffrath:
 I'd vote for magenta as color for the highlighted text. I think it
 also was like this when text-on-background was first introduced in
 GRUB and magenta comes most close to the color of the Debian swirl.
 
 IMHO, this looks just great and provides enough contrast against the
 background image:
 
 --- a/usr/share/desktop-base/grub_background.sh.orig
 +++ b/usr/share/desktop-base/grub_background.sh
 @@ -1,3 +1,3 @@
  WALLPAPER=/usr/share/images/desktop-base/desktop-grub.png
 -COLOR_NORMAL=light-gray/black
 -COLOR_HIGHLIGHT=white/black
 +COLOR_NORMAL=white/black
 +COLOR_HIGHLIGHT=light-magenta/black

Thanks for the patch! :)

 
 ALternatively, one could use black text on a magenta (not
 necessarily light-magenta) background, which would even more
 distinguish selected from non-selected menu entries.
 
  - Fabian

Thanks for your input :)

However, the swirl isn't Magenta -- it's white, and a red seems out of
place on a blue / white color theme.

Let's see if we can't find something that matches the color scheme :)

-Paul

-- 
 .''`.  Paul Tagliamonte paul...@debian.org
: :'  : Proud Debian Developer
`. `'`  4096R / 8F04 9AD8 2C92 066C 7352  D28A 7B58 5B30 807C 2A87
 `- http://people.debian.org/~paultag


signature.asc
Description: Digital signature


Bug#680505: desktop-base: Text has too low contrast with new Grub background

2012-07-10 Thread Fabian Greffrath

Am 10.07.2012 13:04, schrieb Paul Tagliamonte:

However, the swirl isn't Magenta -- it's white, and a red seems out of
place on a blue / white color theme.


The *original* Debian swirl is (nearly) magenta, so we could make the 
highlighted menu entry an innuendo to its original color.



Let's see if we can't find something that matches the color scheme :)


Still on the magenta trail, I found those two quite appealing as well:

(dark)
COLOR_NORMAL=white/black
COLOR_HIGHLIGHT=black/magenta

(light)
COLOR_NORMAL=white/black
COLOR_HIGHLIGHT=white/light-magenta

But I'll leave the final hard decision up to you. ;)

 - Fabian



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#680505: desktop-base: Text has too low contrast with new Grub background

2012-07-07 Thread Francesco Poli
On Fri, 6 Jul 2012 09:35:59 -0400 Paul Tagliamonte wrote:

 On Fri, Jul 06, 2012 at 03:46:16PM +0300, Touko Korpela wrote:
  Package: desktop-base
  Version: 7.0.0
  Severity: normal
  
  With new light background image, white Grub text has too low contrast making
  text hard to read.
 
 I ACK this problem.

I also noticed the same issue, as soon as I rebooted after upgrading to
desktop-base/7.0.0 .

The background image looks nice in Grub2, but the text color really has
to be changed to something with better contrast with respect to the
background colors!

 
  Also image brightness varies in different part of the image.
  Consider making image use more homogenous brightness when text is printed
  over it and if using light colour background make text colour darker.
 
 This is the style the author of the theme has gone with. I'm sure we can
 find a color that works without destorying his vision for the work.

I really hope so.

And I hope this fix can get a freeze exception: by the way, I think
that normal severity is too low for this bug report, I would raise it
to at least important...


-- 
 http://www.inventati.org/frx/frx-gpg-key-transition-2010.txt
 New GnuPG key, see the transition document!
. Francesco Poli .
 GnuPG key fpr == CA01 1147 9CD2 EFDF FB82  3925 3E1C 27E1 1F69 BFFE


pgp9V3Dtgchnt.pgp
Description: PGP signature


Bug#680505: desktop-base: Text has too low contrast with new Grub background

2012-07-07 Thread Paul Tagliamonte
On Sat, Jul 07, 2012 at 05:43:22PM +0200, Francesco Poli wrote:
 On Fri, 6 Jul 2012 09:35:59 -0400 Paul Tagliamonte wrote:
 
  On Fri, Jul 06, 2012 at 03:46:16PM +0300, Touko Korpela wrote:
   Package: desktop-base
   Version: 7.0.0
   Severity: normal
   
   With new light background image, white Grub text has too low contrast 
   making
   text hard to read.
  
  I ACK this problem.
 
 I also noticed the same issue, as soon as I rebooted after upgrading to
 desktop-base/7.0.0 .
 
 The background image looks nice in Grub2, but the text color really has
 to be changed to something with better contrast with respect to the
 background colors!
 
  
   Also image brightness varies in different part of the image.
   Consider making image use more homogenous brightness when text is printed
   over it and if using light colour background make text colour darker.
  
  This is the style the author of the theme has gone with. I'm sure we can
  find a color that works without destorying his vision for the work.
 
 I really hope so.
 
 And I hope this fix can get a freeze exception: by the way, I think
 that normal severity is too low for this bug report, I would raise it
 to at least important...

The release team will look at the bugs, artificially pushing up the
severity assumes the release-team won't take a critical look at what's
changed.

It's early in the cycle and this isn't a vanity package. We should have
no problem, assuming they are bugfixes.

 
 
 -- 
  http://www.inventati.org/frx/frx-gpg-key-transition-2010.txt
  New GnuPG key, see the transition document!
 . Francesco Poli .
  GnuPG key fpr == CA01 1147 9CD2 EFDF FB82  3925 3E1C 27E1 1F69 BFFE

cheers,
 Paul

-- 
 .''`.  Paul Tagliamonte paul...@debian.org
: :'  : Proud Debian Developer
`. `'`  4096R / 8F04 9AD8 2C92 066C 7352  D28A 7B58 5B30 807C 2A87
 `- http://people.debian.org/~paultag


signature.asc
Description: Digital signature


Bug#680505: desktop-base: Text has too low contrast with new Grub background

2012-07-07 Thread Francesco Poli
On Sat, 7 Jul 2012 11:53:31 -0400 Paul Tagliamonte wrote:

 On Sat, Jul 07, 2012 at 05:43:22PM +0200, Francesco Poli wrote:
[...]
  And I hope this fix can get a freeze exception: by the way, I think
  that normal severity is too low for this bug report, I would raise it
  to at least important...
 
 The release team will look at the bugs,

Sure.

 artificially pushing up the
 severity assumes the release-team won't take a critical look at what's
 changed.

What I meant was not suggesting to artificially raise the severity:
I really think that the correct severity for this bug is important,
since it does have a major effect on the usability of the package.

However, I am not the original bug report submitter, nor a maintainer
of this package: I will leave to you to decide whether the severity
should be changed...

 
 It's early in the cycle and this isn't a vanity package. We should have
 no problem, assuming they are bugfixes.

Good!

Thanks a lot for your prompt reply.
Bye.


-- 
 http://www.inventati.org/frx/frx-gpg-key-transition-2010.txt
 New GnuPG key, see the transition document!
. Francesco Poli .
 GnuPG key fpr == CA01 1147 9CD2 EFDF FB82  3925 3E1C 27E1 1F69 BFFE


pgp3jfuf0pJbB.pgp
Description: PGP signature


Bug#680505: desktop-base: Text has too low contrast with new Grub background

2012-07-06 Thread Touko Korpela
Package: desktop-base
Version: 7.0.0
Severity: normal

With new light background image, white Grub text has too low contrast making
text hard to read.
Also image brightness varies in different part of the image.
Consider making image use more homogenous brightness when text is printed
over it and if using light colour background make text colour darker.


-- System Information:
Debian Release: wheezy/sid
  APT prefers testing
  APT policy: (700, 'testing'), (600, 'unstable'), (500, 'experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 3.2.0-2-amd64 (SMP w/2 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages desktop-base depends on:
ii  dpkg 1.16.4.3
ii  librsvg2-common  2.36.1-1

desktop-base recommends no packages.

Versions of packages desktop-base suggests:
ii  xfce4  4.8.0.3

-- no debconf information



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#680505: desktop-base: Text has too low contrast with new Grub background

2012-07-06 Thread Paul Tagliamonte
On Fri, Jul 06, 2012 at 03:46:16PM +0300, Touko Korpela wrote:
 Package: desktop-base
 Version: 7.0.0
 Severity: normal
 
 With new light background image, white Grub text has too low contrast making
 text hard to read.

I ACK this problem.

 Also image brightness varies in different part of the image.
 Consider making image use more homogenous brightness when text is printed
 over it and if using light colour background make text colour darker.

This is the style the author of the theme has gone with. I'm sure we can
find a color that works without destorying his vision for the work.

 
 
 -- System Information:
 Debian Release: wheezy/sid
   APT prefers testing
   APT policy: (700, 'testing'), (600, 'unstable'), (500, 'experimental')
 Architecture: amd64 (x86_64)
 
 Kernel: Linux 3.2.0-2-amd64 (SMP w/2 CPU cores)
 Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
 Shell: /bin/sh linked to /bin/dash
 
 Versions of packages desktop-base depends on:
 ii  dpkg 1.16.4.3
 ii  librsvg2-common  2.36.1-1
 
 desktop-base recommends no packages.
 
 Versions of packages desktop-base suggests:
 ii  xfce4  4.8.0.3
 
 -- no debconf information
 
 
 

I thank you for the report!
 Paul

-- 
 .''`.  Paul Tagliamonte paul...@debian.org
: :'  : Proud Debian Developer
`. `'`  4096R / 8F04 9AD8 2C92 066C 7352  D28A 7B58 5B30 807C 2A87
 `- http://people.debian.org/~paultag


signature.asc
Description: Digital signature


Bug#680505: desktop-base: Text has too low contrast with new Grub background

2012-07-06 Thread Nicolas Dandrimont
Hi,

Le 06/07/2012 à 16:48, Paul Tagliamonte paul...@debian.org écrivit :
 Also image brightness varies in different part of the image.
 Consider making image use more homogenous brightness when text is 
 printed
 over it and if using light colour background make text colour darker.

This is the style the author of the theme has gone with. I'm sure we can
find a color that works without destorying his vision for the work.
   
   I'm concerned about this because I wouldn't want to see the theme
   changed. I hope higher contrast can be achieved for those who wish it by
   simply changing the text color.
  
  I'll see what color can be used. If no one is suitable, I may render a 
  darker
  background for grub.
 
 Let's get s'more opinions. I didn't really have a problem with it, and I
 didn't notice until the reporter pointed it out, so let's get s'more
 opionons before we go and do that :)

I agree the text to background contrast is too low, and I have a hard
time seeing the grub countdown.

   
   It's interesting that you notice it, too. I wonder if the type of
   display (CRT vs LCD or even type of LCD) has something to do with it.
   Looks gorgeous on all the LCDs here.

I have found the contrast to be too low on both the screens I saw it on:
my Thinkpad x61s (laptop, 1024x768), and my desktop (it's a 24 inch
iiyama LCD screen, and I'm not quite certain what is the resolution in
early boot. It's certainly not the native 1920x1080 though).

Thanks for looking into this issue!

Cheers,
-- 
Nicolas Dandrimont



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org