[Digikam-devel] extragear/graphics/digikam/utilities/cameragui

classic Classic list List threaded Threaded
10 messages Options
Reply | Threaded
Open this post in threaded view
|

[Digikam-devel] extragear/graphics/digikam/utilities/cameragui

Gilles Caulier
SVN commit 523419 by cgilles:

digikam from trunk : Camera GUI : moving 'Advanced settings to the right side bar. removed left sidebar to optimized thumb area.

'Advanced Settings' tab is behing 'Properties' and 'Metadata' tabs for technical reasons. Actually, SideBar can only add a new tab to the bottom of tabs list. The current implementation is simple. Moving 'Advanced Settings' tab to the top requires sustentials changes in source codes. Please test the current implementation and give me a feedback.

CCMAIL: [hidden email]

 M  +11 -19    cameraui.cpp  
 M  +16 -12    renamecustomizer.cpp  


--- trunk/extragear/graphics/digikam/utilities/cameragui/cameraui.cpp #523418:523419
@@ -99,7 +99,6 @@
         cancelBtn    = 0;
         splitter     = 0;
         rightSidebar = 0;
-        leftSidebar  = 0;
     }
 
     bool                          busy;
@@ -133,8 +132,6 @@
     AnimWidget                   *anim;
 
     ImagePropertiesSideBarCamGui *rightSidebar;
-    
-    Sidebar                      *leftSidebar;
 };    
 
 CameraUI::CameraUI(QWidget* parent, const QString& title,
@@ -150,22 +147,26 @@
     d = new CameraUIPriv;
     setHelp("camerainterface.anchor", "digikam");
 
-    // -- setup view -----------------------------------------
+    // -------------------------------------------------------------------------
     
     QGridLayout* viewBoxLayout = new QGridLayout(plainPage(), 2, 3);
     viewBoxLayout->setColStretch( 0, 0 );
     viewBoxLayout->setColStretch( 1, 3 );
     viewBoxLayout->setColStretch( 2, 1 );
     viewBoxLayout->setColStretch( 3, 0 );
-    QHBox* widget = new QHBox(plainPage());
-
+    QHBox* widget   = new QHBox(plainPage());
+    d->splitter     = new QSplitter(widget);
+    d->view         = new CameraIconView(this, d->splitter);
+    d->rightSidebar = new ImagePropertiesSideBarCamGui(widget, "CameraGui Sidebar Right", d->splitter,
+                                                       Digikam::Sidebar::Right, true);
+    d->splitter->setOpaqueResize(false);
+    
     // -------------------------------------------------------------------------
 
-    d->leftSidebar      = new Sidebar(widget, "CameraGui Sidebar Left", Digikam::Sidebar::Left, true);
-    d->splitter         = new QSplitter(widget);
-    d->advBox           = new QWidget(d->leftSidebar);
+    d->advBox           = new QWidget(d->rightSidebar);
     QGridLayout* grid   = new QGridLayout( d->advBox, 2, 1, KDialog::marginHint());
     d->renameCustomizer = new RenameCustomizer(d->advBox);
+    d->view->setRenameCustomizer(d->renameCustomizer);
     grid->addMultiCellWidget(d->renameCustomizer, 0, 0, 0, 1);
         
     QVGroupBox* exifBox = new QVGroupBox(i18n("Use Camera Information"), d->advBox);
@@ -181,20 +182,12 @@
     grid->addMultiCellWidget(exifBox, 1, 1, 0, 1);
     grid->setRowStretch(2, 10);
 
-    d->leftSidebar->setSplitter(d->splitter);
-    d->leftSidebar->appendTab(d->advBox, SmallIcon("configure"), i18n("Advanced Settings"));
+    d->rightSidebar->appendTab(d->advBox, SmallIcon("configure"), i18n("Advanced Settings"));
     
     // -------------------------------------------------------------------------
-
-    d->view = new CameraIconView(this, d->splitter);
-    d->view->setRenameCustomizer(d->renameCustomizer);
     
-    d->rightSidebar = new ImagePropertiesSideBarCamGui(widget, "CameraGui Sidebar Right", d->splitter,
-                                                       Digikam::Sidebar::Right, true);
     viewBoxLayout->addMultiCellWidget(widget, 0, 0, 0, 3);
     viewBoxLayout->setRowSpacing(1, spacingHint());
-    d->splitter->setOpaqueResize(false);
-    d->leftSidebar->loadViewState();
     d->rightSidebar->loadViewState();
         
     // -------------------------------------------------------------------------
@@ -346,7 +339,6 @@
 
 CameraUI::~CameraUI()
 {
-    delete d->leftSidebar;
     delete d->rightSidebar;
     delete d;
 }
--- trunk/extragear/graphics/digikam/utilities/cameragui/renamecustomizer.cpp #523418:523419
@@ -95,9 +95,9 @@
 
     // ----------------------------------------------------------------
 
-    d->renameDefault = new QRadioButton(i18n("Camera file names"), this);
+    d->renameDefault = new QRadioButton(i18n("Camera filenames"), this);
     QWhatsThis::add( d->renameDefault, i18n("<p>Toogle on this option to use camera "
-                                            "provided image file names without modifications."));
+                                            "provided image filenames without modifications."));
     mainLayout->addMultiCellWidget(d->renameDefault, 0, 0, 0, 1);
 
     d->renameDefaultBox = new QGroupBox( this );
