Issue 66176 - new chart: Cancel in Wizard in Writer does not work
Summary: new chart: Cancel in Wizard in Writer does not work
Status: CLOSED FIXED
Alias: None
Product: General
Classification: Code
Component: chart (show other issues)
Version: 3.3.0 or older (OOo)
Hardware: All All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: michael.ruess
QA Contact: issues@graphics
URL:
Keywords: new_implementation
Depends on:
Blocks:
 
Reported: 2006-06-06 17:17 UTC by bjoern.milcke
Modified: 2013-02-24 21:20 UTC (History)
2 users (show)

See Also:
Issue Type: TASK
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description bjoern.milcke 2006-06-06 17:17:31 UTC 
Pressing "Cancel" in the Wizard in Writer inserts a chart. As we have a "live"
preview, a chart should be inserted when the Wizard opens, but this must be
deleted when "Cancel" is pressed.
Comment 1 bjoern.milcke 2006-06-06 17:18:29 UTC 
Keyword added.
Comment 2 bjoern.milcke 2006-06-23 10:02:56 UTC 
Changed target to 2.x
Comment 3 bjoern.milcke 2006-07-24 15:25:41 UTC 
->TL: Sorry, I thought I had this issue re-assigned to you.
Please implement "Cancel" for the Wizard. As you know, with the new wizard for
charts, a new chart is inserted into the document before the dialog itself
opens. Then you have the live-preview. If you press "Cancel" in the dialog, it
is no longer sufficient to just close it, but you must remove the previously
inserted chart.
I tried to do that by using the Undo-Action that is created, but calling Undo
does not what I expected. I think it is more complicated to correctly remove the
chart from the Writer model, and also delete the Undo-Action, so that it does
not appear in the Undo later.
Comment 4 thomas.lange 2006-08-03 11:38:09 UTC 
.
Comment 5 thomas.lange 2006-08-03 13:16:19 UTC 
Fixed in CWS chart2mst3.

Files changed:
- sw/source/core/unocore/unoobj.cxx
- sw/source/ui/table/chartins.cxx
Comment 6 thomas.lange 2006-12-13 15:10:39 UTC 
Fixed in CWS chart2mst3 milestone10.
TL->MRU: Please check. Thanks!
Comment 7 IngridvdM 2007-01-18 11:54:52 UTC 
changed target to 2.3
Comment 8 thomas.lange 2007-02-19 12:23:09 UTC 
Works fine in latest build.
Comment 9 michael.ruess 2007-03-28 14:00:36 UTC 
Verified fix in CWS chart2mst3.
Comment 10 michael.ruess 2007-06-21 15:44:14 UTC 
Checked in 680m214.