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 494049 - painting doesn't update the histogram
painting doesn't update the histogram
Status: RESOLVED FIXED
Product: GIMP
Classification: Other
Component: General
2.4.x
Other All
: Normal normal
: 2.4
Assigned To: GIMP Bugs
GIMP Bugs
Depends on:
Blocks:
 
 
Reported: 2007-11-06 08:10 UTC by Sven Neumann
Modified: 2008-10-30 20:03 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
changed paint core to update the drawable (4.66 KB, patch)
2007-11-06 08:13 UTC, Sven Neumann
none Details | Review
also change the histogram to use a timeout (5.38 KB, patch)
2007-11-06 08:22 UTC, Sven Neumann
committed Details | Review

Description Sven Neumann 2007-11-06 08:10:34 UTC
Due to the changes made to fix bug #490182, the histogram is not any longer updated when paint tools are used on the image. This is because the histogram now listens for drawable updates and not any longer for the preview becoming invalid.
Comment 1 Sven Neumann 2007-11-06 08:13:57 UTC
Created attachment 98641 [details] [review]
changed paint core to update the drawable

This patch changes the paint core to update the drawable instead of the image. In order to suppress an update of the drawable preview, the new freeze/thaw functionality is used.

This makes the code cleaner and it fixes the problem. However, it causes the histogram to be updated while the user paints. This happens from an idle handler, but it might still be updating too often. Perhaps we should use a short timeout instead?
Comment 2 Sven Neumann 2007-11-06 08:22:48 UTC
Created attachment 98642 [details] [review]
also change the histogram to use a timeout

This patch seems to work nicely. Additionally to the paint-core changes, it changes the histogram to use a short timeout instead of an idle handler.
Comment 3 Sven Neumann 2007-11-13 11:16:04 UTC
I have applied the change to both branches:

2007-11-13  Sven Neumann  <sven@gimp.org>

        Fix for bug #494049 (painting doesn't update the histogram):

        * app/paint/gimppaintcore.c: freeze the drawable preview while we
        are painting. Update the drawable instead of the image.

        * app/widgets/gimphistogrameditor.c: use a short timeout instead
        of an idle handler to update the histogram.