Bisected to commit 253430ab76348d99c81e3b753b6a32817bdb6666 as the start of the problem. Changed Focus Manager back to recursively calcing subobjects.
Details
- Reviewers
bu5hm4n zmike devilhorns
Scrollable popup worked before this commit and did not after. Applying this patch allows it to work again.
Diff Detail
- Repository
- rEFL core/efl
- Branch
- master
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 6560 Build 6754: arc lint + arc unit
Upon further testing this only fixes the elm_test scrollable popup case and does not fix Ephoto's scrollable popup in settings.
Whatever the reason is for this is, its wrong. we have to call _next here, as otherwise we dont explore the full tree but skip most of the nodes in the tree.
Also, how is this supposed to change anything in regards of placing and rendering ?
After a PM from @stephenmhouston, this patch is not intended to "fix" the issue, it's an attempt to provide more info since he's going afk for a period of time. Perhaps this is in some way helpful to you?
Okay, i wasn't aware of that, i thought you want to have that in master ... :)
Sorry, I am sometimes a bit of a hothead :)