[Bug 210910] New: Please allow locking image zoom on the light table

[Karl Günter Wünsch]

https://bugs.kde.org/show_bug.cgi?id=210910

           Summary: Please allow locking image zoom on the light table
           Product: digikam
           Version: unspecified
          Platform: Compiled Sources
        OS/Version: unspecified
            Status: UNCONFIRMED
          Severity: wishlist
          Priority: NOR
         Component: general
        AssignedTo: [hidden email]
        ReportedBy: [hidden email]


Version:            (using KDE 4.3.2)
Installed from:    Compiled From Sources

When comparing images on the Light Table I usually do so at 100% view to work
out which of a series to keep and which to delete. Upon deleting an image from
the Light Table (moving to trash) the image zoom irritatingly resets to the
scale that shows the whole image. Now I have to zoom in again to 100% to
continue. For the moment this makes using the light table impossible!

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier-4]

https://bugs.kde.org/show_bug.cgi?id=210910


Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]
          Component|general                     |Light Table
            Version|unspecified                 |1.0.0




--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Karl Günter Wünsch]

https://bugs.kde.org/show_bug.cgi?id=210910





--- Comment #1 from  <kgw mineralien-verkauf de>  2009-10-24 21:29:35 ---
Created an attachment (id=37802)
 --> (http://bugs.kde.org/attachment.cgi?id=37802)
Patch to keep a set zoom ratio when changing images...

Please would anyone with the knowledge how to do this properly in the context
of digikam make this an option (setting either true for the previous behavior
and false for the behavior I crave for)...

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Johannes Wienke-3]

https://bugs.kde.org/show_bug.cgi?id=210910


Johannes Wienke <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]
            Summary|Please allow locking image  |Please allow locking image
                   |zoom on the light table     |zoom on the light table
                   |                            |[patch]




--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Karl Günter Wünsch]

https://bugs.kde.org/show_bug.cgi?id=210910


[hidden email] changed:

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




--- Comment #2 from  <kgw mineralien-verkauf de>  2009-10-25 14:16:57 ---
Created an attachment (id=37821)
 --> (http://bugs.kde.org/attachment.cgi?id=37821)
Patch to keep a set zoom ratio when changing images.

I have added a setting into the light table settings group.

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Marcel Wiesweg]

https://bugs.kde.org/show_bug.cgi?id=210910


Marcel Wiesweg <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]




--- Comment #3 from Marcel Wiesweg <marcel wiesweg gmx de>  2009-11-04 12:17:22 ---
Should we reset the zoom at all when loading a new image?
I mean, do we want an option for this or do we want to change it without
offering an option?
If we want an option, what should be the default value?

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier-4]

https://bugs.kde.org/show_bug.cgi?id=210910





--- Comment #4 from Gilles Caulier <caulier gilles gmail com>  2009-11-04 15:32:17 ---
My viewpoint : an option with the current behavior as default settings.

We need more viewpoints here... Please

Gilles

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Karl Günter Wünsch]

https://bugs.kde.org/show_bug.cgi?id=210910





--- Comment #5 from  <kgw mineralien-verkauf de>  2009-11-04 17:56:06 ---
I'm the one who would like this new behavior but the old one has been around
for long enough as to have enough merit as not to abandon it. So even I second
Gilles viewpoint - see my attached patch...

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

https://bugs.kde.org/show_bug.cgi?id=210910


Julien Narboux <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]




--- Comment #6 from Julien Narboux <Julien narboux fr>  2009-11-21 18:54:48 ---
I think the current situation make the light table quite impossible to use.
When comparing picture they should always be at the same zoom ratio. I don't
really understand the current behavior, even when you click on synchronize the
zoom ratio is often reset to "zoom to fit". I think the zoom ratio should never
be changed without user explicitly asking for it.

Julien

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

https://bugs.kde.org/show_bug.cgi?id=210910





--- Comment #7 from Julien Narboux <Julien narboux fr>  2009-11-26 08:18:17 ---
Created an attachment (id=38595)
 --> (http://bugs.kde.org/attachment.cgi?id=38595)
New version of the patch against rev 1054503

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

https://bugs.kde.org/show_bug.cgi?id=210910





--- Comment #8 from Julien Narboux <Julien narboux fr>  2009-11-26 08:20:30 ---
Hi,

I tested the patch with current version it works well. I my opinion it should
not be an option, or at least the new behavior should be the default.

Julien

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier-4]

https://bugs.kde.org/show_bug.cgi?id=210910





--- Comment #9 from Gilles Caulier <caulier gilles gmail com>  2009-11-26 08:22:47 ---
Ok, let's go to take a try to this patch. apply it to svn, it's very simple to
test...

Gilles

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier-4]

https://bugs.kde.org/show_bug.cgi?id=210910





--- Comment #10 from Gilles Caulier <caulier gilles gmail com>  2010-01-22 18:34:50 ---
Julien,

Can you adjust this patch to not include settings panel change and use new
defaut behaviors as well. I would to see it included in 1.1.0 release

Thanks in advance

Gilles Caulier

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

https://bugs.kde.org/show_bug.cgi?id=210910


Julien Narboux <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1




--- Comment #11 from Julien Narboux <Julien narboux fr>  2010-01-22 19:09:38 ---
Gilles,

The patch is already applied, i will remove the option.

Julien

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

https://bugs.kde.org/show_bug.cgi?id=210910





--- Comment #12 from Julien Narboux <Julien narboux fr>  2010-01-23 08:41:53 ---
SVN commit 1078909 by jnarboux:

Undo compit 1054504, this removes the option for updating zoom ratio when new
image is loaded in lighttable.
Make new behaviour be old default, we never update zoom ratio.

CCBUGS: 210910



 M  +1 -7      lighttable/lighttablepreview.cpp  
 M  +0 -1      lighttable/lighttablewindow_p.h  
 M  +0 -9      setup/setuplighttable.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1078909

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

https://bugs.kde.org/show_bug.cgi?id=210910


Julien Narboux <[hidden email]> changed:

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




--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier-4]

https://bugs.kde.org/show_bug.cgi?id=210910

Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Version Fixed In|                            |1.1.0

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel