Normally I would not comment on a thread like this as "the issue" seems 
to be resolved ....

However, I just wanted to add a note for Jeff (and potentially others 
who do distros, themes, etc) and that is:

If You make themes, new code, code changes, etc, which basically "copies 
the default" (then makes modifications) ... The best advise I could give 
would be:

"Keep Track Of Changes (in our git master) For What You Copied"

I realize this "sounds" like beating a dead horse (not what I am trying 
to do tho), but keeping track of those changes would avoid future 
frustration and issues (like this). Also it would make updating to 
future versions (eg: EFL 1.6) much easier ;)

Kind Regards and Happy Hacking ;)

dh


On 11/18/2015 07:08 PM, 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.
>
> 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

Reply via email to