Apparently there are cases where efl_ui_focus_object_focus_manager_get
can return NULL. In order to trap for this, we can simply modify the
do loop slightly and check for a valid manager before we actually
start looping
Test Case: elementary_test -to "Panel Scrollable" and click Toggle
button
ref T7030
Depends on D6703
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
It seems that this patch has no reviewers specified. If you are unsure who can review your patch, please check this wiki page and see if anyone can be added: https://phab.enlightenment.org/w/maintainers_reviewers/
Noep this breaks setting focus to objects that are not visible yet.
the code from line 46 to 52 must be executed.
| src/lib/elementary/efl_ui_focus_util.c | ||
|---|---|---|
| 33 | Would you prefer a check here to make sure that 'm' is valid ? | |
Sorry, completly missed your reply.
Either that, or move the if(!m) block out of the while (after the loop body), that should also do it :)
Okay, deeply checked this. There seems to be a bug when focusing gengrid manually.
https://phab.enlightenment.org/P230
That function body works for me, does that also work for you ?
(Sorry for pinging arround on this one)