Bug 468169

Summary:	Option to delete or to edit flatpak permissions added by the user
Product:	[Applications] systemsettings	Reporter:	postix <postix>
Component:	kcm_flatpak	Assignee:	Plasma Bugs List <plasma-bugs-null>
Status:	RESOLVED NOT A BUG
Severity:	wishlist	CC:	joshiesuhaas0, me, nate
Priority:	NOR	Keywords:	usability
Version First Reported In:	5.27.3
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description postix 2023-04-05 12:37:13 UTC

SUMMARY

You can add new permissions with the button "+ Add new" and disable / enable those as well as reset all made changes.
However, it is not possible to edit or to delete an added rule. It would be great if that was possible.

SOFTWARE/OS VERSIONS
Operating System: openSUSE Tumbleweed 20230403
KDE Plasma Version: 5.27.3
KDE Frameworks Version: 5.104.0

Comment 1 ratijas 2023-04-05 20:02:25 UTC

Unchecking them would mark them for deletion. It will work the same way for Filesystems, D-Bus services and Environment variables, once https://invent.kde.org/plasma/flatpak-kcm/-/merge_requests/91 is merged. Additionally I want to replace checkboxes with icon-only "delete" buttons, and markup text with strike-through attribute and maybe faded text color.

Comment 2 postix 2023-04-06 10:52:54 UTC

> Additionally I want to replace checkboxes with icon-only "delete" buttons,
> and markup text with strike-through attribute and maybe faded text color.

But would your solution allow to temporarily disable a rule? For instance to check if this rule interfered with something or to temporarily disable access? If those rules got deleted, one would need to note down those before deletion in order to add them back later.

Comment 3 ratijas 2023-04-06 12:47:43 UTC

With the current implementation (and the one that's coming to replace it), they won't get actually removed from the list view until you switch pages.  There is (syntactically) no way to preserve them in config between switches or restarts though. The model writes to a config everything that's enabled and actually differs from defaults.

Comment 4 ratijas 2023-04-20 13:41:24 UTC

With https://invent.kde.org/plasma/flatpak-kcm/-/merge_requests/91 merged, all sections and permissions are fully and correctly editable, and custom entries can be unchecked to mark them for deletion. The UI is currently not great about explaining that to a user, but that would be a topic for a follow-up UI improvement, not a bug fix.

Persisting "temporary" deletions of custom entries is not supported nor planned, sorry. Just uncheck them, hit Apply, run your app, and check & Apply again — all while not closing the KCM window.