Bug 380359

Summary: Shouldn't "Email Images" be in "Export" menu instead of "Tools" menu ?
Product: [Applications] digikam Reporter: Andrius <aegoreev>
Component: Plugin-Generic-SendByMailAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: minor CC: caulier.gilles
Priority: NOR    
Version: 5.6.0   
Target Milestone: ---   
Platform: Appimage   
OS: Linux   
Latest Commit: Version Fixed In: 5.7.0
Sentry Crash Report:

Description Andrius 2017-05-30 15:56:57 UTC
Subject. I got used to it but I remember the very first time I was looking for it in Export menu. Just a suggestion.
Comment 1 caulier.gilles 2017-06-28 21:39:36 UTC
Note : Export menu is delegate to send image to remote web service. This task is delegate to kipi tools. A new "Send by Mail" tool have been written instead old kipi tool and will include a lots of improvements. The first stage is to provide an assistant dialog. Later it will support video files.

Gilles Caulier
Comment 2 caulier.gilles 2017-06-28 21:42:11 UTC
Git commit 149b957b020a357feb279fbb44ae51638d9c1184 by Gilles Caulier.
Committed on 28/06/2017 at 21:39.
Pushed by cgilles into branch 'master'.

New core tool dedicated to send items by email.
This tool is an assistant dialog and remplace old kipi tool SendImages.
Later, this tool will be available in image editor, Light table and Showfoto.
It will able to process video too...
FIXED-IN: 5.7.0

M  +1    -0    app/CMakeLists.txt
M  +9    -0    app/main/digikamapp.cpp
M  +4    -3    app/main/digikamapp.h
M  +3    -3    app/main/digikamapp_p.h
M  +2    -1    app/main/digikamui5.rc
M  +12   -0    libs/widgets/mainview/dxmlguiwindow.cpp
M  +8    -0    libs/widgets/mainview/dxmlguiwindow.h
M  +15   -0    tests/assistants/CMakeLists.txt
A  +64   -0    tests/assistants/sendbymail.cpp     [License: GPL (v2+)]
M  +1    -0    utilities/assistants/CMakeLists.txt
A  +36   -0    utilities/assistants/sendbymail/CMakeLists.txt
A  +1    -0    utilities/assistants/sendbymail/TODO
A  +183  -0    utilities/assistants/sendbymail/manager/imageresizejob.cpp     [License: GPL (v2+)]
A  +77   -0    utilities/assistants/sendbymail/manager/imageresizejob.h     [License: GPL (v2+)]
A  +93   -0    utilities/assistants/sendbymail/manager/imageresizethread.cpp     [License: GPL (v2+)]
A  +64   -0    utilities/assistants/sendbymail/manager/imageresizethread.h     [License: GPL (v2+)]
A  +662  -0    utilities/assistants/sendbymail/manager/mailprocess.cpp     [License: GPL (v2+)]
A  +102  -0    utilities/assistants/sendbymail/manager/mailprocess.h     [License: GPL (v2+)]
A  +138  -0    utilities/assistants/sendbymail/manager/mailsettings.cpp     [License: GPL (v2+)]
A  +119  -0    utilities/assistants/sendbymail/manager/mailsettings.h     [License: GPL (v2+)]
C  +28   -17   utilities/assistants/sendbymail/wizard/mailalbumspage.cpp [from: utilities/assistants/videoslideshow/wizard/vidslidealbumspage.cpp - 067% similarity]
A  +57   -0    utilities/assistants/sendbymail/wizard/mailalbumspage.h     [License: GPL (v2+)]
A  +180  -0    utilities/assistants/sendbymail/wizard/mailfinalpage.cpp     [License: GPL (v2+)]
A  +63   -0    utilities/assistants/sendbymail/wizard/mailfinalpage.h     [License: GPL (v2+)]
A  +124  -0    utilities/assistants/sendbymail/wizard/mailimagespage.cpp     [License: GPL (v2+)]
A  +60   -0    utilities/assistants/sendbymail/wizard/mailimagespage.h     [License: GPL (v2+)]
A  +127  -0    utilities/assistants/sendbymail/wizard/mailintropage.cpp     [License: GPL (v2+)]
A  +56   -0    utilities/assistants/sendbymail/wizard/mailintropage.h     [License: GPL (v2+)]
A  +294  -0    utilities/assistants/sendbymail/wizard/mailsettingspage.cpp     [License: GPL (v2+)]
A  +61   -0    utilities/assistants/sendbymail/wizard/mailsettingspage.h     [License: GPL (v2+)]
A  +147  -0    utilities/assistants/sendbymail/wizard/mailwizard.cpp     [License: GPL (v2+)]
A  +64   -0    utilities/assistants/sendbymail/wizard/mailwizard.h     [License: GPL (v2+)]
M  +17   -6    utilities/assistants/videoslideshow/wizard/vidslidealbumspage.cpp
M  +3    -0    utilities/kdesupport/kipi/kipipluginloader.cpp

https://commits.kde.org/digikam/149b957b020a357feb279fbb44ae51638d9c1184