Details
Details
- Reviewers
- None
- Commits
- R134:0376a9727e53: Support snap permissions in Discover
Diff Detail
Diff Detail
- Repository
- R134 Discover Software Store
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
+1 for the feature, but could we use checkboxes instead of toggles? I'm not real thrilled with the ones in those screenshots. It's not at all clear what's on and what's off.
Comment Actions
Also those labels are not very descriptive; are there better strings available? Looking at the screenshots, I have no idea what most of those permissions actually do. A regular user would be even more confused.
Comment Actions
Address issues by upstreap snapdqt maintainer Robert Ancell.
Change to show a check box instead of the slider.
Comment Actions
New in Phabricator. Is this code already merged? Using Kubuntu 18.04 with Backports PPA but I can't see these options.
Comment Actions
Kubuntu 18.04 uses Plasma 5.12 which includes Discover 5.12. This code was added in a later version (likely 5.13).
Comment Actions
That makes sense:
$ dpkg -l | grep discover ii avahi-utils 0.7-3.1ubuntu1.2 amd64 Avahi browsing, publishing and discovery utilities ii plasma-discover 5.12.8-0ubuntu0.1build1~ubuntu18.04~ppa1 amd64 Discover software management suite ii plasma-discover-common 5.12.8-0ubuntu0.1build1~ubuntu18.04~ppa1 all Discover software manager suite (common data files) ii plasma-discover-snap-backend 5.12.8-0ubuntu0.1build1~ubuntu18.04~ppa1 amd64 Discover software manager suite - Snappy support
Thanks.