search

mate-desktop / mate-screensaver

MATE screen saver and locker
https://mate-desktop.org
GNU General Public License v2.0
48 stars 40 forks source link

gs-theme-manager: reload on menu_tree changed #262

Closed rbuj closed 2 years ago

rbuj commented 2 years ago

closes #263

  /**
   * MateMenuTree:changed:
   *
   * This signal is emitted when applications are added, removed, or
   * upgraded.  But note the new data will only be visible after
   * matemenu_tree_load_sync() or a variant thereof is invoked.
   */
  matemenu_tree_signals[CHANGED] =
      g_signal_new ("changed",
                    G_TYPE_FROM_CLASS (klass),
                    G_SIGNAL_RUN_LAST,
                    0,
                    NULL, NULL,
                    g_cclosure_marshal_VOID__VOID,
                    G_TYPE_NONE, 0);
joakim-tjernlund commented 2 years ago

Merge and release a new screensaver? The current version is "lethal" as it blanks the monitor and forces reboot in most cases.

raveit65 commented 2 years ago

Merge and release a new screensaver? The current version is "lethal" as it blanks the monitor and forces reboot in most cases.

Zauberwort? :)

Testing and confirming that this PR fixes your issues would speed up merge and release process.

joakim-tjernlund commented 2 years ago

Been using this patch a few days and no crash. I cannot force the crash though so not 100% sure that my crash is fixed.