Issue 88437 - Add Zoom Slider to Calc!
Summary: Add Zoom Slider to Calc!
Status: CLOSED FIXED
Alias: None
Product: Calc
Classification: Application
Component: ui (show other issues)
Version: DEV300m8
Hardware: All All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: oc
QA Contact: issues@sc
URL:
Keywords:
: 6352 (view as issue list)
Depends on:
Blocks: 72764
  Show dependency tree
 
Reported: 2008-04-18 13:34 UTC by frank.loehmann
Modified: 2013-08-07 15:14 UTC (History)
4 users (show)

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

Attachments
The patch file is about i88437#-v1. (10.28 KB, text/plain)
2008-07-01 08:51 UTC, yonggang.mao 		no flags Details
About i88437#-v2.patch (12.79 KB, text/plain)
2008-07-09 04:24 UTC, yonggang.mao 		no flags Details
The patch file is about i88437#-v3. (12.90 KB, text/plain)
2008-07-18 04:24 UTC, yonggang.mao 		no flags Details
TCS for zoom slider in spreadsheet (7.75 KB, text/html)
2008-12-08 10:01 UTC, oc 		no flags Details
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 frank.loehmann 2008-04-18 13:34:57 UTC 
Add Writer zoom slider to Calc.
Comment 1 frank.loehmann 2008-04-29 08:55:40 UTC 
*** Issue 6352 has been marked as a duplicate of this issue. ***
Comment 2 yonggang.mao 2008-06-26 16:46:21 UTC 
accept
Comment 3 yonggang.mao 2008-06-30 03:22:53 UTC 
Started
Comment 4 yonggang.mao 2008-07-01 08:51:57 UTC 
Created attachment 54851 [details]
The patch file is about i88437#-v1.
Comment 5 yonggang.mao 2008-07-01 08:56:14 UTC 
Hi Niklas,
I have finished the zoom slider in normal mode of calc.Please review the patch 
file.thanks!
Comment 6 yonggang.mao 2008-07-09 04:24:39 UTC 
Created attachment 55018 [details]
About i88437#-v2.patch
Comment 7 yonggang.mao 2008-07-09 04:28:45 UTC 
Hi Niklas,
I have finished the slider control in preview mode of calc.I have tested it,it 
can work well.Please review the patch file! thanks.
Comment 8 niklas.nebel 2008-07-17 12:57:04 UTC 
When a new zoom value is set, the zoom type should also be changed
("SetZoomType( SVX_ZOOM_PERCENT, bSyncZoom )" in tabvwsh3, "eZoom =
SVX_ZOOM_PERCENT" in prevwsh). Otherwise, the patch looks good.
Comment 9 yonggang.mao 2008-07-18 04:24:37 UTC 
Created attachment 55198 [details]
The patch file is about i88437#-v3.
Comment 10 niklas.nebel 2008-07-18 17:51:07 UTC 
The last patch looks good and ready for integration.
Comment 11 niklas.nebel 2008-10-27 17:42:18 UTC 
taking the issue
Comment 12 niklas.nebel 2008-10-27 17:42:57 UTC 
I added the last patch to CWS "calc47".
Comment 13 niklas.nebel 2008-12-04 15:41:57 UTC 
Reassigning to QA for verification
Comment 14 oc 2008-12-08 10:01:17 UTC 
Created attachment 58616 [details]
TCS for zoom slider in spreadsheet
Comment 15 oc 2008-12-09 11:29:31 UTC 
verified in internal build cws_calc47
Comment 16 sgautier.ooo 2009-01-25 18:42:43 UTC 
Verified in DEV300_m39 .deb version, closing - Sophie