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


Commits:
de5a18a7 by wurstsalat at 2025-02-01T16:39:28+01:00
cfix: Profile: Fix leak

- - - - -


1 changed file:

- gajim/gtk/profile.py


Changes:

=====================================
gajim/gtk/profile.py
=====================================
@@ -262,8 +262,6 @@ def _on_back_clicked(self, *args: Any) -> None:
         self._show_profile_page()
 
     def _on_save_clicked(self, _button: Gtk.Button) -> None:
-        self._ui.spinner.start()
-        self._ui.profile_stack.set_visible_child_name("spinner")
         self._ui.add_entry_button.hide()
         self._ui.cancel_button.hide()
         self._ui.save_button.hide()
@@ -272,6 +270,11 @@ def _on_save_clicked(self, _button: Gtk.Button) -> None:
         self._ui.privacy_button.hide()
         self._ui.nickname_entry.set_sensitive(False)
 
+        # Switch page after setting other widget's visibility
+        # This avoids a leak caused by switching stack pages
+        self._ui.spinner.start()
+        self._ui.profile_stack.set_visible_child_name("spinner")
+
         self._vcard_grid.validate()
         self._vcard_grid.sort()
 



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

-- 
View it on GitLab: 
https://dev.gajim.org/gajim/gajim/-/commit/de5a18a78dfe2b4a4d0e8fc4199d299ce8f709e8
You're receiving this email because of your account on dev.gajim.org.


_______________________________________________
Commits mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to