On Thu, 19 Nov 2015 17:11:43 +1030 Simon Lees <si...@simotek.net> said:

> 
> 
> On 11/19/2015 10:38 AM, Jeff Hoogland wrote:
> > I understand why this broke and I appreciate folks pointing out the fix.
> > The point is it still broke and will require more time to fix the new elm
> > we are using / maintaining.
> >
> > My point is that any change like this feels bad and makes updating
> > difficult. Not a big deal, I will make a note when I eventually update EFL
> > / Elementary that I'll need to update all my themes. This just means it
> > will be later rather than sooner.
> I guess the main thing that could have been done better here is 
> communicating that there was a bug in the default theme which was fixed 
> and that themes based off it should incorporate the change, if that sort 
> of info was in the release notes it would be helpful. Having said that I 
> merge dark back into my theme every efl release to make sure that new 
> widgets etc make it into my theme, generally for me this is about a 5 
> minute process as I use git to manage my themes effectively at some 
> point hopefully soon I'll start a wiki page on making custom themes but 
> hopefully i'll have more time for that on the weekend.

it actually is in the NEWS file for the 1.16:

   * elm toolbar 0 size items fix if icon not found (T2710)

ok - not with the same name and description, but the fix is there. :) what do
you expect us to stuff into a single line shortlog? :)

