Issue 79965 - Editing a chart title moves the page offset but not back, when finishing editing
Summary: Editing a chart title moves the page offset but not back, when finishing editing
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: kla
QA Contact: issues@graphics
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-07-24 12:52 UTC by bjoern.milcke
Modified: 2013-02-24 21:22 UTC (History)
3 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this issue.
Description bjoern.milcke 2007-07-24 12:52:45 UTC
1. Create a chart, e.g. in Impress
2. Enter a main title (Insert/Title)
3. Double-Click the title for editing
4. Enter many characters until the cursor reaches the right end of the page.
When entering more characters, the rest of the chart moves to the left.
5. Click outside the title to end the edit-mode (some space inside the chart object)
=> The chart stays scrolled, i.e. some parts of the chart are no longer visible,
and you cannot scroll back.

Work-around: End and Re-enter the edit mode (click into the container document
and double-click the chart again).
Comment 1 bjoern.milcke 2007-07-24 14:41:11 UTC
Note for the work-around: s/edit mode/inplace mode/
Comment 2 Armin Le Grand 2007-07-30 11:49:03 UTC
AW: Created a patch, discussed with RT. May go into m223.
AW->RT: As discussed, set to fixed when in master. Thanks!
Comment 3 rt 2007-07-30 11:59:11 UTC
aw: Really? We were talking about issue 80172, I think.
Comment 4 rt 2007-07-30 12:00:43 UTC
Back to Armin.
Comment 5 Armin Le Grand 2007-07-30 12:01:55 UTC
AW: OOps, wrong task moved. Back to me!
Comment 6 Armin Le Grand 2007-08-06 14:01:52 UTC
AW->CL: Do You have any idea who is responsible for the visible area of an OLE
and who has to set it then inplace mode is left? Framework? The OLE-Application?
The host Application? I hope You know more, if not maybe we ask MBA or Mikahil?
Comment 7 clippka 2007-11-12 13:38:21 UTC
CL->AW: I don't think this has something to do with OLE. If you deactivate the
edit mode then the vis area will be reset and is correct now. I think this is a
problem that editing somehow translates the SdrView and since there are no
scroll bars in chart it can not be scrolled back.
This is the same behavior if you create a rectangle shape in impress and start
typing. The View scrolls left and after editing stays this way.
CL->BM: I think a solution for your problem is to override SdrEndTextEdit and
reset the SdrView position
Comment 8 bjoern.milcke 2007-12-05 10:27:57 UTC
SdrEndTextEdit is non-virtual. Is there a different way to intercept, like an
event when editing has finished? I noticed that in Calc there is no scrolling at
all when the cursor is moved out of the visible region in shape text edit. This
is not very fortunate, but obviously there is a way to intervene here.
Comment 9 Armin Le Grand 2007-12-05 10:53:27 UTC
AW->BM: SdrEndTextEdit (and begin, too) are not virtual by historical reasons.
Since they are NOT virtual, it is not trivial how they call/get called by each
other (e.g. SD calls it in several places, but which...?). CL and i both did one
try to make these virtual, but both did not succeed and the changed versions
leaded to problems in one or another case. So: Do not touch this not very stable
region, please.
Anyways, You need to save the view and restore it at EndTextEdit. If it is
enough to react on EndTextEdit() and reset the view, i just took a look in
svx\source\svdraw\svdedxv.cxx (888) ff and there is indeed an event broadcasted
(look for #99840#, someone else seems to have needed it already).
If You need to save and restore, there is also a hint sent (HINT_BEGEDIT, look
for same taskID) for that.
HTH.
Comment 10 bjoern.milcke 2007-12-05 12:24:24 UTC
Yes, this helped, thanks. Now, I remember the first output devices' map mode in
begin text edit and reset this one in end text edit.
Comment 11 bjoern.milcke 2007-12-13 15:25:45 UTC
Please verify in CWS chart15
Comment 12 bjoern.milcke 2007-12-13 15:42:29 UTC
Please verify in CWS chart15.
Comment 13 kla 2007-12-19 15:01:27 UTC
Looks fine in cws chart15 -> verified
Comment 14 p9w.vu.31122010 2008-02-07 10:30:07 UTC
Verified fixed in m6, closing.