All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Vladimir 'φ-coder/phcoder' Serbinenko" <phcoder@gmail.com>
To: grub-devel@gnu.org
Subject: Re: Bug#575252: grub-pc: black frame around the background image
Date: Thu, 15 Jul 2010 22:33:24 +0200	[thread overview]
Message-ID: <4C3F7094.30503@gmail.com> (raw)
In-Reply-To: <20100706171959.GA1165@riva.ucam.org>

[-- Attachment #1: Type: text/plain, Size: 1837 bytes --]

On 07/06/2010 07:19 PM, Colin Watson wrote:
> On Wed, Mar 24, 2010 at 03:23:20PM +0100, Francesco Muzio wrote:
>   
>> The last uploaded version of grub2 behaves bad with any resolution allowed by vbeinfo
>>
>> The default background image /usr/share/images/desktop-base/moreblue-orbit-grub.png
>> isn't showed in fullscreen but there is a black frame around the grub menu
>>     
> (grub-devel: This was originally filed at
> http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=575252.)
>
> Yes, I see the same thing.  This is because grub_gfxterm_fullscreen
> hardcodes a 10-pixel border around the edge of the screen.  If the
> background is just plain black, then this is fine, but it's not very
> helpful with a background image which tends to be designed for the full
> screen size.
>
> Of course designers *could* design for the full screen size minus 10
> pixels all round, but it seems a rather odd requirement to place on
> them; the 10-pixel border should essentially be so that text isn't drawn
> too close to the edge of the screen, not so much to restrict the
> background image.  And, if you're arranging for the background image to
> stay on the screen and transition relatively smoothly into an OS splash
> screen, then the border looks actively weird: see
> http://people.canonical.com/~cjwatson/tmp/fbcon-handoff.ogv around 0:13
> for an example, as Plymouth repaints the whole background image without
> the black border.
>
> What's the best way to fix this?  Perhaps the background_image command
> should disregard the border?
>
>   
The clean way would be to show image on the whole screen. It would
require some code restructuring. It's just not high enough on the list
of my priorities but I'll gladly accept the patch


-- 
Regards
Vladimir 'φ-coder/phcoder' Serbinenko



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 294 bytes --]

      reply	other threads:[~2010-07-16  9:20 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20100324142320.4804.63311.reportbug@localhost>
2010-07-06 17:19 ` Bug#575252: grub-pc: black frame around the background image Colin Watson
2010-07-15 20:33   ` Vladimir 'φ-coder/phcoder' Serbinenko [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4C3F7094.30503@gmail.com \
    --to=phcoder@gmail.com \
    --cc=grub-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.