martyr-deepin / deepin-screenshot

Deprecated (migrated into deepin-screen-recorder after v20), consider create and maintain your own fork if you love it.
https://www.deepin.org/original/deepin-screenshot/
GNU General Public License v3.0
346 stars 102 forks source link

The -s and -n options can now be used together. fixes#22 #60

Open mblesel opened 5 years ago

mblesel commented 5 years ago

When using the -s and -n options together the -n option was ignored before. I added a noNotify argument to the Screenshot::savePathScreenshot and the MainWindow::savePath functions. MainWindow::savePath now sets MainWindow::m_noNotify if the program was called with -n option. Lastly MainWindow::saveSpecificedPath checks m_noNotify before sending a notification.