Hi all,

The broken dialog and notification windows were introduced in a previous commit of mine. This commit fixes those issues for me.

What do I need to run to reproduce the issues?

--
Danny (herdsman) Hirt

On 10/02/2017 06:10 PM, Al Poole wrote:
Desktop notifications too (in case that's fixed/handled differently).

On Mon, Oct 2, 2017 at 12:42 PM, Carsten Haitzler <[email protected]>
wrote:

On Mon, 02 Oct 2017 01:13:41 -0700 Daniel Hirt <[email protected]>
said:

now all of e's error dialogs are broke with visible tags..

herdsman pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=
eadf313702f81f05d39ba6c47385b8d8420a1330

commit eadf313702f81f05d39ba6c47385b8d8420a1330
Author: Daniel Hirt <[email protected]>
Date:   Mon Oct 2 11:03:37 2017 +0300

     Canvas layout: fix Efl.text (non-legacy) TEXT parts

     Current implementation should also support TEXT parts (like button)
     with usage of efl_text_set.

     Follow-up to 0061a4bb349e6c8e122913791bf83f7f40b0cede.
---
  src/lib/edje/edje_util.c | 24 +++++++++---------------
  1 file changed, 9 insertions(+), 15 deletions(-)

diff --git a/src/lib/edje/edje_util.c b/src/lib/edje/edje_util.c
index db0a3939c7..8db4ffd9a3 100644
--- a/src/lib/edje/edje_util.c
+++ b/src/lib/edje/edje_util.c
@@ -1861,25 +1861,19 @@ _edje_object_part_text_raw_generic_set(Edje *ed,
Evas_Object *obj, Edje_Real_Par else
     if (text)
       {
-        if (legacy)
+        if (legacy || (rp->part->type == EDJE_PART_TYPE_TEXT) ||
set_markup)
            {
               rp->typedata.text->text = eina_stringshare_add(text);
            }
-        else if (rp->part->type == EDJE_PART_TYPE_TEXTBLOCK)
+        else // !legacy && (rp->part->type == EDJE_PART_TYPE_TEXTBLOCK)
+             // && !set_markup
            {
-             if (set_markup)
-               {
-                  rp->typedata.text->text = eina_stringshare_add(text);
-               }
-             else
-               {
-                  char *mkup;
-                  mkup = efl_text_markup_util_text_to_markup(
-                     EFL_TEXT_MARKUP_UTIL_CLASS,
-                     text);
-                  rp->typedata.text->text = eina_stringshare_add(mkup);
-                  free(mkup);
-               }
+             char *mkup;
+             mkup = efl_text_markup_util_text_to_markup(
+                EFL_TEXT_MARKUP_UTIL_CLASS,
+                text);
+             rp->typedata.text->text = eina_stringshare_add(mkup);
+             free(mkup);
            }
       }
     ed->dirty = EINA_TRUE;

--




--
------------- Codito, ergo sum - "I code, therefore I am" --------------
Carsten Haitzler - [email protected]


------------------------------------------------------------
------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to