Daniel Brötzmann pushed to branch master at gajim / gajim

Commits: 677244fd by wurstsalat at 2023-02-17T14:20:48+01:00 imprv: PEPConfig: Ask for confirmation when deleting nodes

- - - - -

1 changed file:

- gajim/gtk/pep_config.py

Changes:

===================================== gajim/gtk/pep_config.py ===================================== @@ -34,6 +34,8 @@

from gajim.gtk.builder import get_builder from gajim.gtk.dataform import DataFormWidget +from gajim.gtk.dialogs import ConfirmationDialog +from gajim.gtk.dialogs import DialogButton from gajim.gtk.dialogs import ErrorDialog from gajim.gtk.dialogs import WarningDialog from gajim.gtk.util import EventHelper @@ -143,15 +145,26 @@ def _on_delete_button_clicked(self, _widget: Gtk.Button) -> None: selection = self._ui.services_treeview.get_selection() if not selection: return + model, iter_ = selection.get_selected() assert isinstance(model, Gtk.ListStore) assert iter_ node = model[iter_][0]

- self._client.get_module('PubSub').delete( - node, - callback=self._on_node_delete, - user_data=node) + def _delete(): + self._client.get_module('PubSub').delete( + node, + callback=self._on_node_delete, + user_data=node) + + ConfirmationDialog( + _('Delete'), + _('Delete PEP node?'), + _('Do you really want to delete this PEP node?'), + [DialogButton.make('Cancel'), + DialogButton.make('Delete', + callback=_delete)], + transient_for=self).show()

def _on_node_delete(self, task: Task) -> None: node = task.get_user_data()

View it on GitLab: https://dev.gajim.org/gajim/gajim/-/commit/677244fd28dc3955b888572ca16ca4674...