After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 340901 - Adjust Colur: Modifying wrong picture
Adjust Colur: Modifying wrong picture
Status: RESOLVED FIXED
Product: f-spot
Classification: Other
Component: Editing
CVS
Other Linux
: Normal major
: ---
Assigned To: F-spot maintainers
F-spot maintainers
Depends on:
Blocks:
 
 
Reported: 2006-05-07 08:31 UTC by Bengt Thuree
Modified: 2006-09-29 04:44 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Patch (2.88 KB, patch)
2006-09-22 22:56 UTC, Bengt Thuree 		reviewed Details | Review

Description Bengt Thuree 2006-05-07 08:31:30 UTC 
1) Select one photo in browse mode
2) Start Adjust Colur (from browse mode) 
3) --> Photo is displayed in Edit mode
4) Go back to browse mode
4) Select another picture
5) Modify the color in the already displayed color dialog
6) The first selected picture colors are adjusted.

(If you change photo by using the next/previous arrows in Edit mode, 
the color dialog change photos as well)

The Color Dialog should be reflect the current selected photo.
Comment 1 Bengt Thuree 2006-05-15 10:52:14 UTC 
Perhaps similar to bug #338842 ?
Comment 2 Larry Ewing 2006-07-26 21:36:11 UTC 
hmmm this is just that selecting the image in  browse mode is not really meant to be the same as chosing to edit the image.  We should probably close the color dialog when you switch back to browse mode.
Comment 3 Bengt Thuree 2006-09-22 22:56:35 UTC 
Created attachment 73249 [details] [review]
Patch

Created a small patch (with good help from Stephane), that 
1) Ensures only one ColorDialog can be open at once
2) Destroys the ColorDialog if we leave Edit mode
3) Let the ColorDialog follow the picture in Full Screen mod
Comment 4 Stephane Delcroix 2006-09-23 10:44:25 UTC 
reviewed. it works.
Comment 5 Gabriel Burt 2006-09-29 04:44:27 UTC 
I committed a modified version of this patch.  It adds the ability to seamlessly switch between Edit and Fullscreen modes w/o resetting the changes you have made to the color adjustment dialog.