To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=16615





------- Additional comments from [email protected] Tue Jun  2 19:30:34 +0000 
2009 -------
Some comments about the patch:

- You have to call SetDocumentModified, for "modified" state and change
notification.

- Cell protection has to be checked (IsEditable) for all formulas, or simply for
the selection, but only the cursor position isn't enough.

- Undo with tracked changes removes only the last change. You have to store the
start and end position in the tracked changes (see nStartChangeAction in
ScUndoReplace, for example).

- Most other undo actions use a ScDocument to store multiple cells (but that's
just a suggestion).

- The method name "DoRefConvertion" is wrong (should be "DoRefConversion").

---------------------------------------------------------------------
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]

Reply via email to