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 305046 - GThumb prints comments incorrectly wrapping wrong
GThumb prints comments incorrectly wrapping wrong
Status: RESOLVED FIXED
Product: gthumb
Classification: Other
Component: general
2.9.x
Other Linux
: High normal
: ---
Assigned To: Paolo Bacchilega
Paolo Bacchilega
Depends on:
Blocks:
 
 
Reported: 2005-05-21 22:52 UTC by William
Modified: 2006-11-01 14:06 UTC
See Also:
GNOME target: ---
GNOME version: 2.9/2.10


Attachments
Patch to fix comment wrapping, comment saving, and series renaming. (18.40 KB, patch)
2006-10-21 19:20 UTC, Michael Chudobiak
none Details | Review

Description William 2005-05-21 22:52:37 UTC
Distribution/Version: Gentoo

When I print a picture with a long comment, it wraps only part of the word
instead of the whole word. For example:

A string like this might become

A string like this mi
ght become
Comment 1 Michael Chudobiak 2006-10-19 16:51:27 UTC
Confirmed in CVS HEAD (2.9.0). This is actually pretty annoying. Marking priority as "high".

- Mike
Comment 2 Michael Chudobiak 2006-10-19 17:16:13 UTC
I think the function pci_get_next_line_to_print_delimiter in libgthumb/print-callbacks.c needs to be modified so that it returns the string and line width only up to the last space before max_width. 

Right now, it stops at the first newline or the first character after max_width.

- Mike
Comment 3 Michael Chudobiak 2006-10-21 19:20:26 UTC
Created attachment 75151 [details] [review]
Patch to fix comment wrapping, comment saving, and series renaming.

Paolo,

Here is a patch that:

1. Fixes bug 305046, by making printed comments wrap between words, rather than blindly wrapping in the middle of words.

2. Fixes bug 338623 and bug 339037, by adding a "%e" file-extension symbol when renaming a series of files. This allows extensions to be changed, and fixes a crash when files without extensions were renamed.

3. Fixes bug 310647. The "Close" and "Save" buttons in the comments dialog have been changed to "Cancel" and "OK". The save is automatic when OK is clicked. I find this much more intuitive. Similar changes were made to the categories dialog.
Comment 4 Michael Chudobiak 2006-10-21 20:50:59 UTC
Paolo,

I forgot to say "Please commit".

- Mike
Comment 5 Michael Chudobiak 2006-11-01 14:06:05 UTC
Patch committed to CVS HEAD.