@@ -105,7 +105,7 @@
     d->renameDefaultBox->setInsideMargin(0);
     d->renameDefaultBox->setColumnLayout(0, Qt::Vertical);
 
-    d->renameDefaultCase = new QLabel( i18n("Change case to"), d->renameDefaultBox );
+    d->renameDefaultCase = new QLabel( i18n("Change case to:"), d->renameDefaultBox );
     d->renameDefaultCase->setSizePolicy( QSizePolicy::Minimum, QSizePolicy::Preferred );
 
     d->renameDefaultCaseType = new QComboBox( d->renameDefaultBox );
@@ -113,7 +113,9 @@
     d->renameDefaultCaseType->insertItem(i18n("Upper"), 1);
     d->renameDefaultCaseType->insertItem(i18n("Lower"), 2);
     d->renameDefaultCaseType->setSizePolicy(QSizePolicy::Minimum, QSizePolicy::Preferred);
-
+    QWhatsThis::add( d->renameDefaultCaseType, i18n("<p>Set here the method to use to change case "
+                                                    "of image filenames."));
+                                          
     QHBoxLayout* boxLayout = new QHBoxLayout( d->renameDefaultBox->layout() );
     boxLayout->addSpacing( 10 );
     boxLayout->addWidget( d->renameDefaultCase );
@@ -124,9 +126,9 @@
     // -------------------------------------------------------------
 
     d->renameCustom = new QRadioButton(i18n("Customize"), this);
-    QWhatsThis::add( d->renameCustom, i18n("<p>Toogle on this option to customize image file names "
+    mainLayout->addMultiCellWidget(d->renameCustom, 2, 2, 0, 1);
+    QWhatsThis::add( d->renameCustom, i18n("<p>Toogle on this option to customize image filenames "
                                            "during download."));
-    mainLayout->addMultiCellWidget(d->renameCustom, 2, 2, 0, 1);
 
     d->renameCustomBox = new QGroupBox(this);
     d->renameCustomBox->setFrameStyle(QFrame::NoFrame|QFrame::Plain);
@@ -141,17 +143,19 @@
 
     d->renameCustomPrefix = new KLineEdit(d->renameCustomBox);
     renameCustomBoxLayout->addMultiCellWidget(d->renameCustomPrefix, 0, 0, 2, 2);
-
+    QWhatsThis::add( d->renameCustomPrefix, i18n("<p>Set here the string to use like a prefix of "
+                                                 "image filenames."));
+                                                
     d->renameCustomExif = new QCheckBox(i18n("Add date and time"), d->renameCustomBox);
+    renameCustomBoxLayout->addMultiCellWidget(d->renameCustomExif, 1, 1, 1, 2);
     QWhatsThis::add( d->renameCustomExif, i18n("<p>Toogle on this option to add to filename the "
-                                               "camera provided date and time"));
-    renameCustomBoxLayout->addMultiCellWidget(d->renameCustomExif, 1, 1, 1, 2);
+                                               "camera provided date and time."));
 
     d->renameCustomSeq = new QCheckBox(i18n("Add sequence number"), d->renameCustomBox);
-    QWhatsThis::add( d->renameCustomSeq, i18n("<p>Toogle on this option to add to filename a "
-                                              "sequence number"));
     renameCustomBoxLayout->addMultiCellWidget(d->renameCustomSeq, 2, 2, 1, 2);
-
+    QWhatsThis::add( d->renameCustomSeq, i18n("<p>Toogle on this option to add to filenames a "
+                                              "sequence number."));
+    
     mainLayout->addMultiCellWidget(d->renameCustomBox, 3, 3, 0, 1);
 
     // -- setup connections -------------------------------------------------
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

Re: [Digikam-devel] extragear/graphics/digikam/utilities/cameragui

Sebastian Roeder
Hi Gilles,

