[Digikam-devel] [Bug 131034] New: Display a mini-review of the photo currently transfered, i.e better visual feedback

[FACORAT Fabrice]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=131034         
           Summary: Display a mini-review of the photo currently transfered,
                    i.e better visual feedback
           Product: digikam
           Version: unspecified
          Platform: Mandriva RPMs
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: wishlist
          Priority: NOR
         Component: general
        AssignedTo: digikam-devel kde org
        ReportedBy: f.faber-pro ifrance com


Version:            (using KDE KDE 3.5.3)
Installed from:    Mandriva RPMs
OS:                Linux

When importing photo from a digital camera, digikam will just use the filename to tell which photo is currently downloaded and the %.
What could be interesting is to display the image ( a mini-review/icon done from digikam generated preview ) next to the filename. Indeed for the user the image/photo have more meaning than the filename ( which is most of the times somewhat "cryptic" ). This will improve visual feedback for the user.

Another interesting feature should be also to add a visual feedback in the image/photo preview mode/pane. For example when an image is completly downloaded/imported, it becomes greyed. We could also think about an icon being added ( at the top of the photo for example ) to tell that the image have been imported.

Even more crazy : show image download progress directly on the photo in the preview pane by greying x% of the image depending on the download/import progression
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[FACORAT Fabrice]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=131034         




------- Additional Comments From f.faber-pro ifrance com  2006-07-18 20:04 -------
Created an attachment (id=17029)
 --> (http://bugs.kde.org/attachment.cgi?id=17029&action=view)
mockup ( image taken from showcast of digikam 0.9 ) showing the preview of the
currently imported image

mockup ( image taken from showcast of digikam 0.9 ) showing the preview of the
currently imported image.

I take a random image because the right image was not completly visible. The
purpose is to illustrate what I mean
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=131034         




------- Additional Comments From caulier.gilles free fr  2006-07-19 15:03 -------
Salut Fabrice,

I fact digiKam camera gui implementation provide already a solution show witch item have already downloaded or not. The code have been never finalized by Renchi Raju (the original author of digiKam).

I have complete this job and also improved this one :

- camera gui will show a little icon on the top/right of item :

==> a star when it a non-downloaded item.
==> a blue gear when it's currenlty under download.
==> a green sign when it's downloaded sucessfuly.
==> a red cross if it cannot be downloaded from camera.

- During download, the current item in the icon view is always visible (if you have a big list of picture).
- Bonus : camera gui now support digiKam theme !
- The cameragui C++ object implementation have been very polished again, especially for some object where i haven't already checked the code. All object use a d private class to speed compilation. I have found also some little indeep bugs.

There is a screenshot at this url :

http://digikam3rdparty.free.fr/Screenshots/camera_gui_with_theme_and_icon_download_status.png

Attached a patch to close this B.K.O file (:=)))... I haven't yet commited the code in svn because digiKam is under finalization for 0.9.0-beta1 release. I would waiting than Marcel (an another digiKam developer) complete this release.

Also, this patch have been only tested with an UMS camera, not yet with a gphoto2 camera. Normally this is not a problem, but i want delayed a little the svn intergartion to have a feedback from users.

Bon tests (:=)))

Gilles Caulier
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=131034         




