Bug 446420

Summary:	"Unknown protocol" is not friendly enough for 1-point user
Product:	[Applications] gwenview	Reporter:	Fushan Wen <qydwhotmail>
Component:	importer	Assignee:	Gwenview Bugs <gwenview-bugs-null>
Status:	RESOLVED FIXED
Severity:	wishlist	CC:	bugseforuns, nate
Priority:	NOR	Keywords:	usability
Version:	Git (add output of "git log -1 --oneline" to description)
Target Milestone:	---
Platform:	openSUSE
OS:	Linux
See Also:	https://bugs.kde.org/show_bug.cgi?id=446274
Latest Commit:	https://invent.kde.org/graphics/gwenview/commit/5719e09b588c3bdaa466e1c64b2aaa1a16289605	Version Fixed In:	22.04

Description Fushan Wen 2021-12-03 14:02:24 UTC

SUMMARY
When kamera is not installed, the importer will report "Unknown protocol" error and leaves the dialog empty without any further instruction. The behavior is not friendly for 1-point user.


STEPS TO REPRODUCE
1. Make sure kamera is not installed
2. Import from a camera

OBSERVED RESULT
"Unknown protocol" error is shown and the dialog is empty.

EXPECTED RESULT
There should be some instructions to guide the user to install the protocol support.

SOFTWARE/OS VERSIONS
Information for package gwenview5:
----------------------------------
Repository     : unstable-kde-applications
Name           : gwenview5
Version        : 22.03.70git.20211130T184007~3d0f70c8-ku.28.1
Arch           : x86_64
Vendor         : obs://build.opensuse.org/KDE:Unstable
Installed Size : 3.8 MiB
Installed      : Yes
Status         : up-to-date
Source package : gwenview5-22.03.70git.20211130T184007~3d0f70c8-ku.28.1.src
Summary        : Image Viewer by KDE

Comment 1 Bug Janitor Service 2021-12-03 14:22:05 UTC

A possibly relevant merge request was started @ https://invent.kde.org/graphics/gwenview/-/merge_requests/121

Comment 2 Fushan Wen 2021-12-16 08:45:26 UTC

Git commit 5719e09b588c3bdaa466e1c64b2aaa1a16289605 by Fushan Wen.
Committed on 14/12/2021 at 11:49.
Pushed by fusionfuture into branch 'master'.

importer: Show "Install Protocol Support" button when protocol support is missing

Currently, when the protocol support is not installed (e.g. kamera),
the importer will report "Unknown protocol" error and leave
the dialog empty without any further instruction. The behavior
is not friendly for 1-dot users as mentioned in the Nate's
blog article.

This commit adds a button named "Install protocol support for %1". The
button is only visible when `DocumentDirFinder` emits
`KIO::Error::ERR_CANNOT_CREATE_SLAVE` error. The button can

1. open Discover and directly open kamera page if the scheme is
   "camera".
2. open Discover and search the scheme for other schemes.

To make `DocumentDirFinder` able to emit error signal, a new signal
named `error()` is added to `DocumentDirFinder`.

M  +2    -0    importer/CMakeLists.txt
M  +10   -0    importer/documentdirfinder.cpp
M  +1    -0    importer/documentdirfinder.h
M  +156  -0    importer/thumbnailpage.cpp
M  +2    -0    importer/thumbnailpage.h

https://invent.kde.org/graphics/gwenview/commit/5719e09b588c3bdaa466e1c64b2aaa1a16289605