Edje entry: Use markup text for password in preedit mode 20/65920/1
authorSubodh Kumar <s7158.kumar@samsung.com>
Thu, 3 Dec 2015 16:04:07 +0000 (16:04 +0000)
committerYoungbok Shin <youngb.shin@samsung.com>
Thu, 14 Apr 2016 04:40:33 +0000 (13:40 +0900)
Summary:
Use markup text for password in preedit mode

Problem is that when text comes in preedit mode
it is always markup text and when it is committed
to entry it got committed with markup tags along with
the original string, so for preedit mode password
should be filtered all markup to set the proper text.

@fix

Test Plan:
In preedit mode, enter any character say 'A'
preedit markup like <preedit>A</preedit>
will come in preedit changed callback

Result: Text is set as plain text like below
&lt;preedit&gt;A&lt;/preedit&gt;

Reviewers: thiepha, jihoon, herdsman, shilpasingh, tasn

Reviewed By: shilpasingh, tasn

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2929

Change-Id: Ib53e9cc414ab2e84bba27adfd6676c5d32d150ac

src/lib/edje/edje_entry.c

index 77f5408..5a654b8 100644 (file)
@@ -4466,7 +4466,7 @@ _edje_entry_imf_event_preedit_changed_cb(void *data, Ecore_IMF_Context *ctx EINA
              Edje_Entry_Change_Info *info;
 
              _edje_entry_hide_visible_password(ed, en->rp);
-             info = _text_filter_text_prepend(ed, en, en->cursor,
+             info = _text_filter_markup_prepend(ed, en, en->cursor,
                                               eina_strbuf_string_get(buf),
                                               "+ password=off",
                                               "- password",