Issue 70991

Summary: row height extends to page size when dragging border with mouse in table breaking across pages
Product: Writer Reporter: liujiaxiang
Component: codeAssignee: AOO issues mailing list <issues>
Status: CONFIRMED --- QA Contact:
Severity: Trivial    
Priority: P3 CC: issues, pagalmes.lists, peter.junge
Version: OOo 2.0.3   
Target Milestone: ---   
Hardware: All   
OS: Windows, all   
Issue Type: DEFECT Latest Confirmation in: ---
Developer Difficulty: ---
Issue Depends on:    
Issue Blocks: 72764    
Attachments:
Description Flags
the patch of this bug none

Description liujiaxiang 2006-10-30 09:07:37 UTC
In the OOo 2.0.3 
change the height of the row (the last row of a table be divided into two 
pages.)make the row is full of the page. 
The bug can be described as follows : 
1. build a new word-processing document
2. insert a table
3. Input Enter(s) in the random cell of the last row of the table and make the 
cell be dvivded into two pages.
4. locate the cursor in another cell of the first page.
5. change the height of the last row of the second page by Mouse.
you will find the row is full of the second page.
I have repaired the bug.
Comment 1 michael.ruess 2006-10-30 09:48:20 UTC
MRU->FME: insert a table into document and add content to the last row so that
the table braks onto next page. Place Cursor in a different cell on page 1. Now
change the height of the last row by dragging the bottom table border -> row now
covers whole page
Comment 2 liujiaxiang 2006-11-03 00:57:35 UTC
I have repaired the bug.It is something wrong with file ndtbl.cxx.
Comment 3 frank.meies 2006-11-03 06:44:44 UTC
fme->liutao: Sounds good. Can you please attach the patch to this issue?
Comment 4 liujiaxiang 2006-11-06 06:49:30 UTC
Created attachment 40352 [details]
the patch of this bug
Comment 5 frank.meies 2006-11-06 08:10:26 UTC
fme->liutao: Thank you for your support. I had a look at your patch. Please
correct me if I'm wrong, but in both cases, if and else, the
SetLastRowAllowedToChange flag at the table will be set to true, so basically
your patch replaces 

rFill.SetLastRowAllowedToChange( !pTab->HasFollowFlowLine() );	

with

rFill.SetLastRowAllowedToChange( true );

The meaning of the SetLastRowAllowedToChange flag is this: If the last row of
the current table frame is split across two pages, it should not be allowed to
change the height of the last row. Your patch might work in your scenario, but I
think this issue needs some deeper digging. 
Comment 6 Martin Hollmichel 2006-12-19 15:43:24 UTC
setting the issue type to patch
reassign back to liutao for rework as fme suggested.
set taget to not determined as it is of type patch now.
Comment 7 peter.junge 2006-12-20 01:58:55 UTC
added myself to CC
Comment 8 Mathias_Bauer 2006-12-21 08:59:38 UTC
liutao, this patch seems to be insufficient. Can we expect that it will be
reworked? Or can we close this issue?
Comment 9 peter.junge 2006-12-26 06:06:28 UTC
Matthias,
MRU confirmed the Issue. It cannot be closed due to a failed patch was proposed.
-> Reset to defect
Is 3.0 a acceptable target?

Peter
Comment 10 Mathias_Bauer 2007-03-20 17:43:28 UTC
I think compared to other defects this issue doesn't look so important. As we
already have several hundred defects for target 2.x+3.0 I think "OOoLater" is
more appropriate as we most probably will not have time to work on it. 

Of course if other developers like to work on this issue and come up with a
patch we will help to integrate it.
Comment 11 Marcus 2017-05-20 11:25:00 UTC
Reset assigne to the default "issues@openoffice.apache.org".
Comment 12 Marcus 2017-05-20 11:25:57 UTC
Reset assigne to the default "issues@openoffice.apache.org".