Issue 89145 - scroll whell doesn't work in page preview in calc
Summary: scroll whell doesn't work in page preview in calc
Alias: None
Product: Calc
Classification: Application
Component: ui (show other issues)
Version: OOo 2.4.0
Hardware: All All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: oc
QA Contact: issues@sc
: 23074 (view as issue list)
Depends on:
Blocks: 72764
  Show dependency tree
Reported: 2008-05-08 00:49 UTC by ivanii
Modified: 2013-08-07 15:14 UTC (History)
5 users (show)

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

The patch file is about i89145#-v1. (13.67 KB, text/plain)
2008-07-29 08:24 UTC, yonggang.mao
no flags Details
The patch file is about i89145#-v2. (14.37 KB, text/plain)
2008-08-13 16:05 UTC, yonggang.mao
no flags Details
TestCaseSpecification (6.89 KB, text/html)
2008-12-08 13:14 UTC, oc
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
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 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
Comment 4 yonggang.mao 2008-07-16 09:43:34 UTC
Comment 5 yonggang.mao 2008-07-21 02:44:29 UTC
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]
Comment 19 oc 2008-12-09 11:39:15 UTC
verified in internal build cws_calc47
Comment 20 2009-01-25 18:55:43 UTC
Verified in DEV300_m39 .deb version - Closing - Sophie