In this scenario:
I expect using elm_object_focus_next(root_obj, RIGHT or UP) the focus to move to the top-right buttons,
but nothing happend with current/new focus implementation.
The 3 buttons are thus unreachable
Also the behaviour changed too much (IMO) from last stable release.
| Status | Assigned | Task | ||
|---|---|---|---|---|
| Resolved | bu5hm4n | T3088 Elementary needs a complete focus overhaul | ||
| Wontfix | bu5hm4n | T6453 Elm focus unreachable widgets |
Yeah, @woohyun was also talking about that to me.
I am not too sure what we should do about that... :(
I will work on a proposal next week
Propsoal:
We are doing the scanning for the relations [Right, Left, Top, Down] twice.
Once with the behaviour of now, and once with the behaviour that the nearest element (nearest by 2D distance) in the direction of the relation will taken, illustrated sample:
What do you think?
ok, the latest commit make it (quite) work.
There are now 2 remaining issues:
I wrote a new test in elm_test (Focus 6), it is pratically the same layout of my mediacenter and it have similar issues.
Please also give a look at the test code there is another bug (commented out) that need to be fixed.
@DaveMDS can you recheck ? The normal test should work now as expected i guess.
(There might still be the case of a weird highlight arround a wrong widget, i still have to dig into that what that is, but at least keystrokes are working correctly)
Will dig into the commented out stuff.
what I need to check? I cannot see any commit from you in master nor in any branch of yours...
current master still broken as before
are you the drunken one that forgot to push? or I'm the drunken one that cannot find your fix?
Haha!
I committed the fix a few days ago, and forgot to comment. on my system i can switch between The button and the genlist ...
As per today irc discussion:
Currently the only way to make the focus-test-6 works is:
start the test, click on btn, click on genlist, click in btn, click on genlist... focus now works as expected.
So no, the suff you committed some days ago did not fixed the test case.
...me waiting for a real fix.
Some update on this: I just added the ability in test_focus6 to move the focus by API, that is what I'm doing in EpyMC, but it works as expected in the test so... this is not my issue :(
in the meantime I discovered another bug in the test, to reproduce:
as you can see 3 times is not enough to go back to the buttons, you need to press UP again some more times to focus the buttons. The same happend moving the focus with the bottom buttons, numbers are a bit different in this case (because items are not focused) but you can see the same odd behaviour.
Re-opening this... as I found 2 new issue:
First issue:
Second issue:
Well, all of this is not a unfocusable widget ... Rather a new issue ... Also the 100% CPU issue, where does it spin ?
when it freeze I see this last error:
EFL ERROR Focusable 0x4000001325df (Elm.Genlist.Item) not registered in manager 0x400000140351
and I catched this backtrace from gdb:
EFL ERROR Focusable 0x4000001c71cc (Elm.Genlist.Item) not registered in manager 0x4000000e2d5e
^C
Thread 1 "python" received signal SIGINT, Interrupt.
0x00007ffff5698412 in eina_log_print_unlocked (domain=75, level=EINA_LOG_LEVEL_DBG, file=0x7fffe671eac0 "lib/elementary/efl_ui_focus_manager_calc.c", fnc=0x7fffe671f420 <__FUNCTION__.57453> "_request_subchild", line=1535,
fmt=0x7fffe671ec53 "Found node %p", args=0x7fffffffc760) at lib/eina/eina_log.c:1385
1385 {
(gdb) bt full
#0 0x00007ffff5698412 in eina_log_print_unlocked (domain=75, level=EINA_LOG_LEVEL_DBG, file=0x7fffe671eac0 "lib/elementary/efl_ui_focus_manager_calc.c", fnc=0x7fffe671f420 <__FUNCTION__.57453> "_request_subchild", line=1535, fmt=0x7fffe671ec53 "Found node %p", args=0x7fffffffc760) at lib/eina/eina_log.c:1385
d = <optimized out>
#1 0x00007ffff5699813 in eina_log_print (domain=75, level=level@entry=EINA_LOG_LEVEL_DBG, file=file@entry=0x7fffe671eac0 "lib/elementary/efl_ui_focus_manager_calc.c", fnc=fnc@entry=0x7fffe671f420 <__FUNCTION__.57453> "_request_subchild", line=line@entry=1535, fmt=fmt@entry=0x7fffe671ec53 "Found node %p") at lib/eina/eina_log.c:2259
args = {{gp_offset = 48, fp_offset = 48, overflow_arg_area = 0x7fffffffc870, reg_save_area = 0x7fffffffc780}}
#2 0x00007fffe66c264e in _request_subchild (node=<optimized out>) at lib/elementary/efl_ui_focus_manager_calc.c:1535
target = <optimized out>
#3 0x00007fffe66c27e7 in _request_subchild (node=<optimized out>) at lib/elementary/efl_ui_focus_manager_calc.c:1520
target = 0x0
n = <optimized out>
node = 0x555555bd0140
__FUNCTION__ = "_efl_ui_focus_manager_calc_unregister"
#4 0x00007fffe66c27e7 in _request_subchild_except (except=0x555555bd0140, n=<optimized out>) at lib/elementary/efl_ui_focus_manager_calc.c:1015
n = <optimized out>
node = 0x555555bd0140
__FUNCTION__ = "_efl_ui_focus_manager_calc_unregister"
#5 0x00007fffe66c27e7 in _efl_ui_focus_manager_calc_unregister (obj=0x400000006d2d, pd=0x555555a83a20, child=<optimized out>) at lib/elementary/efl_ui_focus_manager_calc.c:1042
n = <optimized out>
node = 0x555555bd0140
__FUNCTION__ = "_efl_ui_focus_manager_calc_unregister"
#6 0x00007fffe66c0b2f in efl_ui_focus_manager_calc_unregister (obj=0x40000000692c, child=child@entry=0x4000000a47a3) at lib/elementary/efl_ui_focus_manager_calc.eo.c:42
___op = 941
___generation = 1
___call =
{eo_id = 0x400000006d2d, obj = 0x555555a83990, func = 0x7fffe66c26a0 <_efl_ui_focus_manager_calc_unregister>, data = 0x555555a83a20, extn1 = 0x4000000259a8, extn2 = 0x400000000000, extn3 = 0xba, extn4 = 0x7fffe6716c00}
_func_ = <optimized out>
#7 0x00007fffe6679038 in _efl_ui_widget_focus_state_apply (obj=0x4000000a47a3, pd=<optimized out>, current_state=..., configured_state=0x7fffffffca50, redirect=<optimized out>) at lib/elementary/efl_ui_widget.c:423
registered = 1 '\001'
__FUNCTION__ = "_efl_ui_widget_focus_state_apply"
#8 0x00007fffe667865d in efl_ui_widget_focus_state_apply (obj=obj@entry=0x4000000a47a3, current_state=..., configured_state=configured_state@entry=0x7fffffffca50, redirect=redirect@entry=0x0)
at lib/elementary/efl_ui_widget.eo.c:186
_r = <optimized out>
___op = 714
___generation = 1
___call =
{eo_id = 0x4000000a47a3, obj = 0x555556536f20, func = 0x7fffe6678f20 <_efl_ui_widget_focus_state_apply>, data = 0x555556537170, extn1 = 0x0, extn2 = 0x7fffe66c65f2 <efl_ui_focus_object_focus_manager_get+114>, extn3 = 0x4000000259a8, extn4 = 0x55555627aaa0}
_func_ = <optimized out>
#9 0x00007fffe668073d in _focus_state_eval (want_full=<optimized out>, should=0 '\000', pd=0x555556537170, obj=0x4000000a47a3) at lib/elementary/efl_ui_widget.c:505
configuration = {manager = 0x0, parent = 0x0, logical = 0 '\000'}
old_registered_parent = 0x4000000259a8
old_registered_manager = 0x40000000692c
should = 0 '\000'
want_full = 0 '\000'
#10 0x00007fffe668073d in _full_eval (obj=0x4000000a47a3, pd=0x555556537170) at lib/elementary/efl_ui_widget.c:608
old_registered_parent = 0x4000000259a8
old_registered_manager = 0x40000000692c
should = 0 '\000'
want_full = 0 '\000'
#11 0x00007fffe667644f in efl_ui_widget_focus_allow_set (obj=0x4000000a47a3, can_focus=<optimized out>) at lib/elementary/efl_ui_widget.eo.c:34
___op = 676
___generation = 1
___call =
{eo_id = 0x4000000a47a3, obj = 0x555556536f20, func = 0x7fffe6687800 <_efl_ui_widget_focus_allow_set>, data = 0x555556537170, extn1 = 0x7fffcfb74668, extn2 = 0x555556916640, extn3 = 0xffffffffffffffff, extn4 = 0x0}
_func_ = <optimized out>
#12 0x00007fffe667e509 in elm_widget_can_focus_set (obj=<optimized out>, can_focus=<optimized out>) at lib/elementary/efl_ui_widget.eo.c:461
#13 0x00007fffe65fd8be in elm_object_focus_allow_set (obj=<optimized out>, enable=<optimized out>) at lib/elementary/elm_main.c:1614
__FUNCTION__ = "elm_object_focus_allow_set"
#14 0x00007fffe6cc9436 in __pyx_pf_3efl_10elementary_8__init___6Object_11focus_allow_2__set__ (__pyx_v_self=0x7fffdb3a30e8, __pyx_v_allow=<optimized out>) at efl/elementary/__init__.c:273152
__pyx_r = <optimized out>
#15 0x00007fffe6cc9436 in __pyx_pw_3efl_10elementary_8__init___6Object_11focus_allow_3__set__ (__pyx_v_allow=<optimized out>, __pyx_v_self=0x7fffdb3a30e8) at efl/elementary/__init__.c:10987
#16 0x00007fffe6cc9436 in __pyx_setprop_3efl_10elementary_8__init___6Object_focus_allow (o=0x7fffdb3a30e8, v=<optimized out>, x=<optimized out>) at efl/elementary/__init__.c:11799
#17 0x00007ffff724cf7c in _PyObject_GenericSetAttrWithDict () at /usr/lib/libpython3.6m.so.1.0I gave up on using the elm focus in my media center some weeks ago, I wrote a custom focus handling instead.
So I don't have anymore a test for all the issues in this ticket