2016-05-07 15:19 GMT+02:00 Carsten Haitzler <ras...@rasterman.com>:
> On Sat, 7 May 2016 13:57:44 +0200 Davide Andreoli <d...@gurumeditation.it>
> said:
>
> > 2016-05-06 19:01 GMT+02:00 Davide Andreoli <d...@gurumeditation.it>:
> >
> > > 2016-05-06 14:00 GMT+02:00 Carsten Haitzler <ras...@rasterman.com>:
> > >
> > >> On Fri, 6 May 2016 10:08:40 +0200 Davide Andreoli <
> d...@gurumeditation.it>
> > >> said:
> > >>
> > >> > 2016-05-06 4:01 GMT+02:00 Carsten Haitzler <ras...@rasterman.com>:
> > >> >
> > >> > > On Thu, 5 May 2016 23:16:56 +0200 Davide Andreoli <
> > >> d...@gurumeditation.it>
> > >> > > said:
> > >> > >
> > >> > > or use png's - keep the svg's in the tree as "originals" but
> export to
> > >> > > bimap
> > >> > > from inkscape.
> > >> > >
> > >> >
> > >> > This will duplicate the files in git, they are lots of files, we
> are now
> > >> > something like
> > >> > 3MB of icons, I was trying to avoid cluttering the repo. The svg
> format
> > >> > also works
> > >> > better with git as it's a text format, so future icon changes
> commits (I
> > >> > plan lots of
> > >> > them) will be slim.
> > >>
> > >> also another problem. the resulting theme will depend on how much the
> > >> host rsvg
> > >> lib can handle and what it does and does not render correctly from
> the svg
> > >> file. of course often this is due to inkscape using extensions of its
> > >> own...
> > >> but the point still stands. someone can't be guaranteed the same .edj
> file
> > >> contents because system will alter it as at edje_cc the svg is
> rendered
> > >> to a
> > >> bitmap by librsvg.
> > >>
> > >
> > > indeed this is a good point, different rsvg can be a problem, and no
> rsvg
> > > installed can be a problem too.
> > >
> > >
> > >>
> > >> having more png data in the tree is no harm. the text format is
> > >> irrelevant as
> > >> no one will be DIFFING the svg to figure stuff out. it's a visual
> thing -
> > >> the
> > >> svg is not hand edited. it's generated by a tool. treat it as a visual
> > >> thing.
> > >> you want visual diffs. text is not relevant as it's not hand edited.
> :)
> > >>
> > >
> > > My point was not on reading the diff, it was just on not having the
> binary
> > > file change at each icon commit.
> > >
> > >
> > >>
> > >> > Another reason I'm trying to avoid this is to keep my workflow as
> > >> simple as
> > >> > possible,
> > >> > that will means I can provide more icons in less time.
> > >>
> > >> by doing so you set up our build to produce a different theme on
> different
> > >> systems. :)
> > >>
> > >
> > > I start to think you are right... I will then add all the png version
> if
> > > no one
> > > have other valid point on using svg.
> > >
> >
> > I'm in the process of converting to png right now... the question is:
> > at what size should we rasterize them? I'm using 128x128 atm
> > it's this size too small? should I do them at 256? or both?
>
> 256 i would say. a script can actually generate 128x128, 64x64 etc.
> versions
> and we could use image {} sets to choose the appropriate one. a script or
> configure script segment could generate it. :)
>
I'm already using image sets for 16px icons that are less detailed than the
bigger
ones. Indeed 256 seems necessary also to me, but I'm not going to provide
all
the other sizes (any decent toolkit can handle the scaling).
So I will build:
* 128 and 256 from the same svg source
* 16px version from a different svg when it make sense.
> > > Ok, just leave out my icon problem for a moment... do we still want to
> > > merge
> > > the generic loaders? it make sense to make the merge to me also without
> > > the current issue. We are merging elm in this release, causing trouble
> to
> > > users and packagers, if we think it's sensible to also merge the last 2
> > > pieces
> > > this is the right moment IMO.
> > > My point is: do it now or never do it.
> > >
> > >
> > >>
> > >> > > beware librsvg != inskcape. inskcape has features it can put into
> > >> svg's
> > >> > > that
> > >> > > rsvg CAN'T draw right. it's SAFER to generate the png's and use
> those.
> > >> > >
> > >> >
> > >> > Yes, I'm aware of this. It's just a matter of checking your icons
> after
> > >> the
> > >> > draw and
> > >> > don't use "strange" inkscape feature. I also have the idea (not
> tested
> > >> yet)
> > >> > to
> > >> > save the files as "plain svg" instead of "inkscape svg".
> > >>
> > >> last time i tried, masking i think combined with clipping and blur did
> > >> this.
> > >> these are not strange filters. they are really common/basic. :) these
> are
> > >> incredibly useful and having to avoid them is seriously a bit pain.
> > >>
> > >> > > > ok, we are hitting a problem with evas-generic-loaders because
> it's
> > >> not
> > >> > > > merged in the efl tree.
> > >> > > >
> > >> > > > The problem is that we are now using svg icons in the elementary
> > >> theme,
> > >> > > all
> > >> > > > the new FDO icons are in svg format. But, now that elm is
> merged in
> > >> efl,
> > >> > > > the default theme is built before the user can install
> > >> > > evas-generic-loaders
> > >> > > > and thus the theme cannot be build for the lack of the svg
> loader.
> > >> > > >
> > >> > > > After some chatting we agreed that the "best" way to solve the
> > >> issue is
> > >> > > to
> > >> > > > also merge evas-generic-loaders in efl.
> > >> > > > The alternative solution is to duplicate all the icons in a
> raster
> > >> > > format,
> > >> > > > that seems really a waste of space/bandwidth/git-history on each
> > >> icon
> > >> > > > change and will make my work of providing icons much harder.
> > >> > > >
> > >> > > > Any objections? problems? me and cedric already agreed on this
> > >> solution,
> > >> > > > and also
> > >> > > > agreed that make sense to also merge emotion-generics, so we
> will
> > >> finally
> > >> > > > have a single efl package!
> > >> > > >
> > >> > > >
> > >> > > > If no objections arise I will ask TAsn to do the same git work
> he
> > >> did for
> > >> > > > elm (merging the git tree in a branch?) so that cedric can do
> the
> > >> > > required
> > >> > > > autofoo stuff (already agreed with him)
> > >> > > >
> > >> > > >
> > >> > > > In the mean time I have to disable :( all the new icons so that
> > >> everyone
> > >> > > > can build efl without problems. I hope this will not create some
> > >> side
> > >> > > > effects, as it is now possible that some old icons are missing.
> Let
> > >> me
> > >> > > know
> > >> > > > for any issue.
> > >> > > >
> > >> > > > davemds
> > >> > > >
> > >> > >
> > >>
> ------------------------------------------------------------------------------
> > >> > > > Find and fix application performance issues faster with
> Applications
> > >> > > Manager
> > >> > > > Applications Manager provides deep performance insights into
> > >> multiple
> > >> > > tiers of
> > >> > > > your business applications. It resolves application problems
> > >> quickly and
> > >> > > > reduces your MTTR. Get your free trial!
> > >> > > > https://ad.doubleclick.net/ddm/clk/302982198;130105516;z
> > >> > > > _______________________________________________
> > >> > > > enlightenment-devel mailing list
> > >> > > > enlightenment-devel@lists.sourceforge.net
> > >> > > >
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
> > >> > > >
> > >> > >
> > >> > >
> > >> > > --
> > >> > > ------------- Codito, ergo sum - "I code, therefore I am"
> > >> --------------
> > >> > > The Rasterman (Carsten Haitzler) ras...@rasterman.com
> > >> > >
> > >> > >
> > >>
> > >>
> > >> --
> > >> ------------- Codito, ergo sum - "I code, therefore I am"
> --------------
> > >> The Rasterman (Carsten Haitzler) ras...@rasterman.com
> > >>
> > >>
> > >
>
>
> --
> ------------- Codito, ergo sum - "I code, therefore I am" --------------
> The Rasterman (Carsten Haitzler) ras...@rasterman.com
>
>
------------------------------------------------------------------------------
Find and fix application performance issues faster with Applications Manager
Applications Manager provides deep performance insights into multiple tiers of
your business applications. It resolves application problems quickly and
reduces your MTTR. Get your free trial!
https://ad.doubleclick.net/ddm/clk/302982198;130105516;z
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
