Bug 109992

Created attachment 12036 [details]
current layout of comments editor

Problem ist, that its only a thumbnail. There should be a resized Imaga, thats as big as possible, plus the fields should be resizeable.

If you resize it now, the image still stays that small, since thumbnails are at fixed size.

This will not be done for 0.8. Remark about the screenie: we have to develop for 800x600.

No, thats not true, you dont have to develop for 800x600. You have to develop for as many resolutions as possible, including at least 800x600 - 1600x1200. With a "resizable" image, everyone with 800x600 and 1600x1200 would be happy, with a fixed image, only users with 800x600 might be happy, but thats only a small minority.

That is not true, KDE HIG say it has to fit 800x600. 

So we can not make it larger at the moment. We have to implementing rescaling as a solution. Which will not be in 0.8.

> Problem ist, that its only a thumbnail. There should be a
> resized Imaga, thats as big as possible, 


Should be.

> plus the fields should be
> resizeable.


Here I don't agree. Dialog is already quite heavy visually (all those
frames). Width of tags field can be set arbitrarily, same for height of
comments field. In fact there is no reason to increase their dimensions
when changing size of dialog.

> Remark about the screenie: we
> have to develop for 800x600.


This is size where everything has to work. Not that everyone has to live
with.

I would like to have all three areas in this dialog resizable (preview, comments, tags). I personally would make the comments area a lot smaller and gain a lot more space for the thumbnail. 

SVN commit 482869 by cgilles:

Big changes in digiKam. 
This is the new code dedicaced to 0.9.0 unstable developement branch!!!
digiKam users, please take a care...

Core:

==> New image properties sidebar implementation to display exif, histogram, and image comments & tags to the left of windows using sidebar. 
    This way remove all image properties dialogs and reduce main contextual popup menu (BUG 109096, 109817, 109992, 113103).

==> New DImg framework to work "à la QImage" with image in 8/16 bits/color/pixel(BUG 103176). 
    The code is based on incomplete DImage implementation from Renchi Raju. I had fixed and improved this one.
    Native file formats implemented are JPEG, PNG, TIFF, RAW (dcraw), PPM.
    ICC profils and Meta-data are preserved. lcms is used to manage colors rendering manually or automaticly (BUG 87823).
    Now, There is a new digiKam depency with lcms.

Showfoto: 

- Image properties sidebar support
- All images from a folder can be loaded at the same time.
- Add DImg support.
- Thumbbar moved to the left. Image properties side bar to the right to make a consistant GUI with digikam window.

Image Editor:

- Image properties sidebar support
- Add DImg support.

PENDING TASK: 

- DImg                : add cache file method to improve loading operations.
- DImg                : add metadata transfert rules between file formats.
- DImg                : add native support of JPEG2000, OpenEXR, and Krita file formats.
- DImg                : add rules to detect camera type use to take a photo and applying automaticly an ICC profil (RAW file formats especially).
- DImg                : add Load/save operation in separed thread to unfreeze GUI.
- Image Editor        : add progress bar during load/save operation (depending of precedent point) 
- Image Editor        : add Events support between IE and main window about comments & tag sidebar 
- Core                : porting all imageplugins to DImg and support 16 bits images (actually only BCG tool is ported).
- Core                : porting all image filters to 16 bits (histogram, curves, color effects, etc.)
- Core                : Saving digiKam comments in PNG & TIFF files like JPEG.
- Core                : Removed imlib2 depency.
- Setup               : add new configuration dialog tab to set ICC profils management.
- DigikamImagePlugins : porting all plugins to DImg and support 16 bits images. All plugins work fine in 8 bits.
- DigikamImagePlugins : add new plugin to apply manually an ICC profil.

All contributors welcome to finalize digiKam 0.9.0...