> digikam from trunk : Camera GUI : moving 'Advanced settings to the right
> side bar. removed left sidebar to optimized thumb area.
>
> 'Advanced Settings' tab is behing 'Properties' and 'Metadata' tabs for
> technical reasons. Actually, SideBar can only add a new tab to the bottom
> of tabs list. The current implementation is simple. Moving 'Advanced
> Settings' tab to the top requires sustentials changes in source codes.
> Please test the current implementation and give me a feedback.
>
> CCMAIL: [hidden email]
>
>  M  +11 -19    cameraui.cpp  
>  M  +16 -12    renamecustomizer.cpp  

Wow that was fast. Will try it immediately and take some screens for
comparison. I guess it's not THAT important whether the tab is the first or
the last one so it is OK for me you did it the easier way ;-)

I missed one thing to mention in my "review": I am not sure whether "advanced
settings" is a good name for the tab. I was a little confused by that name
cause one automatically looks for the "standard settings" tab the "advanced"
might refer to. Well, and we both know such a tab doens't exist. It is not
clear in my opinion that the three buttons on the bottom are refered to
as "standard settings" and the tab provides "advanced settings".

I hope one can understand my thoughts about the name of the "advanced
settings". I vote for renaming it to a term that realy refers to what the
tab "does" (advanced settings for what? metadata? view? download?). Something
like "download settings/transfer settings/filename settings" would be more
clear I'd think. (Everybody please make better recommendations if you have an
idea).  
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

Re: [Digikam-devel] extragear/graphics/digikam/utilities/cameragui

Gilles Caulier-2
On Tuesday 28 March 2006 10:24 am, Sebastian Röder wrote:

> Hi Gilles,
>
> > digikam from trunk : Camera GUI : moving 'Advanced settings to the right>
> > side bar. removed left sidebar to optimized thumb area.>> 'Advanced
> > Settings' tab is behing 'Properties' and 'Metadata' tabs for> technical
> > reasons. Actually, SideBar can only add a new tab to the bottom> of tabs
> > list. The current implementation is simple. Moving 'Advanced> Settings'
> > tab to the top requires sustentials changes in source codes.> Please test
> > the current implementation and give me a feedback.>> CCMAIL:
> > [hidden email]>>  M  +11 -19    cameraui.cpp  >  M  +16 -12  
> >  renamecustomizer.cpp  
>
> Wow that was fast.

I'm working on digiKam today. I have complete to fix Camera Gui now. I
continue to work on others digiKam parts about to use Exiv2 instead libKexif.
Actually i'm checking kio-thumbnails generator...

> Will try it immediately and take some screens for
> comparison. I guess it's not THAT important whether the tab is the first or
> the last one so it is OK for me you did it the easier way ;-)

ok

> I missed one
> thing to mention in my "review": I am not sure whether "advanced settings"
> is a good name for the tab. I was a little confused by that name cause one
> automatically looks for the "standard settings" tab the "advanced" might
> refer to. Well, and we both know such a tab doens't exist. It is not clear
> in my opinion that the three buttons on the bottom are refered to as
> "standard settings" and the tab provides "advanced settings". I hope one
> can understand my thoughts about the name of the "advanced settings". I
> vote for renaming it to a term that realy refers to what the tab "does"
> (advanced settings for what? metadata? view? download?). Something like
> "download settings/transfer settings/filename settings" would be more clear
> I'd think. (Everybody please make better recommendations if you have an
> idea).

I'm agree to change it like "download settings" or something like that. You
have a svn acess to write now (:=))). You can fix it yourself. It's easy
(:=)))...

A nice day

Gilles
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

Re: [Digikam-devel] extragear/graphics/digikam/utilities/cameragui

Sebastian Roeder
In reply to this post by Gilles Caulier
> > The metainfo tabs are empty on when a photo is selected in the
> > "oberview".
>
> I don't understand what you mean (:=))). Witch view exactly ?

I meant the thumbnail view (album view like) - but you understood me
anyway ;-)

> > Only when you open the image in the image editor then you get the
> > exif/makernote infos. It exif/makernote not yet implemented in the
> > cameragui or is this a bug?
>
> Cameragui, editor and main interface use the same implementation. The only
> difference with cameragui is with gphoto cameras, where all Exif metadata
> are extracted using a gphoto command. With any camera, the Exif info are
> empty. I suspect a bug in gphoto. I have the same problem with my old
> Olympus C3000Z camera. When you open a image into IE from camera interface,
> the file is donwloaded from camera to computer. The Exif info can be
> displayed properlly. If your camera is a Gphoto like, You can check the
> size of Exif info extracted by Gphoto to look on the console. There is a
> debug message like this :
>
> Size of Exif metadata from camera = 8214
>
> if size are small, like any bytes, gphoto failed to get Exif info from
> camera.
>
> Note : this problem do not exist with USM mass storage camera !