------- Additional Comments From caulier.gilles free fr  2006-07-19 15:05 -------
Created an attachment (id=17034)
 --> (http://bugs.kde.org/attachment.cgi?id=17034&action=view)
patch to improve camera gui
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=131034         
caulier.gilles free fr changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|general                     |Camera GUI
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=131034         




------- Additional Comments From caulier.gilles free fr  2006-07-19 17:21 -------
Created an attachment (id=17035)
 --> (http://bugs.kde.org/attachment.cgi?id=17035&action=view)
second version of cameragui improvement patch !!!

In this version the theme support in camera gui is complete (missing camera
icon item theme colored)
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=131034         
caulier.gilles free fr changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #17034|0                           |1
        is obsolete|                            |
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=131034         




------- Additional Comments From mikmach wp pl  2006-07-19 22:45 -------
Dnia �roda, 19 lipca 2006 15:04, Gilles Caulier napisa�:
> ==> a star when it a non-downloaded item.


That could be really useful if Digikam could detect if image was
previously downloaded. But this is a dream. Real comparing could be done
only when comparing MD5 sums of images which would be killing for
performance.

Also use of star may be misleading: it is used
for rating in main window.

Solutions:

- no icon at all
- use another icon - big yellow dot?

> ==> a blue gear when it's currenlty under download.
> ==> a green sign when it's downloaded sucessfuly.
> ==> a red cross if it cannot be downloaded from camera.


> Also, this patch have been only tested with an UMS camera, not yet with
> a gphoto2 camera. Normally this is not a problem, but i want delayed a
> little the svn intergartion to have a feedback from users.


Also tested only with UMS camera (2nd ver.).
Works very nice.

_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Colin Guthrie-6]

>> ==> a star when it a non-downloaded item.
>
> Also use of star may be misleading: it is used
> for rating in main window.
>
> Solutions:
>
> - no icon at all
> - use another icon - big yellow dot?

How about a no-entry sign?

http://www.health-safety-signs.uk.com/productimages/No-Entry-outdoor-sign.gif


It's not a cross which means failed, so hopefully it'll convey the fact
that it was deliberatly not downloaded?

Just a random thought.

Col.

_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=131034         
caulier.gilles free fr changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |marcus jet franken de



------- Additional Comments From caulier.gilles free fr  2006-07-20 07:22 -------
Normaly, if i have understand correctly the code, Gphoto2 camera set a flag to the camera when a picture have downloaded.

Marcus, can you give me some hints about this point ?

MD5 checksum is out of this B.K.O file. There is already a file about this subject in B.K.O (#107316). It's more complicated to do and is only require by UMS camera (witch don't use Gphoto2 drivers in fact).

Gilles
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Marcus Meissner-4]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=131034         




------- Additional Comments From marcus jet franken de  2006-07-20 08:33 -------
Well, it would be possible in generalal if the libgphoto instance is open,
but this is not a common use case ;)

There is camera driver support (info.file.status == GP_FILE_STATUS_DOWNLOADED)
when (info.file.fields & GP_FILE_INFO_STATUS) is set, but only the Canon
camera driver supports this.
Other camera classes do not store such information.

Suggestions:
- Check name, filesize and filetime ... those are querieable from the camera
  without retrieving the full image.
- You could also retrieve EXIF information and compare this blob with the
  one on-disk already.

(to be really really sure you have to check md5sums though ;)
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=131034         




------- Additional Comments From caulier.gilles free fr  2006-07-20 08:51 -------
Thanks Marcus for these informations.

Well a common solution will be an MD5 checksum stored somewhere in database for example. If we will use the database to store an items download status for each camera, the database structure need to be changed. I would do it later 0.9.0

Marcel, what do you think to use database for that ? Do you know another solution ?

Gilles
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

Dnia czwartek, 20 lipca 2006 00:20, Colin Guthrie napisał:

> >> ==> a star when it a non-downloaded item.
> >
> > Also use of star may be misleading: it is used
> > for rating in main window.
> >
> > Solutions:
> >
> > - no icon at all
> > - use another icon - big yellow dot?
>
> How about a no-entry sign?
>
> http://www.health-safety-signs.uk.com/productimages/No-Entry-outdoor-sig
>n.gif
>
>
> It's not a cross which means failed, so hopefully it'll convey the fact
> that it was deliberatly not downloaded?
>

Rather not. This sign speaks "Forbidden. Don't dare". Digikam needs
something like "Look at me! I am new here!". Yellow dot is doing exactly
this (IMO). Yellow is used often as increasing awareness color (change
colors in traffic lights, jackets of police/firefighters, etc.)

Golden star has good color but 5-point star in KDE has different
meaning: "Favorite. I like this item much".

If not yellow dot it could be 8-point star. It is used as "New"
indicator in "Net tab" icon in Konsole and Konqueror.

m.

_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=131034         
caulier.gilles free fr changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #17035|0                           |1
        is obsolete|                            |



