Issue 51656

Summary: Add resizeable margin on page preview
Product: Calc Reporter: utomo99 <utomo.prawiro>
Component: formattingAssignee: oc
Status: CLOSED FIXED QA Contact: issues@sc <issues>
Severity: Trivial    
Priority: P3 CC: chengxiuzhi, cno, issues, kamataki, liujiaxiang, niklas.nebel, pagalmes.lists, yonggang.mao
Version: OOo 1.1.4Keywords: oooqa
Target Milestone: ---   
Hardware: All   
OS: All   
Issue Type: PATCH Latest Confirmation in: ---
Developer Difficulty: ---
Issue Depends on:    
Issue Blocks: 72764    
Attachments:
Description Flags
margin resizeable on excel
none
It is the specification for the design, please take a look at it
none
the patch for the SC resizeable margin in Sc page preview
none
The patch is about page margins!
none
The patch file is about page margins!
none
The patch file is about issue51656#.
none
The patch file is about i51656#.
none
patch-i51656#(v5)
none
TestCaseSpecification none

Description utomo99 2005-07-07 07:40:31 UTC
please Add resizeable margin on page preview. so we can easily formating the 
documents when we need to print. 
look at the excel which have this feature
Comment 1 utomo99 2005-07-07 08:02:44 UTC
Created attachment 27754 [details]
margin resizeable on excel
Comment 2 frank 2005-07-18 14:16:26 UTC
one for requirements
Comment 3 frank 2005-09-23 14:57:35 UTC
*** Issue 54344 has been marked as a duplicate of this issue. ***
Comment 4 liujiaxiang 2007-03-12 09:12:32 UTC
Hello ALL,
What is the situation of this new feature?
Where can I have the Specification?
Our team(RedOffice Team)have written a Specification and want to implement 
this new feature.Any one can give me some useful information?
Comment 5 niklas.nebel 2007-03-12 11:22:16 UTC
LiuTao,
Gao Zemin (of your team) announced to start with this feature. Is it the same
effort?
You can add the spec as an attachment to this issue, or start a wiki page.
Comment 6 liujiaxiang 2007-03-13 03:18:25 UTC
LiuTao->nn :yes,of course.It is the same effort.
I will start a wiki page ASAP.
Comment 7 jacky23 2007-04-04 07:00:13 UTC
Created attachment 44195 [details]
It is the specification for the design, please take a look at it
Comment 8 frank 2007-04-12 13:59:57 UTC
Hi Matthias,

please have a look at the attached Specification.

Frank
Comment 9 jacky23 2007-04-19 03:04:07 UTC
Created attachment 44532 [details]
the patch for the SC resizeable margin in Sc page preview
Comment 10 niklas.nebel 2007-04-24 18:35:18 UTC
Please always make diffs against the revisions that you actually edited, and use
some consistent indentation.

Some observations:

- Instead of the TWIPS_PER_PIXEL define, you should use OutputDevice's
PixelToLogic method, to avoid small inaccuracies.

- The column handles look quite irregular because they're not aligned to pixel
boundaries. They don't take a printing scale into account (this is also true for
the header size), and they don't handle hidden columns.

- The should be some kind of feedback while dragging one of the lines.

But, before all of this, has it been decided yet if we want something
ruler-based instead?
Comment 11 yonggang.mao 2007-12-21 12:19:12 UTC
Created attachment 50529 [details]
The patch is about page margins!
Comment 12 utomo99 2007-12-21 12:26:38 UTC
The patch is for SC only or also for others ? such as writer ? 
I hope we can have one patch for all. 

Thanks
Comment 13 yonggang.mao 2008-01-09 07:49:36 UTC
Created attachment 50753 [details]
The patch file is about page margins!
Comment 14 yonggang.mao 2008-01-15 10:28:29 UTC
Hello Niklas,
Please have a look at the patch file,thanks!
Comment 15 niklas.nebel 2008-01-16 17:15:52 UTC
The new patch looks better. There are still some problems:

- Simple print scaling works now, but not "Fit print range(s) on number of pages".

- ScPrintFunc::GetRange doesn't skip empty pages (the problem mentioned by
yunjingtao on IRC last week).

- Right-to-left sheet orientation doesn't work.
Comment 16 niklas.nebel 2008-01-16 18:05:29 UTC
One more: Repeated columns don't work either. Maybe it's better to use
ScPreviewLocationData (now used for the accessibility API) instead of
re-implementing all these options in ScPrintFunc::GetRange.
Comment 17 yonggang.mao 2008-01-24 09:06:48 UTC
Created attachment 51129 [details]
The patch file is about issue51656#.
Comment 18 yonggang.mao 2008-01-24 09:12:08 UTC
Hello Niklas,
Please review the latest patch,I'm fixing that when nNewColWidth is 
negative.When I finished it,I will commit the other patch.thanks!
Comment 19 niklas.nebel 2008-01-25 18:04:26 UTC
The handles look much better now. "Scale to pages" also works. If there are
repeated columns, only the following columns have handles, but I think that's
ok. Only right-to-left still doesn't work completely.

Frank from the User Experience team agreed to the look of the feature, so if the
few remaining problems are solved, we can integrate it soon.
Comment 20 yonggang.mao 2008-01-30 08:10:48 UTC
Created attachment 51247 [details]
The patch file is about i51656#.
Comment 21 yonggang.mao 2008-01-30 15:40:41 UTC
Hello Niklas,
I just have finished "Reduce/enlarge printout" problem,when dragging the page 
changes into several pages that also have finished,I will commit it tomorrow.
Comment 22 yonggang.mao 2008-02-01 06:42:05 UTC
Created attachment 51302 [details]
patch-i51656#(v5)
Comment 23 niklas.nebel 2008-02-01 15:07:02 UTC
reassigning to myself
Comment 24 niklas.nebel 2008-02-01 15:08:42 UTC
The feature is on CWS "pagemargins".
Comment 25 niklas.nebel 2008-02-01 16:54:10 UTC
back to QA for verification
Comment 26 oc 2008-02-12 10:00:05 UTC
Created attachment 51472 [details]
TestCaseSpecification
Comment 27 oc 2008-02-12 10:01:04 UTC
verified in internal version cws_pagemargins
Comment 28 frank.loehmann 2008-02-12 15:13:09 UTC
Please find the final spec here:
http://specs.openoffice.org/calc/features/Page_Margins_Specification.odt
Comment 29 maddox2 2008-02-18 18:36:42 UTC
Tested margins in m247 (Pavels Build) and found 2 issues.

1. The Word margin is not (yet) translated.

2. Changing margins don't raise the "File changed" flag. 
Comment 30 niklas.nebel 2008-06-16 09:26:12 UTC
*** Issue 11395 has been marked as a duplicate of this issue. ***
Comment 31 amy2008 2009-04-13 09:07:40 UTC
Verified in OOo310m9 on WinXP and Fedora9
Closing
Li Meiying