> Cheers
> Simon
> >
> > On Wed, Nov 18, 2015 at 5:51 PM, Carsten Haitzler <ras...@rasterman.com>
> > wrote:
> >
> >> On Wed, 18 Nov 2015 13:20:14 -0600 Jeff Hoogland <jeffhoogl...@gmail.com>
> >> said:
> >>
> >>> Good to know the theme API still is allowed to break release to release.
> >>>
> >>> Guess I'll stick with 1.15 for awhile so I don't have to worry about
> >>> chasing down issues with each release.
> >> no one said it was allowed to break and no single commit to toolbar code
> >> between 1.15 and 1.16 goes explicitly breaking any theme api - go read the
> >> commits and their diffs. we emit more signals than before - but we didn't
> >> remove or rename any. nor any swallow parts, text part names etc. etc.
> >>
> >> before you go making accusations - make sure you have at least some proof.
> >> the
> >> ONLY change to the default theme regarding toolbar was a bugfix in the
> >> default
> >> theme (yes - it has BUGS - funny that, and then you copy the theme, do your
> >> own, and very likely DON'T fix corresponding bugs... but let's not dwell on
> >> that)... let's go grab some.... facts.
> >>
> >> only 2 commits happened to default theme between 1.15 a 1.16 affecting
> >> toolbar.
> >>
> >> ed948dc90829704860d355cc17b7da201d744719 by mike that simply removed a line
> >> that was a default value anyway.
> >>
> >> 740cc0751807bb0c0bbb393d6e0d2e1588eb0678 that FIXED ZERO sized items. not
> >> zero
> >> sized icons tho... it makes item_centered just an alias instead of an
> >> inherited
> >> part (saves space as the inherit made no changes) and it fixes the
> >> item_horizontal which is not used in your app. it is what is used in
> >> terminology tho...
> >>
> >> so no changes in default theme related to your issue.
> >>
> >> so lets look at code. 1.15 came out august 5. so:
> >>
> >> git diff -u --color src/lib/elm_toolbar.c
> >>
> >> all commits newer than 6008c2c3867e3198d468b66d703aa39ee21445a5 ... go
> >> read the
> >> diffs. find me one that goes brazenly breaking theme api. go on. find it.
> >> none.
> >> the icon_size data field is used nowhere in default theme - so changes to
> >> making it apply elm scale wouldn't break anything as you'd have to be
> >> setting
> >> scale to 0 or have an infinite base scale for this to change the value, so
> >> if
> >> anything you already had it and it returned 0. so it can't sensibly be
> >> that.
> >>
> >> getting focus? no - not related to icon sizing.
> >>
> >> sep 12 look - handling 0 item size ... we emitted the incorrect signals to
> >> items
> >> i fixed it. with some extra emits to explicitly make icon visible (not your
> >> problem unless your theme is hiding icons when the signal says to make them
> >> visible... is it? that'd be a theme bug) and of course emitting signals if
> >> there is no icon (check code that is patched... the first if () is if
> >> (it->icon) ...) so it's handling the "i have no icon" case as well as the
> >> "i
> >> have no label" case which it didn't before.
> >>
> >> sep 15  - other focus changes. not icon or theme.
> >>
> >> sep 23 then select mode changes, not icon/theme.
> >>
> >> oct 2 - then atspi changes only doing things when atspi mode is on.
> >>
> >> oct 12 - then a toolbar shrink mode change only applies if you have no
> >> items...
> >> not your case.
> >>
> >> now oct 15 - some moving of sizing evaluation to defer it to later to save
> >> cpu
> >> time. not a theme api break.
> >>
> >> now some more atspi stuff - not theme or icons...
> >>
> >> now oct 19 some more limiting of re-calcs to save cpu time
> >>
> >> nov 5 - atspi stuff again
> >>
> >> then 1.16 release.
> >>
> >> look. there's a reason i'm snapping back here. if you make an accusation
> >> like
> >> we are happily allowing theme api to break (and we care and actually try
> >> not
> >> to) then at least have something to back it up. show me a commit, change
> >> etc.
> >> that is obviously some theme api break we should have caught and not done?
> >> show
> >> me. i've BOTHERED to read commits and found nothing obvious to hunt.
> >> whatever
> >> has happened will take far more time and poking around and we have no idea
> >> WHAT
> >> the problem is.
> >>
> >> it could quite easily be simply a theme bug in your theme that now just
> >> HAPPENS
> >> to be triggered. it could easily be that it got away by LUCK before, not by
> >> design. much like if your distro booted right to a root prompt before and
> >> maybe
> >> now has decided to force a login instead?  if maybe before i did char *x =
> >> malloc(10); free(x); *x = 1; and it WORKED before and NOW crashes because
> >> it's
> >> now stricter and actually fails when you do something wrong.
> >>
> >> i don't know, BUT i can say this. the toolbar icons are elm icon widgets.
> >> they
> >> are swallowed. unless they set a minimum size of their own - they will very
> >> likely size down to 0 unless the toolbar itself is forcibly expanded to be
> >> bigger so there is space for icons to grow. icon size can be determined by
> >> theme and it can be overridden by api. elm icon itself doesn't determine
> >> its own
> >> min size at all. in fact i'm kind of baffled by icons are not 0 sized by
> >> default. it seems by luck they are not - but what luck, i don't know
> >>
> >> so jeff - please at least dig a little bit before making unsubstantiated
> >> accusations.
> >>
> >>> On Sat, Nov 14, 2015 at 8:21 PM, Jeff Hoogland <jeffhoogl...@gmail.com>
> >>> wrote:
> >>>
> >>>> Opened a ticket -> https://phab.enlightenment.org/T2848
> >>>>
> >>>> On Sat, Nov 14, 2015 at 8:11 PM, Simon Lees <si...@simotek.net> wrote:
> >>>>
> >>>>>
> >>>>> On 11/15/2015 12:31 PM, Jeff Hoogland wrote:
> >>>>>> This seems to be theme related, icons appear as expected in the
> >> default
> >>>>>> theme on Elementary 1.16
> >>>>>>
> >>>>>> Guess I'll open a bug report.
> >>>>> I can confirm this on a older theme i have, the ones I merged with
> >> dark
> >>>>> in the last month or so seem fine.
> >>>>>
> >>>>>> On Sat, Nov 14, 2015 at 7:54 PM, Jeff Hoogland <
> >> jeffhoogl...@gmail.com>
> >>>>>> wrote:
> >>>>>>
> >>>>>>> Anyone else seeing an issue with images on toolbars with 1.16?
> >> ePad and
> >>>>>>> ePhoto are both missing icons on their tool bar here with the 1.16
> >>>>> bindings
> >>>>>>> VS 1.15.1
> >>>>>>>
> >>>>>>> Screenshot:
> >>>>>>>
> >>>>>>> http://i.imgur.com/wmuiJYB.png
> >>>>>>>
> >>>>>>> Exact same ePad code, the one on the right is using elementary
> >> 1.15.1
> >>>>> the
> >>>>>>> one on the left is using 1.16
> >>>>>>>
> >>>>>>> On Mon, Nov 9, 2015 at 5:09 AM, Stefan Schmidt <
> >> ste...@osg.samsung.com
> >>>>>>> wrote:
> >>>>>>>
> >>>>>>>> = Enlightenment Foundation Libraries 1.16 and Friends are Out =
> >>>>>>>>
> >>>>>>>> After three months of development work we are proud to announce
> >> the
> >>>>>>>> release of
> >>>>>>>> version 1.16 of the EFL, Elementary, Evas Generic Loaders and
> >> Emotion
> >>>>>>>> Generic
> >>>>>>>> Players. In these 12 weeks we got over 700 commits from 58
> >> authors in
> >>>>>>>> EFL alone.
> >>>>>>>> We slowed down a bit from last release (by around 300 commits).
> >>>>>>>> Elementary has another
> >>>>>>>> 382 commits by 52 authors. Great job everyone! Some highlights are
> >>>>>>>> listed below.
> >>>>>>>>
> >>>>>>>> == Download ==
> >>>>>>>>
> >>>>>>>> http://download.enlightenment.org/rel/libs/efl/efl-1.16.0.tar.gz
> >>>>>>>> 35af0783d6dd87ebb435c46fd662ef5a648510d74fb6dc6540a3165c9b221e50
> >>>>>>>>
> >>>>>>>>
> >>>>>>>>
> >> http://download.enlightenment.org/rel/libs/elementary/elementary-1.16.0.tar.gz
> >>>>>>>> 18b1f2d55af46840d0a9477a615d0c5a033b24f06d8fcf539da33aa5db357bd0
> >>>>>>>> <
> >> http://download.enlightenment.org/rel/libs/elementary/elementary-1.16.0.tar.gz18b1f2d55af46840d0a9477a615d0c5a033b24f06d8fcf539da33aa5db357bd0
> >>>>>>>>
> >>>>>>>>
> >> http://download.enlightenment.org/rel/libs/emotion_generic_players/emotion_generic_players-1.16.0.tar.gz
> >>>>>>>> cbb34faa0b6d114a3a0a5e4451f03f8423c5a48fd3fe9a6c6ecdfa65f4083c80
> >>>>>>>> <
> >> http://download.enlightenment.org/rel/libs/emotion_generic_players/emotion_generic_players-1.16.0.tar.gzcbb34faa0b6d114a3a0a5e4451f03f8423c5a48fd3fe9a6c6ecdfa65f4083c80
> >>>>>>>>
> >>>>>>>>
> >> http://download.enlightenment.org/rel/libs/evas_generic_loaders/evas_generic_loaders-1.16.0.tar.gz
> >>>>>>>> 0928a675bca95f7929626550eae85559d296b33b3ecc111aece4744601369a1a
> >>>>>>>> <
> >> http://download.enlightenment.org/rel/libs/evas_generic_loaders/evas_generic_loaders-1.16.0.tar.gz0928a675bca95f7929626550eae85559d296b33b3ecc111aece4744601369a1a
> >>>>>>>> ----
> >>>>>>>>
> >>>>>>>> = What's New =
> >>>>>>>>
> >>>>>>>> Even if we had only three months a lot has happened during the
> >> 1.16
> >>>>>>>> release cycle.
> >>>>>>>> We only cover some highlights here. For the full list please look
> >> at
> >>>>> the
> >>>>>>>> NEWS files
> >>>>>>>> if you are interested in the details. As usual we have been
> >> working on
> >>>>>>>> fixing bugs,
> >>>>>>>> optimising our code for speed and memory footprint and adding new
> >>>>>>>> features.
> >>>>>>>> Below we describe just some highlights and by no means provide a
> >> full
> >>>>>>>> list.
> >>>>>>>>
> >>>>>>>> = EFL =
> >>>>>>>>
> >>>>>>>> == Wayland ==
> >>>>>>>> The focus of work on our wayland areas have been stabilisation and
> >>>>>>>> filling of gaps for the upcoming E20 release. To make 1.16 a solid
> >>>>> base
> >>>>>>>> for it work has been gone to make the evas drm engine more solid
> >> and
> >>>>>>>> adding support for things like adjusting keyboard repeat and rate
> >>>>> delay,
> >>>>>>>> etc
> >>>>>>>>
> >>>>>>>> == Evas GL ==
> >>>>>>>> Besides some normal optimisation evas GL got support for the new
> >>>>> partial
> >>>>>>>> update extension.
> >>>>>>>>
> >>>>>>>> == Eina ==
> >>>>>>>> To aid debugging and tooling infrastructure eina thread does now
> >>>>> support
> >>>>>>>> setting thread names and many internal threads are already named.
> >>>>>>>>
> >>>>>>>> == Eo (BETA) ==
> >>>>>>>> A lot of optimisation, memory as well as CPU time, has been gone
> >> into
> >>>>> EO
> >>>>>>>> this time. Initially we hoped to declare EO stable after this
> >>>>>>>> development cycle, but we need to postpone this for at least
> >> another
> >>>>>>>> cycle as some more optimisation are being planned which might
> >> break
> >>>>> ABI.
> >>>>>>>> == Elementary ==
> >>>>>>>> We improved our thumbscroll finger scrolling to be smoother and
> >> worked
> >>>>>>>> on various parts of focus handling of items. On the ATSPI front we
> >>>>> added
> >>>>>>>> a translation domain for name and description as well as support
> >> for
> >>>>> the
> >>>>>>>> org.a11y.atspi.Collection interface.
> >>>>>>>>
> >>>>>>>> ----
> >>>>>>>>
> >>>>>>>> = Statistics =
> >>>>>>>>
> >>>>>>>> == EFL ==
> >>>>>>>> (git log --pretty=oneline v1.15.0..v1.16.0 | wc -l)
> >>>>>>>> Number of commits: 701
> >>>>>>>>
> >>>>>>>> (git shortlog -ns v1.15.0..v1.16.0 | wc -l)
> >>>>>>>> Number of authors: 58
> >>>>>>>>
> >>>>>>>> (git diff --stat v1.15.0..v1.16.0 | tail -1)
> >>>>>>>> 546 files changed, 52121 insertions(+), 24088 deletions(-)
> >>>>>>>>
> >>>>>>>> == Elementary ==
> >>>>>>>> (git log --pretty=oneline v1.15.0..v1.16.0 | wc -l)
> >>>>>>>> Number of commits: 382
> >>>>>>>>
> >>>>>>>> (git shortlog -ns v1.15.0..v1.16.0 | wc -l)
> >>>>>>>> Number of authors: 52
> >>>>>>>>
> >>>>>>>> (git diff --stat v1.15.0..v1.16.0 | tail -1)
> >>>>>>>> 305 files changed, 23711 insertions(+), 15250 deletions(-)
> >>>>>>>>
> >>>>>>>> ----
> >>>>>>>>
> >>>>>>>> =Building and Dependencies=
> >>>>>>>>
> >>>>>>>> If you have an existing EFL or Elementary install, you may wish to
> >>>>>>>> delete its header files and libraries before compiling and
> >> installing
> >>>>> to
> >>>>>>>> avoid possible conflicts during compilation. If you are compiling
> >> the
> >>>>>>>> above, please compile them in the following order:
> >>>>>>>>
> >>>>>>>> efl
> >>>>>>>> elementary
> >>>>>>>> emotion_generic_players
> >>>>>>>> evas_generic_loaders
> >>>>>>>> Please refer to the respective README files in each release for a
> >> full
> >>>>>>>> list of dependencies, explanations on configure flags and other
> >>>>> relevant
> >>>>>>>> information (Just scroll down to see the README already displayed
> >>>>> nicely).
> >>>>>>>> EFL
> >>>>>>>> Elementary
> >>>>>>>> Emotion Generic Players
> >>>>>>>> Evas Generic Loaders
> >>>>>>>> Recommended dependencies are for all of the above are:
> >>>>>>>>
> >>>>>>>> bullet
> >>>>>>>> libpng
> >>>>>>>> libjpeg
> >>>>>>>> gstreamer (1.x) (ensure you have all the gstreamer codecs you
> >>>>> need/want.)
> >>>>>>>> zlib
> >>>>>>>> luajit (optional lua 5.1 or 5.2)
> >>>>>>>> libtiff
> >>>>>>>> openssl
> >>>>>>>> curl
> >>>>>>>> dbus
> >>>>>>>> libc
> >>>>>>>> fontconfig
> >>>>>>>> freetype2
> >>>>>>>> fribidi
> >>>>>>>> libpulse
> >>>>>>>> libsndfile
> >>>>>>>> libx11
> >>>>>>>> libxau
> >>>>>>>> libxcomposite
> >>>>>>>> libxdamage
> >>>>>>>> libxdmcp
> >>>>>>>> libxext
> >>>>>>>> libxfixes
> >>>>>>>> libxinerama
> >>>>>>>> libxrandr
> >>>>>>>> libxrender
> >>>>>>>> libxss
> >>>>>>>> libxtst
> >>>>>>>> libxcursor
> >>>>>>>> libxp
> >>>>>>>> libxi (2.2 or newer)
> >>>>>>>> libgl (opengl/glx or opengl-es2/egl)
> >>>>>>>> giflib
> >>>>>>>> util-linux (limbount + libblkid)
> >>>>>>>> systemd / libudev
> >>>>>>>> harfbuzz
> >>>>>>>> systemd
> >>>>>>>> libwebp
> >>>>>>>> vlc (libvlc)
> >>>>>>>> poppler
> >>>>>>>> libraw
> >>>>>>>> libspectre
> >>>>>>>> librsvg
> >>>>>>>>
> >>>>>>>> It is suggested you add the following options when building the
> >> EFL to
> >>>>>>>> get a better experience:
> >>>>>>>> --enable-xinput22 \
> >>>>>>>> --enable-multisense \
> >>>>>>>> --enable-systemd \
> >>>>>>>> --enable-image-loader-webp \
> >>>>>>>> --enable-harfbuzz
> >>>>>>>>
> >>>>>>>> On linux to get framebuffer support you may want to also do:
> >>>>>>>> --enable-fb
> >>>>>>>> --disable-tslib
> >>>>>>>>
> >>>>>>>> If you want wayland support also add:
> >>>>>>>> --enable-wayland
> >>>>>>>>
> >>>>>>>> At runtime if you want thumbnailing for DOC/PPT/XLS etc. files
> >> also
> >>>>>>>> provide:
> >>>>>>>> libreoffice
> >>>>>>>>
> >>>>>>>>
> >>>>>>>>
> >>>>>>>>
> >>>>>>>>
> >> ------------------------------------------------------------------------------
> >>>>>>>> Presto, an open source distributed SQL query engine for big data,
> >>>>>>>> initially
> >>>>>>>> developed by Facebook, enables you to easily query your data on
> >>>>> Hadoop in
> >>>>>>>> a
> >>>>>>>> more interactive manner. Teradata is also now providing full
> >>>>> enterprise
> >>>>>>>> support for Presto. Download a free open source copy now.
> >>>>>>>> http://pubads.g.doubleclick.net/gampad/clk?id=250295911&iu=/4140
> >>>>>>>> _______________________________________________
> >>>>>>>> enlightenment-devel mailing list
> >>>>>>>> enlightenment-devel@lists.sourceforge.net
> >>>>>>>> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
> >>>>>>>>
> >>>>>>>
> >>>>>>> --
> >>>>>>> ~Jeff Hoogland <http://jeffhoogland.com/>
> >>>>>>> My Projects on GitHub <https://github.com/JeffHoogland>
> >>>>>>>
> >>>>>>
> >>>>>
> >>>>>
> >>>>>
> >> ------------------------------------------------------------------------------
> >>>>> _______________________________________________
> >>>>> enlightenment-devel mailing list
> >>>>> enlightenment-devel@lists.sourceforge.net
> >>>>> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
> >>>>>
> >>>>
> >>>>
> >>>> --
> >>>> ~Jeff Hoogland <http://jeffhoogland.com/>
> >>>> My Projects on GitHub <https://github.com/JeffHoogland>
> >>>>
> >>>
> >>>
> >>> --
> >>> ~Jeff Hoogland <http://jeffhoogland.com/>
> >>> My Projects on GitHub <https://github.com/JeffHoogland>
> >>>
> >> ------------------------------------------------------------------------------
> >>> _______________________________________________
> >>> 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
> >>
> >>
> >
> 
> 
> ------------------------------------------------------------------------------
> _______________________________________________
> 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


------------------------------------------------------------------------------
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to