Bug 216491

Install mplayerthumbs package and all will be fine...

Video thumbs generation is delegate to KDE framework.

Gilles Caulier

View option only play with image, using editor. It doesn't play movies for the moment, using Phonon...

Gilles Caulier

1. ok thanks i made this report because
in digikam if you select a movie then click "view" button then this launch the movie player

thus
i was expected same feature in kameraklient

it's inconsistent

2. i installed mplayerthumbs and mplayer and win32 codecs

then
in dolphin now i have a preview thumbnail in the right panel
but no preview thumbnail icon for the file in dolphin or in kameraclient
just the standard movie icon

ok now i understand your principles

is it possible to open a film editor if one is installed ?

ok

now i have preview icon for movie everywhere
- in dolphin the file list and in the right panel
- in digikam in window showing album contents

but not in camera gui
 is it a kde camera protocol pb or a digikam pb ?

How do you have fixed movies icons preview with maplyersthumbs. With new Mandriva 2010.0, no thumbs are generated for video in icon view ?

About cameragui, it can be relevant of your camera driver : Gphoto do not support video thumb extraction. With UMS camera and digiKam driver, no problem.

Gilles

ok thanks for explanation

for complete activation of preview icon for movie

http://forum.mandriva.com/viewtopic.php?t=121288

sorry it's in french

i complete the pb definition (see description)

the default filter is "all images" it must be "all images and movies"

you can select movie by typing *.* as default filter

digikam 1.0.0
kipi-pluigins 1.0.0

pb still there

you cannot select a movie due to filter "all images"
and
when you use *.* filter you can select a movie
but there is no preview for movie (yes there is preview in dolphin)

for no movie preview in dolphin selector i assume this is a dolphin pb not a digikam one


see comment #8

ignore my comment #8 and 9
it's about another bug

digikam 1.0.0
kipi-plugins 1.0.0

pb still there : for a movie an image application is launched instead of a movie editor

digikam 1.2
kipi-plugins 1.2

pb still there : for a movie an image application is launched instead of a
movie editor or a movie viewer

*** Bug 225151 has been marked as a duplicate of this bug. ***

Git commit 593806836c8c26a9df718394ffc39fd3cb639d37 by Islam Wazery.
Committed on 23/08/2012 at 04:42.
Pushed by wazery into branch 'development/3.0.0'.

Added mediaplayer view to import interface

M  +1    -0    utilities/importui/CMakeLists.txt
M  +4    -11   utilities/importui/views/importiconview.cpp
M  +1    -3    utilities/importui/views/importiconview_p.h
M  +28   -34   utilities/importui/views/importstackedview.cpp
M  +2    -2    utilities/importui/views/importstackedview.h
M  +1    -1    utilities/importui/views/importview.cpp
A  +306  -0    utilities/importui/views/mediaplayerview.cpp     [License: GPL (v2+)]
A  +107  -0    utilities/importui/views/mediaplayerview.h     [License: GPL (v2+)]

http://commits.kde.org/digikam/593806836c8c26a9df718394ffc39fd3cb639d37

I implemented a mediaplayer preview for import tool, I will fix the issue of movies thumbnail loading ASAP.

thanks

Islam,

I take a look to your ImportUI mediaplayerview implementation and a lots of code are redondant with AlbumGUI mediaplayerview.

We need to factorize this code. The difference is that ImportUI use CameraInfo where AlbumGUI use ImageInfo containers.

But the code to wrap Phonon widget to host GUI is the same.

What do you think about ?

Gilles Caulier

Git commit d38b2e188ddde0e04b5526847829889ba463ab09 by Gilles Caulier.
Committed on 23/08/2012 at 12:54.
Pushed by cgilles into branch 'development/3.0.0'.

base MediaPlayer class on KUrl, not ImageInfo or CameraImageInfo, for future factoring

M  +4    -6    digikam/views/mediaplayerview.cpp
M  +3    -8    digikam/views/mediaplayerview.h
M  +3    -3    digikam/views/stackedview.cpp
M  +3    -3    utilities/importui/views/importstackedview.cpp
M  +4    -6    utilities/importui/views/mediaplayerview.cpp
M  +3    -7    utilities/importui/views/mediaplayerview.h

http://commits.kde.org/digikam/d38b2e188ddde0e04b5526847829889ba463ab09

Thanks Gilles for factorizing it, I reused albumgui mediaplayerview now.

Islam,

I think a similar factoring can be done with ImagePreviewView and ImportPreviewView.

If you are agree, i can take a look...

Gilles Caulier

Sure Gilles, If you have more important thing to do I can handle it.

I will review it both classes. there is always code to check and polish. I will give you a feedback accordingly...

Gilles

Both class can be factored. Not sure that all methods can be considerated as common, like with mediaplayer classes. This depend of which features can be implemented in ImportUI as AlbumGUI, as face tag for ex.

At least, i recommend to factorize all that you can in a common and parents class, and just implement necessary code to childrens classes.

Question : what's about image rotation in ImportUI ? It have a sense to rotate item as well in Import icon--iew and preview mode, as it will done automatically at download time ? Also, to rotate item using FileActionMngr with GPhoto2 camera will not work (excepted if you download it previously and upload back to camera (:=))))

Gilles Caulier

Git commit bb09c7272caa0dddb7556ce8d96ecad8ffd27f6e by Islam Wazery.
Committed on 24/08/2012 at 16:18.
Pushed by wazery into branch 'development/3.0.0'.

Removed view item option (alternate is preview mode) and fixed thumbnail issue for files other than images.
Now all files other than images have an icon representing their types,
the video files with thumbnails "THM files" are loaded correctly.
Is that enough for this B.K.O?

M  +6    -60   utilities/importui/backend/cameracontroller.cpp
M  +1    -3    utilities/importui/backend/cameracontroller.h
M  +29   -29   utilities/importui/items/importdelegate.cpp
M  +1    -32   utilities/importui/main/cameraui.cpp
M  +0    -3    utilities/importui/main/cameraui.h
M  +0    -1    utilities/importui/main/cameraui.rc
M  +0    -2    utilities/importui/main/cameraui_p.h
M  +0    -1    utilities/importui/views/importiconview.cpp
M  +0    -1    utilities/importui/views/importpreviewview.cpp
M  +1    -1    utilities/importui/views/importstackedview.cpp

http://commits.kde.org/digikam/bb09c7272caa0dddb7556ce8d96ecad8ffd27f6e

Philippe can you test the new import tool from development/3.0.0 branch and tell me if this report is still valid.

I think this report can be closed now.