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 99954 - gnome_canvas_item_get_bounds does not work with a Rich Text item
gnome_canvas_item_get_bounds does not work with a Rich Text item
Status: RESOLVED FIXED
Product: libgnomecanvas
Classification: Deprecated
Component: core
2.0.x
Other Linux
: High normal
: ---
Assigned To: libgnomecanvas maintainers
Federico Mena Quintero
Depends on:
Blocks:
 
 
Reported: 2002-11-30 09:52 UTC by Jean Bréfort
Modified: 2005-05-19 13:28 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
a patch to fix this bug (1.74 KB, patch)
2004-01-21 06:52 UTC, Jean Bréfort 		needs-work Details | Review
Oops! My first patch had bugs. Here is a new one which should fix the problem. (1.72 KB, patch)
2004-01-21 11:10 UTC, Jean Bréfort 		none Details | Review

Description Jean Bréfort 2002-11-30 09:52:50 UTC 
Calling gnome_canvas_item_get_bounds with a GnomeCanvasRichText return
bounds all equal to 0.

Adding a line like:
      item_class->bounds = get_bounds;

in gnome_canvas_rich_text_class_init along with an appropriate header seems
to fix the problem. May be get_bounds should be renamed
gnome_canvas_rich_text_get_bounds?

Jean Bréfort <jeau.brefort@ac-dijon.fr>
Comment 1 Kjartan Maraas 2004-01-20 13:13:32 UTC 
Is this still wrong in the current sources?
Comment 2 Jean Bréfort 2004-01-21 06:52:49 UTC 
Created attachment 23585 [details] [review]
a patch to fix this bug
Comment 3 Jean Bréfort 2004-01-21 11:10:00 UTC 
Created attachment 23590 [details] [review]
Oops! My first patch had bugs. Here is a new one which should fix the problem.
Comment 4 Kjartan Maraas 2004-01-23 20:09:23 UTC 
Please review the patch. Looks good to me, but I'm hardly the canvas
expert here.
Comment 5 Kjartan Maraas 2004-03-29 10:24:10 UTC 
Please, someone review this patch?
Comment 6 Tim Janik 2005-05-16 14:05:39 UTC 
i can't really make an authoritative judgement here, i have too little knowledge
on the text item. if you want a call though, apply it, it most probably improves
things and doesn't make them worse ;)
Comment 7 Kjartan Maraas 2005-05-19 13:28:46 UTC 
Ok. Commiting this on HEAD at least then.