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 142968 - Zoom motion blur should "push" the image outwards
Zoom motion blur should "push" the image outwards
Status: RESOLVED FIXED
Product: GIMP
Classification: Other
Component: Plugins
2.0.x
Other Linux
: Normal trivial
: 2.4
Assigned To: GIMP Bugs
GIMP Bugs
Depends on:
Blocks:
 
 
Reported: 2004-05-22 17:28 UTC by Chad Daelhousen
Modified: 2005-06-22 11:49 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Change sign to blur outwards (697 bytes, patch)
2004-05-22 17:29 UTC, Chad Daelhousen
none Details | Review

Description Chad Daelhousen 2004-05-22 17:28:42 UTC
GIMP's motion blur in zoom mode "pulls" pixels inwards, resulting in
less-than-desirable quality around the edges of the image. It also goes against
how every other program I've used does this type of blur.

Trivial patch coming soon.
Comment 1 Chad Daelhousen 2004-05-22 17:29:43 UTC
Created attachment 27935 [details] [review]
Change sign to blur outwards

...and here it is.
Comment 2 Sven Neumann 2004-05-22 17:42:18 UTC
We can definitely not change the plug-in behaviour in 2.0 because it might break
scripts that rely on the current behaviour. But I think we should accept this
patch for gimp-2.2. If it turns out to be an improvement we can probably just do
it and do not need to provide a way to get the old behaviour when the plug-in is
used via the PDB.
Comment 3 Sven Neumann 2004-05-31 11:42:50 UTC
2004-05-31  Sven Neumann  <sven@gimp.org>

	* plug-ins/common/mblur.c (mblur_zoom): push pixels outwards not
	to the center as suggested by Chad Daelhousen (bug #142968).
Comment 4 Evan Langlois 2005-06-02 20:45:00 UTC
I really need the old behavior (example: 
http://www.rtistx.com/logos/rtistx_dark.jpg).  This image was made by using a
motion blur zoom on a duplicate layer of the text and further processing for the
desired color effects.   With the zoom going out instead of in, there would be
no easy way to do this sort of effect.   Now, I need to redo the image without a
black background and in a size big enough for printing and Gimp won't do it.

Can you allow the slider to go negative or something to get the old behavior?
I'm going to see if I can reverse the patch for now so I can get this stuff
done, but I would definately appreciate some fix to get the old behavior!
Comment 5 Sven Neumann 2005-06-02 20:48:30 UTC
The old behaviour was just plain broken.
Comment 6 David Odin 2005-06-02 20:51:25 UTC
This plug-in should really allow both way of zooming. Should be trivial to fix.
I'm reopening this bug so we won't forget about this.
Comment 7 weskaggs 2005-06-21 23:53:27 UTC
Okay, I added a checkbox (only sensitive for zoom blur) that allows a choice of
blurring outward or inward.  I also added an extra, optional, argument to the
pdb call.  Given that this is really an enhancement rather than a bug, these
changes seem too extensive for backporting to the stable branch, so I will
resolve this as FIXED.  (...the ui might possibly benefit from a bit of
tweaking, however)

For HEAD:

2005-06-21  Bill Skaggs  <weskaggs@primate.ucdavis.edu>

	* plug-ins/common/mblur.c: add option of blurring inward
	or outward for zoom, should fix bug #142968.
Comment 8 Sven Neumann 2005-06-22 11:49:44 UTC
I have taken care of tweaking the dialog after your changes.