Page MenuHomePhabricator

CtxPopup: Add implementation for getting focused item
ClosedPublic

Authored by godlytalias on Nov 14 2017, 6:16 AM.

Details

Summary

Adding the focused_item_get implementation for ctxpopup widget for
getting the focused item

Signed-off-by: Godly T.Alias <godlytalias@yahoo.co.in>

Test Plan

elm_object_focused_item_get

Diff Detail

Repository
rEFL core/efl
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
godlytalias created this revision.Nov 14 2017, 6:16 AM
shilpasingh accepted this revision.Nov 14 2017, 6:37 AM
This revision is now accepted and ready to land.Nov 14 2017, 6:37 AM

I thought ctxpopup could have content other than a list?
@Blackmole can you check this please?

jpeg added a comment.Nov 15 2017, 12:12 AM
In D5474#93626, @jpeg wrote:

I thought ctxpopup could have content other than a list?
@Blackmole can you check this please?

Oh wait, this API only makes sense for lists? If so then it is OK. @Blackmole please check this patch first.

Focused item get API will be valid only when ctxpopup holds items,
If there is no items then this API don't have to handle anything.

jpeg accepted this revision.Nov 15 2017, 2:28 AM
Closed by commit rEFL67313bf8aa0f: CtxPopup: Add implementation for getting focused item (authored by godlytalias, committed by Jean-Philippe Andre <jp.andre@samsung.com>). · Explain WhyNov 15 2017, 2:30 AM
This revision was automatically updated to reflect the committed changes.