Code review comment for lp:~jjed/unity/tooltips-redux

Revision history for this message
Jjed (jjed) wrote :

Okay, changed most of that.

> 371 + void SetHover(bool on_launcher);
>
> What about a nux::Property<bool> for this?

I don't think that makes sense, given that `hover_` is private and the TooltipMannager doesn't need any internal hooks for when it changes. LauncherHoverMachine already has a signal for changing hover-state as well.

« Back to merge proposal