[Webkit-unassigned] [Bug 134674] [EFL][WK2] Add a "focus, control, handover" signal.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=134674





--- Comment #12 from Gyuyoung Kim <gyuyoung.kim at webkit.org>  2014-07-16 17:44:11 PST ---
(From update of attachment 234990)
View in context: https://bugs.webkit.org/attachment.cgi?id=234990&action=review

>>> Source/WebKit2/UIProcess/API/efl/ewk_view.h:42
>>> + * - "focus,control,handover", Ewk_Focus_Direction*: reports that user can take the focus from the ewk_view because
>> 
>> This signal is to notify if focus direction is changed. EWK_FOCUS_DIRECTION_FORWARD or EWK_FOCUS_DIRECTION_BACKWARD. This signal name doesn't contain meaning for user. "focus,direction,changed" looks better. Besides your description regarding "handover focus control bla bla..." is redundant. We only need to explain what does this signal report.
> 
> IMHO, This signal is to notify that there is no focusable element when we press the tab or tab + shift keys regardless of focus direction.
> Purpose of this signal is to notify that application can handover focus control.
> Therefore I think this signal name is not bad but you still prefer to "focus,direction,changed", I'll fix it.

Doesn't Application get focus direction eventually ? Anyway, I think we should use a signal name from the signal occurrence of view.

Look at existing signals

"title,changed",
"url,changed",
"webprocess,crashed",

If this signal is for notifying there isn't any focusable element, it looks "focus,notfound" can be one of candidates.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.