To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=77648
------- Additional comments from [EMAIL PROTECTED] Thu May 31 14:41:42 +0000
2007 -------
Unfortunately I do not have the time to test your patch in real life, but I had
a look at the code. I think that it solves the situation I mentioned so far,
but...
I could imagine more problems:
1. Please check the "Redo" functionality; I doubt it will work for cells with
more than one paragraph.
2. Please check the Undo if your cell contains "Hello world" with an attribute
like 'bold' attached to the second word ('world'), I fear this attribute will be
deleted by "Undo"?!
The solution:
The old code did two things if it found one paragraph, it changed the text and
it changed the cell format and recorded both changes.
You could separate this two items ( box format changes, text changes).
The box format changes should be done always, undoable and redoable. The text
change should only be done, if IsValidNumTxtNd() does not delivers ULONG_MAX. In
this case, all the text node related code has to be done in Ctor, Undo and Redo.
---------------------------------------------------------------------
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
