Code review comment for ~3v1n0/ubuntu/+source/gnome-shell:ubuntu/master-xubuntu-cancel-search

Revision history for this message
Marco Trevisan (TreviƱo) (3v1n0) wrote :

> I saw that you made some changes on the MR after Florian's comment on your
> patches. Is there anything you can borrow and refresh here?

Yes, I wanted to refresh in case a review (like in this case) was coming in time, so nothing functional has been changed there, but I can refresh it on that.

As per the two questions, let me know if you want get rid of that variable or use a property bound to the cancellable instead.

« Back to merge proposal