Skip to content
  • Kai Uwe Broulik's avatar
    Remove visibilityChanged connection in favor of existing eventFilter · d4af86d8
    Kai Uwe Broulik authored
    KMainWindow compresses saving its configuration by a 500ms timer. However, for QDockWidget visibilityChanged
    a direct connection to the setSettingsDirty method is made, which immediately saves settings.
    Since there is already an eventFilter also listening for QEvent::Show and QEvent::Hide, the connection to
    visibilityChanged is redundant since that signal is emitted from the respective event handler anyway.
    The main benefit is that the eventFilter code compresses the configuration save,
    not jeopardizing startup time for applications with lots of dock widgets like Dolphin.
    
    Differential Revision: https://phabricator.kde.org/D22523
    d4af86d8