------- Additional Comments From caulier.gilles free fr  2006-07-20 12:28 -------
Created an attachment (id=17048)
 --> (http://bugs.kde.org/attachment.cgi?id=17048&action=view)
new version witch fix B.K.O #127614
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=131034         
caulier.gilles free fr changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From caulier.gilles free fr  2006-07-20 16:54 -------
SVN commit 564596 by cgilles:

digikam from trunk : camera gui improvement and fix :

- fix rename customizer rule about custom prefix widget focus (#127614)
- Add small icon on top/roght of camera ites to indicate download status (#131034)
- Fix some minor internal bugs.
- Polish implementation.
- using d private class to speed up compilation
- camera gui support digiKam theme now. A screenshot :

http://digikam3rdparty.free.fr/Screenshots/camera_gui_with_theme_and_icon_download_status.png

BUG: 127614, 131034

CCMAIL: digikam-devel kde org



 M  +3 -1      NEWS  
 M  +0 -2      TODO  
 M  +14 -14    digikam/albumiconitem.cpp  
 M  +5 -0      digikam/albumiconitem.h  
 M  +166 -52   libs/imageproperties/cameraitempropertiestab.cpp  
 M  +1 -0      libs/imageproperties/cameraitempropertiestab.h  
 M  +13 -0     libs/imageproperties/imagepropertiessidebarcamgui.cpp  
 M  +1 -0      libs/imageproperties/imagepropertiessidebarcamgui.h  
 M  +1 -0      utilities/cameragui/Makefile.am  
 M  +158 -143  utilities/cameragui/cameracontroller.cpp  
 M  +1 -1      utilities/cameragui/cameracontroller.h  
 M  +245 -103  utilities/cameragui/cameraiconitem.cpp  
 M  +12 -22    utilities/cameragui/cameraiconitem.h  
 M  +99 -40    utilities/cameragui/cameraiconview.cpp  
 M  +10 -6     utilities/cameragui/cameraiconview.h  
 M  +28 -19    utilities/cameragui/cameraui.cpp  
 M  +1 -1      utilities/cameragui/cameraui.h  
 M  +5 -5      utilities/cameragui/dkcamera.cpp  
 M  +3 -3      utilities/cameragui/gpcamera.cpp  
 M  +13 -1     utilities/cameragui/gpiteminfo.h  
 M  +8 -2      utilities/cameragui/renamecustomizer.cpp  
 M  +7 -2      utilities/cameragui/renamecustomizer.h  
 M  +3 -3      utilities/cameragui/umscamera.cpp
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[FACORAT Fabrice]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=131034         




------- Additional Comments From f.faber-pro ifrance com  2006-07-20 17:21 -------
great job, this is awesome. Much better than I thought initially.

Thanks for all.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[FACORAT Fabrice]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=131034         
f.faber-pro ifrance com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |UNCONFIRMED
         Resolution|FIXED                       |



------- Additional Comments From f.faber-pro ifrance com  2006-07-21 14:38 -------
Sorry to reopen the bug report, I was was thinking about the implementation under my shower ( yeah, good for inspiration ), and I noticed that marking the photos which are going to be download with a star is somewhat useless and have no real meaning for an user.

So I agree with the green check for downloaded photos ( evident meaning ) and the gear for currently downloaded photo ( the same ), but IMHO the stars are too much ( and doesn't means really something, mostf of the times use for bookmarked stuffs ). The user know which ones need to be downloaded ... the ones selected. A better feedback would be to use a checkbox to clearly show wich photos are selected and which may ease the selection. see http://bugs.kde.org/show_bug.cgi?id=131036

Just my 0.02€
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=131034         
caulier.gilles free fr changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |WORKSFORME



------- Additional Comments From caulier.gilles free fr  2006-07-25 12:03 -------
Fabrice.

Let's me explain better what i want with icon annotations under camera item thumbs. Current implementation from svn give :

1/ A gray "?" if the download status is unknow.
2/ A blue gear during download.
3/ A green check when the download is complete.
4/ A red cross when download is broken.
5/ A yellow start when the item is new in camera.

2/, 3/, and 4/ are not a problem.

1/ and 5/ will be drived by a future implementation to manage the "already downloaded" status of camera items :

- Provided by Gphoto2 cameras using libgphoto2.
- Provided by digiKam itself for UMS cameras (TODO).

In fact look into #107316 B.K.O file about download status, and post comments into this thread...

Gilles
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[FACORAT Fabrice]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=131034         




------- Additional Comments From f.faber-pro ifrance com  2006-07-25 14:27 -------
OK, sorry. All of this sounds sensible.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel