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 307314 - Gradient Editor can be wider than GIMP_VIEWABLE_MAX_PREVIEW_SIZE
Gradient Editor can be wider than GIMP_VIEWABLE_MAX_PREVIEW_SIZE
Status: RESOLVED FIXED
Product: GIMP
Classification: Other
Component: User Interface
2.2.x
Other All
: Normal normal
: 2.6
Assigned To: GIMP Bugs
GIMP Bugs
Depends on:
Blocks:
 
 
Reported: 2005-06-11 18:51 UTC by tomr
Modified: 2008-01-23 19:42 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description tomr 2005-06-11 18:51:55 UTC
Distribution/Version: FC3

Open Gradient Editor. Make it very wide. Notice that gradient is centred, but
arrows are left-justified. Like this:
http://loki.aceldama.com/stuff/gimp/gradient-editor-misalignment.png
Comment 1 Michael Natterer 2005-06-11 19:02:50 UTC
The preview size is limited to 1024.

We should probably drop this limitation since the only part of GIMP
that really needs large previews (the gradient editor), needs even
larger ones.
Comment 2 Sven Neumann 2005-06-13 15:40:24 UTC
Shouldn't the gradient editor deal gracefully with the fact that the preview is
not necessarily of the requested size?
Comment 3 Sven Neumann 2005-06-14 10:41:03 UTC
I had a look at the code and while it seems desirable to deal with any preview
size, it would add quite some complexity to already complex code. Might be
easier to just increase the limit to a point where we don't get this problem any
longer. What about increasing the limit to 4096?
Comment 4 weskaggs 2006-05-20 23:49:32 UTC
Increasing priority to Urgent because this should be tried now if it is going to be done for 2.4, so that the results can get some testing.
Comment 5 Sven Neumann 2006-05-29 16:05:23 UTC
Increasing the constant to 4096 will increase the heap size by 36 KB (see app/widgets/gimprender.c:gimp_render_setup_notify). That is probably tolerable.
Comment 6 Sven Neumann 2006-08-15 17:11:02 UTC
I've gone for something in between. 2048 should be enough to avoid the problem.
Bumping the bug to 2.6 and lowering priority.

2006-08-15  Sven Neumann  <sven@gimp.org>

	* app/core/gimpviewable.h: increased GIMP_VIEWABLE_MAX_PREVIEW_SIZE
	to 2048 to workaround bug #307314.
Comment 7 Sven Neumann 2008-01-23 19:42:55 UTC
The gradient editor doesn't use the pre-allocated buffer any longer. It is only used for the previews and even that is going to change soon. Closing as FIXED.