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/677244fd28dc3955b888572ca16ca46746b0b652 -- View it on GitLab: https://dev.gajim.org/gajim/gajim/-/commit/677244fd28dc3955b888572ca16ca46746b0b652 You're receiving this email because of your account on dev.gajim.org.
_______________________________________________ Commits mailing list Commits@gajim.org https://lists.gajim.org/cgi-bin/listinfo/commits