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 120959 - Add 'Merge Visible Layers' menu entry to the Layers dialog context menu
Add 'Merge Visible Layers' menu entry to the Layers dialog context menu
Status: RESOLVED FIXED
Product: GIMP
Classification: Other
Component: User Interface
1.x
Other All
: Normal enhancement
: 2.0
Assigned To: GIMP Bugs
GIMP Bugs
Depends on:
Blocks:
 
 
Reported: 2003-08-29 00:14 UTC by gunther_oz
Modified: 2003-08-31 18:09 UTC
See Also:
GNOME target: ---
GNOME version: Unversioned Enhancement


Attachments
patch with a proposed fix (3.47 KB, patch)
2003-08-31 12:47 UTC, Sven Neumann
none Details | Review

Description gunther_oz 2003-08-29 00:14:24 UTC
'Bug' found when using GIMP 1.3.18 on winxp.

The absence of the 'merge visible' function is a notable loss from the 
layers dialog popup menu. 
Multiple uses of the 'merge down' function serve the same purpose if all 
layers use normal blend modes, but this falls apart if dodge/overlay 
layers have been used. Another workaround is also to use the 'copy 
visible' function on the selected layers and then paste back into the 
image.

Both of these solutions are somewhat inefficient though.
Could the function be returned to the layers menu?
If this is a recent inclusion in cvs, then just ignore my stupidity.
Comment 1 Henrik Brix Andersen 2003-08-29 08:07:29 UTC
Seems "Merge Visible" got losts somewhere doing the development of
1.3.x. It should definately be re-added.
Comment 2 Sven Neumann 2003-08-29 11:10:29 UTC
It's located in the Image menu. I don't think it should be in the
Layer context menu since it is a function that affects the image, not
a particular layer.
Comment 3 gunther_oz 2003-08-31 03:10:21 UTC
I would respectfully disagree. :-)
For me, it makes more sense that if you want to merge, for example 
layers 1,3 and 6 from an image, that you select the relevant layers 
in the layers dialog box and then right click at that point and 
select the merge function.
Having it in the image menu tree means switching focus to the image 
and scrolling through an extra menu. Not a huge problem but it 
doesn't seem as logical to me as it pertains to layer manipulation 
not the image as such.

It seems i'm not the only one that didn't intuitively think to scan 
through the image menu given Henrik's statement. 
Anyhoo, good luck with it all.
Comment 4 Sven Neumann 2003-08-31 09:56:56 UTC
I did not disagree with you. I only stated the the function is there.
I would appreciate if you could rephrase the summary of this report to
make it a request for the addition of a "Merge Visible Layers" entry
to the context menu of the layers dialog.
Comment 5 Sven Neumann 2003-08-31 12:47:23 UTC
Created attachment 19627 [details] [review]
patch with a proposed fix
Comment 6 Sven Neumann 2003-08-31 15:53:47 UTC
My patch got the menu label wrong. It needs to be "Merge Visible
Layers..." because it opens a dialog.

However, the question here is, do we need "Flatten Image" as well and
should these two perhaps be separated from the other entries that
operate on a particular layer?
Comment 7 Raphaël Quinet 2003-08-31 16:38:49 UTC
Yes to both.
Comment 8 Sven Neumann 2003-08-31 18:09:36 UTC
2003-08-31  Sven Neumann  <sven@gimp.org>

  * app/gui/layers-commands.[ch]	
  * app/gui/layers-menu.c: added "Merge Visible Layers..." and
  "Flatten Image" menu entries as suggested in bug #120959.