Edje entry: Use markup text for password in preedit mode
authorSubodh Kumar <s7158.kumar@samsung.com>
Thu, 3 Dec 2015 16:04:07 +0000 (16:04 +0000)
committerTom Hacohen <tom@stosb.com>
Thu, 3 Dec 2015 16:04:08 +0000 (16:04 +0000)
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

src/lib/edje/edje_entry.c

index 38fb5ff..9ed6ff7 100644 (file)
@@ -4297,7 +4297,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",