Issue 89145

Summary:

scroll whell doesn't work in page preview in calc

Product:

Calc

Reporter:

ivanii <ivan.icin>

Component:

Assignee:

Status:

CLOSED FIXED

QA Contact:

issues@sc <issues>

Severity:

Trivial

Priority:

CC:

issues, liujiaxiang, niklas.nebel, peter.junge, yonggang.mao

Version:

OOo 2.4.0

Target Milestone:

---

Hardware:

All

OS:

All

Issue Type:

PATCH

Latest Confirmation in:

---

Developer Difficulty:

---

Issue Depends on:

Issue Blocks:

72764

Attachments:

Description	Flags
The patch file is about i89145#-v1.	none
The patch file is about i89145#-v2.	none
TestCaseSpecification	none

Description ivanii 2008-05-08 00:49:47 UTC

In Excel, if you use scroll with mouse wheel in print preview, you browse
between pages. As your hand is usually on mouse after pressing print preview
button, this is the easiest way to do it (and you are not distracted with
looking for icons when you want to look at the pages to compare them). In OOo it
is just not possible.

Comment 1 chl207 2008-07-06 07:50:39 UTC

This issue can be reproduced in DEV300_m21_en WinXP. 
In addition ,I personally think it may be related to issue 23074.

Comment 2 oc 2008-07-15 10:43:27 UTC

reassigning features and enhancements to user requirements@openoffice.org which
will be the default owner for those tasks (was introduced some time ago)

Comment 3 yonggang.mao 2008-07-16 09:43:06 UTC

started

Comment 4 yonggang.mao 2008-07-16 09:43:34 UTC

accepted

Comment 5 yonggang.mao 2008-07-21 02:44:29 UTC

Accept

Comment 6 yonggang.mao 2008-07-29 08:24:12 UTC

Created attachment 55416 [details]
The patch file is about i89145#-v1.

Comment 7 yonggang.mao 2008-07-29 08:32:14 UTC

*** Issue 23074 has been marked as a duplicate of this issue. ***

Comment 8 yonggang.mao 2008-08-11 15:46:52 UTC

Hi Niklas,
I have submitted a patch, can you review it?

Comment 9 niklas.nebel 2008-08-12 14:53:00 UTC

This breaks the change from issue 7269, which allows continuous scrolling with
the "Page Down" key when the page is bigger than the window (scroll down within
the page, and switch to next page if already at the bottom).

For the scroll wheel, your handling seems better, but we shouldn't change the
behavior of the Page Up/Down keys.

Comment 10 niklas.nebel 2008-08-12 14:57:06 UTC

Also, if there is data on several sheets, with your change, the scroll wheel
only scrolls to the end of the first sheet, because the page count (and with it,
the scroll bar) is initialized only to the first sheet's pages.

Comment 11 yonggang.mao 2008-08-13 16:05:52 UTC

Created attachment 55749 [details]
The patch file is about i89145#-v2.

Comment 12 yonggang.mao 2008-08-13 16:08:46 UTC

Hi Niklas,
This two problems have been solved in patch file.I hope you can review it when 
you have time.

Comment 13 niklas.nebel 2008-08-19 17:56:43 UTC

The new patch looks good.

Comment 14 yonggang.mao 2008-08-21 03:39:51 UTC

The mouse wheel can be scrolled among pages in print preview, Up/Down key can 
be scrolled among pages, Home key can be scrolled the first page,End key can 
be scrolled the last page.

Comment 15 niklas.nebel 2008-10-27 18:31:44 UTC

taking the issue

Comment 16 niklas.nebel 2008-10-27 18:32:12 UTC

I added the last patch to CWS "calc47".

Comment 17 niklas.nebel 2008-12-04 15:40:17 UTC

Reassigning to QA for verification

Comment 18 oc 2008-12-08 13:14:04 UTC

Created attachment 58620 [details]
TestCaseSpecification

Comment 19 oc 2008-12-09 11:39:15 UTC

verified in internal build cws_calc47

Comment 20 sgautier.ooo 2009-01-25 18:55:43 UTC

Verified in DEV300_m39 .deb version - Closing - Sophie