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 53113 - Cell comments not exported to XL97/2k/XP
Cell comments not exported to XL97/2k/XP
Status: RESOLVED FIXED
Product: Gnumeric
Classification: Applications
Component: import/export MS Excel (tm)
git master
Other All
: Normal normal
: ---
Assigned To: Jody Goldberg
Jody Goldberg
: 76460 108990 124311 126219 136726 347536 469373 517475 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2001-04-11 18:25 UTC by chris
Modified: 2008-02-19 16:12 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description chris 2001-04-11 18:25:16 UTC 
I have a Excel 2000 spreadsheet with comment/notes on it. When importing to
gnumeric none of the notes are imported.
Comment 1 Morten Welinder 2001-04-12 00:32:58 UTC 
We haven't implemented this yet.

(Also, currently comments cause lots of leaks during import.)
Comment 2 Jody Goldberg 2001-04-12 04:22:39 UTC 
We'll get to it, but there are several things ahead of this.
Comment 3 Jody Goldberg 2002-10-14 06:09:29 UTC 
*** Bug 76460 has been marked as a duplicate of this bug. ***
Comment 4 Morten Welinder 2003-03-25 20:06:36 UTC 
*** Bug 108990 has been marked as a duplicate of this bug. ***
Comment 5 Andreas J. Guelzow 2003-07-22 02:32:01 UTC 
The NEWS file includes: 

	* Import cell comments from >= xl97.

Does that means that this has been implemented in current CVS?
Comment 6 Jody Goldberg 2003-07-22 02:41:00 UTC 
yes.  Thanks for remembering this bug.
We only export them to XL95, and the current comment implementation is
... (/me searches for a polite phrase) sub-optimal.  However, the
worst of the heavy lifting is done.  I'm confident we understand the
format now and could generate it with a couple hours of work.

To make this truely useful we need a few things
1) To split the comment indicator and the comment display box objects
2) To add support for rich text import (could be useful in lots of places)
3) xl97 export
Comment 7 Morten Welinder 2003-10-10 19:31:30 UTC 
*** Bug 124311 has been marked as a duplicate of this bug. ***
Comment 8 Morten Welinder 2003-11-04 22:24:26 UTC 
*** Bug 126219 has been marked as a duplicate of this bug. ***
Comment 9 Andreas J. Guelzow 2004-03-10 23:51:47 UTC 
*** Bug 136726 has been marked as a duplicate of this bug. ***
Comment 10 Jon Kåre Hellan 2004-08-19 20:34:57 UTC 
Is this still low priority?
Comment 11 Jody Goldberg 2004-08-19 22:11:29 UTC 
jon_kare : It's certainly a desireable enhancement, but it will be alot of work.

The comment framework needs some adjustments to do this right.  Comments are
really just special cases of labels.  In XL97 they store the text object's
position along with markup.  Then the comment itself is associated with a cell
seperately.  In Gnumeric the cell, and the text box position are intertwined.

We can kludge a solution that will just guess as position for export.  Or we can
rework the comment object to inherit from the text box which is a heck of alot
of work.
Comment 12 Andreas J. Guelzow 2006-07-16 06:07:54 UTC 
see 347536
Comment 13 Andreas J. Guelzow 2006-07-17 21:44:35 UTC 
*** Bug 347536 has been marked as a duplicate of this bug. ***
Comment 14 Andreas J. Guelzow 2006-07-17 21:45:55 UTC 
because of 347536 I am changing the status of this from enhancement to bug. If we claim that we support the XL format, people can expect that we handle comments too.
Comment 15 Bengt Olsson 2006-12-23 10:24:56 UTC 
This bug is the sole reason I can't use Gnumeric at work. Comments often contains mission critical information. Don't think I'm alone on this. As a data loss bug it should have higher priority/severity.
Comment 16 Morten Welinder 2006-12-27 02:26:18 UTC 
The WriteExcel perl module supports this and the code doesn't look too
difficult to follow.
Comment 17 Jody Goldberg 2007-03-05 23:31:15 UTC 
Some code is in svn now, and will be in the next release (1.7.9).  There are some improvements to make to store the background colour properly before I close this.
Comment 18 Andreas J. Guelzow 2007-08-22 20:43:37 UTC 
*** Bug 469373 has been marked as a duplicate of this bug. ***
Comment 19 Morten Welinder 2008-02-19 16:12:21 UTC 
*** Bug 517475 has been marked as a duplicate of this bug. ***