OK looks like I am hit by this gphoto2 bug with my Nikon D70 in PTP mode. I
switched to USB Mass Storage on the camera but now I don't know how to
configure it inside digikam. The auto detect fails and in the add camera list
there is no USB Mass Storage anymore.  
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

Re: [Digikam-devel] extragear/graphics/digikam/utilities/cameragui

Gilles Caulier-2
On Tuesday 28 March 2006 02:31 pm, Sebastian Röder wrote:

> > > The metainfo tabs are empty on when a photo is selected in the
> > > "oberview".
> >
> > I don't understand what you mean (:=))). Witch view exactly ?
>
> I meant the thumbnail view (album view like) - but you understood me
> anyway ;-)
>
> > > Only when you open the image in the image editor then you get the
> > > exif/makernote infos. It exif/makernote not yet implemented in the
> > > cameragui or is this a bug?
> >
> > Cameragui, editor and main interface use the same implementation. The
> > only difference with cameragui is with gphoto cameras, where all Exif
> > metadata are extracted using a gphoto command. With any camera, the Exif
> > info are empty. I suspect a bug in gphoto. I have the same problem with
> > my old Olympus C3000Z camera. When you open a image into IE from camera
> > interface, the file is donwloaded from camera to computer. The Exif info
> > can be displayed properlly. If your camera is a Gphoto like, You can
> > check the size of Exif info extracted by Gphoto to look on the console.
> > There is a debug message like this :
> >
> > Size of Exif metadata from camera = 8214
> >
> > if size are small, like any bytes, gphoto failed to get Exif info from
> > camera.
> >
> > Note : this problem do not exist with USM mass storage camera !
>
> OK looks like I am hit by this gphoto2 bug with my Nikon D70 in PTP mode. I
> switched to USB Mass Storage on the camera but now I don't know how to
> configure it inside digikam. The auto detect fails and in the add camera
> list there is no USB Mass Storage anymore.

Since 0.8.x serie "USB Mass Storage" have been renamed to amore generic name :
"Mounted Camera". Take a look into handbook (:=))) :

http://docs.kde.org/development/en/extragear-graphics/digikam/using-kapp-setup.html#setupdialog

==> at "Camera Settings" section...

Sure, in setup dialog, Gphoto and UMS camera must be separate in the future.
There is a file in B.K.O about this point...

Gilles
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

Re: [Digikam-devel] extragear/graphics/digikam/utilities/cameragui

Sebastian Roeder
In reply to this post by Gilles Caulier
Hello agin Gilles,

> Cameragui, editor and main interface use the same implementation. The only
> difference with cameragui is with gphoto cameras, where all Exif metadata
> are extracted using a gphoto command. With any camera, the Exif info are
> empty. I suspect a bug in gphoto. I have the same problem with my old
> Olympus C3000Z camera. When you open a image into IE from camera interface,
> the file is donwloaded from camera to computer. The Exif info can be
> displayed properlly. If your camera is a Gphoto like, You can check the
> size of Exif info extracted by Gphoto to look on the console. There is a
> debug message like this :
>
> Size of Exif metadata from camera = 8214
>
> if size are small, like any bytes, gphoto failed to get Exif info from
> camera.
>
> Note : this problem do not exist with USM mass storage camera !
OK, got USB Mass Storage to work and now I get the exif infos to. Is the bug
known to the gphoto2 folks and did anybody test ghoto2-2.1.99 yet?

> > Little UI improvement recommendation: the animated circle in the lower
> > right corner that show whether the UI is "bussy" is to small for the area
> > with the digikam banner - there are gray stripes on the top/bottom of the
> > animated graphic.
>
> Hum, i can't see what you mean. Give me a little screenshot please...

Here you have a little screeny ;-)

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

