All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Daniel P. Berrangé" <berrange@redhat.com>
To: Gerd Hoffmann <kraxel@redhat.com>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH 1/3] ui: install logo icons to $prefix/share/icons
Date: Wed, 19 Dec 2018 10:23:11 +0000	[thread overview]
Message-ID: <20181219102311.GD20465@redhat.com> (raw)
In-Reply-To: <20181219101957.yonihjoa2a5crpr6@sirius.home.kraxel.org>

On Wed, Dec 19, 2018 at 11:19:57AM +0100, Gerd Hoffmann wrote:
>   Hi,
> 
> > > How did you create the png files?
> > > Can you drop a script or Makefile for that into the ui/icons directory?
> > 
> > I used GIMP so that it created high quality antialiased pngs.
> 
> Ok, so that'll be hardly scriptable.
> A short README will do then I guess.
> 
> > > Should we commit them to git, or generate at build time?
> > > Is the tool used something we can expect to be installed?
> > 
> > I tried using ImageMagick to convert SVG to PNG but the results
> > were really poor compared to GIMP - very fuzzy. Running with
> > -verbose I can see it is invoking inkscape to atually do the
> > conversion, and it tells inkscape to export a small image which
> > it then resizes, hence the quality loss.
> 
> Tried to call inkscape directly?
> Having something scriptable would be nice.

I'll have a play with calling inkscape directly.

> But I'm fine with a README saying gimp gives good results too.
> 
> > The icons are small enough that we shouldn't really worry about
> > having pre-rendered icons in GIT IMHO. They're not ever going to
> > change unless we replace our logo again.
> 
> And inkscape as package build dependency wouldn't be nice either.

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

  reply	other threads:[~2018-12-19 10:23 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-12-18 14:26 [Qemu-devel] [PATCH 0/3] Fix & improve icon display on GTK and SDL frontends Daniel P. Berrangé
2018-12-18 14:26 ` [Qemu-devel] [PATCH 1/3] ui: install logo icons to $prefix/share/icons Daniel P. Berrangé
2018-12-19  7:20   ` Gerd Hoffmann
2018-12-19 10:03     ` Daniel P. Berrangé
2018-12-19 10:19       ` Gerd Hoffmann
2018-12-19 10:23         ` Daniel P. Berrangé [this message]
2018-12-18 14:26 ` [Qemu-devel] [PATCH 2/3] ui: fix icon display for GTK frontend under GNOME Shell with Wayland Daniel P. Berrangé
2018-12-18 14:26 ` [Qemu-devel] [PATCH 3/3] sdl: add support for high resolution window icon Daniel P. Berrangé
2018-12-25  1:34 ` [Qemu-devel] [PATCH 0/3] Fix & improve icon display on GTK and SDL frontends no-reply

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=20181219102311.GD20465@redhat.com \
    --to=berrange@redhat.com \
    --cc=kraxel@redhat.com \
    --cc=qemu-devel@nongnu.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.