Issue 86894

Summary: Cell range selection not highlighted when pasted - approved
Product: Calc Reporter: kyoshida
Component: codeAssignee: oc
Status: CLOSED FIXED QA Contact: issues@sc <issues>
Severity: Trivial    
Priority: P2 CC: issues
Version: OOH680m9Keywords: regression
Target Milestone: ---   
Hardware: All   
OS: Linux, all   
Issue Type: PATCH Latest Confirmation in: ---
Developer Difficulty: ---
Issue Depends on:    
Issue Blocks: 88258    
Attachments:
Description Flags
simple change to fix this regression none

Description kyoshida 2008-03-11 01:16:16 UTC 
When pasting a cell range, Calc no longer highlights the pasted range even
though the cell range is logically selected.

Steps to reproduce:

1. Select A1:C3, and press Ctrl-C.
2. Move the cursor to D5.
3. Press Ctrl-V.

The cell range gets pasted, but the range is not highlighted.  Note that even in
this circumstance, the bottom-right marker is displayed & the subsequent ENTER
key presses navigates the cell cursor within the pasted range.

I've tested this using OOH680 m10.
Comment 1 kyoshida 2008-03-11 01:18:36 UTC 
I think this should be fixed for 2.4.  It's a regression.
Comment 2 niklas.nebel 2008-03-11 10:01:58 UTC 
For 2.4 we only include blocker issues as determined by the release status
meeting. I don't think we have to further delay the release because of this issue.
Comment 3 kyoshida 2008-03-11 19:52:20 UTC 
Created attachment 52038 [details]
simple change to fix this regression
Comment 4 kyoshida 2008-03-11 19:54:35 UTC 
I cherry-picked from calcselection the minimum change necessary to fix this
regression.  Changing the issue type to PATCH.
Comment 5 pavel 2008-03-13 20:43:53 UTC 
Move target.
Comment 6 kyoshida 2008-03-13 20:56:53 UTC 
@pjanik: this one is only relevant to 2.4.  3.0 doesn't suffer from this
regression.  So, the target should change to 2.4.1 which doesn't exist yet.
Comment 7 kyoshida 2008-03-13 21:58:27 UTC 
setting target back to 2.4.  Please reset target to 2.4.1 once that target
number becomes available.
Comment 8 Martin Hollmichel 2008-03-14 20:06:15 UTC 
at this stage of release I suggest to choose target 2.4.1 for this.

@nn: can you please review the suggested fix
Comment 9 ooo 2008-03-17 14:13:04 UTC 
Niklas is on vacation. To me the patch looks viable, but I'm not familiar with
the selection overlay and whether calling an update during cut/paste might be
harmful.
Comment 10 bettina.haberer 2008-04-07 14:22:41 UTC 
Set keyword regression.
Comment 11 uwe.luebbers 2008-04-29 13:18:30 UTC 
added "approved" to the title, because it will be easier to work with the 2.4.1 meta issue 
during release status meetings.
Comment 12 niklas.nebel 2008-04-29 18:52:16 UTC 
I committed this to CWS "pasteselection". I also added the corresponding call
for "Find all" (issue 88244, caused by the same change).
Comment 13 niklas.nebel 2008-04-29 18:54:08 UTC 
*** Issue 88244 has been marked as a duplicate of this issue. ***
Comment 14 niklas.nebel 2008-04-30 09:39:02 UTC 
reassigning to QA for verification
Comment 15 oc 2008-05-06 08:42:14 UTC 
verified in internal build cws_pasteselection
Comment 16 oc 2008-09-19 10:28:58 UTC 
closed because fix available in OOo2.4.1