cosmetic_cleanup.png (8K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [Digikam-devel] extragear/graphics/digikam/utilities/cameragui

Gilles Caulier-2
On Tuesday 28 March 2006 03:03 pm, Sebastian Röder wrote:

> Hello agin Gilles,
>
> > Cameragui, editor and main interface use the same implementation. The
> > only difference with cameragui is with gphoto cameras, where all Exif
> > metadata are extracted using a gphoto command. With any camera, the Exif
> > info are empty. I suspect a bug in gphoto. I have the same problem with
> > my old Olympus C3000Z camera. When you open a image into IE from camera
> > interface, the file is donwloaded from camera to computer. The Exif info
> > can be displayed properlly. If your camera is a Gphoto like, You can
> > check the size of Exif info extracted by Gphoto to look on the console.
> > There is a debug message like this :
> >
> > Size of Exif metadata from camera = 8214
> >
> > if size are small, like any bytes, gphoto failed to get Exif info from
> > camera.
> >
> > Note : this problem do not exist with USM mass storage camera !
>
> OK, got USB Mass Storage to work and now I get the exif infos to. Is the
> bug known to the gphoto2 folks and did anybody test ghoto2-2.1.99 yet?

I don't know. You need to contact gphoto2 devel ML for that. Sorry...

>
> > > Little UI improvement recommendation: the animated circle in the lower
> > > right corner that show whether the UI is "bussy" is to small for the
> > > area with the digikam banner - there are gray stripes on the top/bottom
> > > of the animated graphic.
> >
> > Hum, i can't see what you mean. Give me a little screenshot please...
>
> Here you have a little screeny ;-)

ok. This problem do not exists here because i use standard KDE colors
settings. I will fix it.

Gilles
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

Re: [Digikam-devel] extragear/graphics/digikam/utilities/cameragui

Sebastian Roeder
In reply to this post by Gilles Caulier
And again ... it's me :-)

> I'm agree to change it like "download settings" or something like that. You
> have a svn acess to write now (:=))). You can fix it yourself. It's easy
> (:=)))...

You are right - leave the renaming up to me (already done locally, I
think "Download Settings fits best). This is a good chance to test whether
everything wents fine when I commit something to SVN (easy to restore when
something goes wrong ;-)).

About the cameragui there is one more issue related to it (though not to the
gui changes): when you choose to rename the images and to automatically add
an index number, then this number starts on the first picture no matter which
images are selected. It would be better if the index numbers are only applied
for the selected images instead.

Example: I have 5 images on the camera, choose to rename them "example" and
enable the index number. Now the photos are named example-0001.jpg to
example-0005.jpg. When I select the last three images, then they still have
the names example-0003.jpg to example-0005.jpg. I'd like better to have them
named example-0001.jpg to example-0003.jpg.

If you have not the time to fix this today I will fill a bugreport about it.
Happy coding today! And don't forget to take a break once in a while ;-)

Sebastian
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

Re: [Digikam-devel] extragear/graphics/digikam/utilities/cameragui

Gilles Caulier-2
On Tuesday 28 March 2006 04:14 pm, Sebastian Röder wrote:

> And again ... it's me :-)
>
> > I'm agree to change it like "download settings" or something like that.
> > You have a svn acess to write now (:=))). You can fix it yourself. It's
> > easy (:=)))...
>
> You are right - leave the renaming up to me (already done locally, I
> think "Download Settings fits best). This is a good chance to test whether
> everything wents fine when I commit something to SVN (easy to restore when
> something goes wrong ;-)).
>
> About the cameragui there is one more issue related to it (though not to
> the gui changes): when you choose to rename the images and to automatically
> add an index number, then this number starts on the first picture no matter
> which images are selected. It would be better if the index numbers are only
> applied for the selected images instead.
>
> Example: I have 5 images on the camera, choose to rename them "example" and
> enable the index number. Now the photos are named example-0001.jpg to
> example-0005.jpg. When I select the last three images, then they still have
> the names example-0003.jpg to example-0005.jpg. I'd like better to have
> them named example-0001.jpg to example-0003.jpg.

yes, i understand.

>
> If you have not the time to fix this today I will fill a bugreport about
> it. Happy coding today!

Hum. Not more time. Please make file in B.K.O and start to study the code to
make a patch yourself. Please, this changes must going later 0.9.0.

> And don't forget to take a break once in a while
> ;-)

Tomorrow, i will don't work on computer. I'm staying at home to make some
photographs (:=))

Gilles
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

Re: [Digikam-devel] extragear/graphics/digikam/utilities/cameragui

Gilles Caulier-2
In reply to this post by Sebastian Roeder
Le Mardi 28 Mars 2006 15:03, Sebastian Röder a écrit :
> > > Little UI improvement recommendation: the animated circle in the lower
> > > right corner that show whether the UI is "bussy" is to small for the
> > > area with the digikam banner - there are gray stripes on the top/bottom
> > > of the animated graphic.
> >
> > Hum, i can't see what you mean. Give me a little screenshot please...
>
> Here you have a little screeny ;-)

Marcel, i cannot reproduce this problem on digikam logo from cameragui. I have
tried to change KDE colors and themes from KDE control pannel without
success. I have checked the code to find some problem in implementation.
nothing (:=)))....

Can you give me more informations about you current KDE theme, widget styles,
colors, etc. sets into KDE control pannel. Thanks in advance

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