Code review comment for lp:~bilalakhtar/unity-lens-applications/fix-750262

Revision history for this message
Mikkel Kamstrup Erlandsen (kamstrup) wrote :

Awesome Bilal! I saw your ping on IRC yesterday just when my laptop's battery died. I still think that AppInfo.launch_default_for_uri() is the right choice. The apt:foo thing is mainly for power users and I am not convinced they appreciate a full S-C launched.

And using the GNOME URI handling we give the platform team the option to install the software-center as apt: URI handler if they see that as better. Which hardcoding S-C doesn't make room for.

Your patch will be included in todays tarball.

review: Approve

« Back to merge proposal