Page MenuHomePhabricator

efl_ui_position_manager: a way to announce new entities

Authored by cedric on Sep 15 2019, 11:37 PM.



there are situations where the entity is not ready yet when the initial
placing does happen. With this API you can tell the position manager
that the placing of the items can be reapplied at the entities are
availble now.

Depends on D10033

Diff Detail

rEFL core/efl
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
cedric created this revision.Sep 15 2019, 11:37 PM

I would say this looks fine ... but i am the author :-D

zmike requested changes to this revision.Sep 16 2019, 6:17 AM

pls run your code through spellcheck before submitting



This revision now requires changes to proceed.Sep 16 2019, 6:17 AM

It seems phab doesn't preserve the git author. It should be correct in the branch I pushed and we should land from there.

cedric updated this revision to Diff 25190.Sep 19 2019, 1:14 PM
cedric edited the summary of this revision. (Show Details)

rebase and take comment into account.

zmike resigned from this revision.Sep 19 2019, 1:43 PM
This revision was not accepted when it landed; it landed in state Needs Review.Sep 19 2019, 2:42 PM
Closed by commit rEFLfc935e99d936: efl_ui_position_manager: a way to announce new entities (authored by Marcel Hollerbach <>, committed by cedric). · Explain Why
This revision was automatically updated to reflect the committed changes.

I do not understand how this commit has landed when you say "Depends on D10033" which is still being reviewed.
If build didn't break, this means this patch didn't really depend on the other one, right?
Building the right patch dependency tree would simplify reviewing and generate less phab mail flood...