CCBUGS: 87823, 103176, 109096, 109817, 109992, 113103
CCMAIL: digikam-devel@kde.org

 M  +10 -0     configure.in.bot  
 M  +47 -0     configure.in.in  
 M  +9 -5      digikam.kdevelop  
 M  +25 -25    digikam/Makefile.am  
 M  +12 -44    digikam/albumiconview.cpp  
 M  +23 -26    digikam/albumiconview.h  
 M  +6 -1      digikam/albumsettings.cpp  
 M  +20 -47    digikam/digikamapp.cpp  
 M  +20 -25    digikam/digikamapp.h  
 M  +0 -4      digikam/digikamui.rc  
 M  +97 -54    digikam/digikamview.cpp  
 M  +41 -36    digikam/digikamview.h  
 M  +1 -1      digikam/main.cpp  
 M  +16 -15    imageplugins/Makefile.am  
 M  +2 -2      imageplugins/imageeffect_bcg.cpp  
 M  +21 -24    imageplugins/imageeffect_hsl.cpp  
 M  +1 -1      imageplugins/imageplugin_core.h  
 M  +17 -12    kioslave/Makefile.am  
 M  +1 -1      libs/Makefile.am  
 A             libs/dcraw (directory)  
 A             libs/dcraw/Makefile.am  
 A             libs/dcraw/parse.c   [License: UNKNOWN]
 A             libs/dcraw/parse_old.c   [License: UNKNOWN]
 A             libs/dcraw/rawfiles.h   [License: GPL]
 A             libs/dimg (directory)  
 A             libs/dimg/Makefile.am  
 A             libs/dimg/README  
 A             libs/dimg/colormodifier.cpp   [License: GPL]
 A             libs/dimg/colormodifier.h   [License: GPL]
 A             libs/dimg/dimg.cpp   [License: GPL]
 A             libs/dimg/dimg.h   [License: GPL]
 A             libs/dimg/dimgloader.cpp   [License: GPL]
 A             libs/dimg/dimgloader.h   [License: GPL]
 A             libs/dimg/dimgprivate.h   [License: GPL]
 A             libs/dimg/dimgscale.cpp   [License: GPL]
 A             libs/dimg/iccjpeg.c   [License: no copyright]
 A             libs/dimg/iccjpeg.h   [License: no copyright]
 A             libs/dimg/icctransform.cpp   [License: GPL]
 A             libs/dimg/icctransform.h   [License: GPL]
 A             libs/dimg/jpegloader.cpp   [License: GPL]
 A             libs/dimg/jpegloader.h   [License: GPL]
 A             libs/dimg/pamloader.cpp   [License: GPL]
 A             libs/dimg/pamloader.h   [License: GPL]
 A             libs/dimg/pngloader.cpp   [License: GPL]
 A             libs/dimg/pngloader.h   [License: GPL]
 A             libs/dimg/ppmloader.cpp   [License: GPL]
 A             libs/dimg/ppmloader.h   [License: GPL]
 A             libs/dimg/qimageloader.cpp   [License: GPL]
 A             libs/dimg/qimageloader.h   [License: GPL]
 A             libs/dimg/rawloader.cpp   [License: GPL]
 A             libs/dimg/rawloader.h   [License: GPL]
 A             libs/dimg/tiffloader.cpp   [License: GPL]
 A             libs/dimg/tiffloader.h   [License: GPL]
 A             libs/imageproperties (directory)  
 A             libs/imageproperties/Makefile.am  
 A             libs/imageproperties/imagedescedittab.cpp   [License: GPL]
 A             libs/imageproperties/imagedescedittab.h   [License: GPL]
 A             libs/imageproperties/imagepropertiesexiftab.cpp   [License: GPL]
 A             libs/imageproperties/imagepropertiesexiftab.h   [License: GPL]
 A             libs/imageproperties/imagepropertieshistogramtab.cpp   [License: GPL]
 A             libs/imageproperties/imagepropertieshistogramtab.h   [License: GPL]
 A             libs/imageproperties/imagepropertiessidebar.cpp   [License: GPL]
 A             libs/imageproperties/imagepropertiessidebar.h   [License: GPL]
 A             libs/imageproperties/imagepropertiessidebardb.cpp   [License: GPL]
 A             libs/imageproperties/imagepropertiessidebardb.h   [License: GPL]
 A             libs/imageproperties/navigatebarwidget.cpp   [License: GPL]
 A             libs/imageproperties/navigatebarwidget.h   [License: GPL]
 M  +3 -0      libs/levels/imagelevels.h  
 M  +20 -4     libs/thumbbar/thumbbar.h  
 M  +5 -4      showfoto/main.cpp  
 A             showfoto/mimetypes (directory)  
 A             showfoto/mimetypes/Makefile.am  
 A             showfoto/mimetypes/x-raw.desktop  
 AM            showfoto/pics/hi22-mime-raw.png  
 AM            showfoto/pics/hi32-mime-raw.png  
 AM            showfoto/pics/hi48-mime-raw.png  
 AM            showfoto/pics/hi64-mime-raw.png  
 M  +106 -89   showfoto/showfoto.h  
 M  +2 -1      utilities/imageeditor/Makefile.am  
 A             utilities/imageeditor/canvas (directory)  
 M  +6 -2      utilities/imageeditor/canvas.cpp  
 M  +1 -1      utilities/imageeditor/canvas.h  
 A             utilities/imageeditor/canvas/Makefile.am  
 A             utilities/imageeditor/canvas/canvas.cpp   [License: GPL]
 A             utilities/imageeditor/canvas/canvas.h   [License: GPL]
 A             utilities/imageeditor/canvas/dimginterface.cpp   [License: GPL]
 A             utilities/imageeditor/canvas/dimginterface.h   [License: GPL]
 A             utilities/imageeditor/canvas/imageplugin.cpp   [License: GPL]
 A             utilities/imageeditor/canvas/imageplugin.h   [License: GPL]
 A             utilities/imageeditor/canvas/imagepluginloader.cpp   [License: GPL]
 A             utilities/imageeditor/canvas/imagepluginloader.h   [License: GPL]
 A             utilities/imageeditor/canvas/undoaction.cpp   [License: GPL]
 A             utilities/imageeditor/canvas/undoaction.h   [License: GPL]
 A             utilities/imageeditor/canvas/undocache.cpp   [License: GPL]
 A             utilities/imageeditor/canvas/undocache.h   [License: GPL]
 A             utilities/imageeditor/canvas/undomanager.cpp   [License: GPL]
 A             utilities/imageeditor/canvas/undomanager.h   [License: GPL]
 M  +0 -3      utilities/imageeditor/digikamimagewindowui.rc  
 A             utilities/imageeditor/editor (directory)  
 A             utilities/imageeditor/editor/Makefile.am  
 A             utilities/imageeditor/editor/digikamimageplugin.desktop  
 A             utilities/imageeditor/editor/digikamimagewindowui.rc  
 A             utilities/imageeditor/editor/imageiface.cpp   [License: GPL]
 A             utilities/imageeditor/editor/imageiface.h   [License: GPL]
 A             utilities/imageeditor/editor/imageprint.cpp   [License: GPL]
 A             utilities/imageeditor/editor/imageprint.h   [License: GPL]
 A             utilities/imageeditor/editor/imageresizedlg.cpp   [License: GPL]
 A             utilities/imageeditor/editor/imageresizedlg.h   [License: GPL]
 A             utilities/imageeditor/editor/imagewidget.cpp   [License: GPL]
 A             utilities/imageeditor/editor/imagewidget.h   [License: GPL]
 A             utilities/imageeditor/editor/imagewindow.cpp   [License: GPL]
 A             utilities/imageeditor/editor/imagewindow.h   [License: GPL]
 M  +119 -118  utilities/imageeditor/imagewindow.cpp  
 M  +80 -63    utilities/imageeditor/imagewindow.h  
 M  +115 -33   utilities/imageeditor/imlibinterface.cpp  
 M  +1 -1      utilities/imageeditor/imlibinterface.h  

This file can be closed now.