Apache OpenOffice (AOO) Bugzilla – Issue 64391
Column of complex table can be dragged out of view
Last modified: 2013-08-07 14:43:03 UTC
1 insert a 3*3 table 2 change the columns' width to 9.39cm, 1.94cm and 5.67cm 3 merge cell C2 and C3 4 change the column by mouse such the attachment "before.jpg" 5 find the table is wrong such attachment "after.gif"
Created attachment 35692 [details] before change column's width
Created attachment 35693 [details] after change column's width
confirmed with 680_m162 on WinXP Pro SP2
MRU->OS: as described, create a 3x3 table, merge cells C2 and C3. Then drag (with mouse) the left column's border close to its right border. The vertical border of A2.2.1:A2.2.2 seems to be moved right over the page border.
Target adjusted and reassigned
move to target 3.x according http://wiki.services.openoffice.org/wiki/Target_3x
lijian->fme: This issue has not existed since (maybe) m223 which uses new table. Please check this.
fme->lijian: Anyway, the 'old' table model will still be used if you create a table with an old version of OOo. So you have to create a bugdoc with an older version.
lijian->fme: So I will work on this issue. :-)
Created attachment 50254 [details] patch file
lijian->fme: Please have a look at this patch. Thank you !
fme->lijian: I like your idea that the first box in the last line of an upper cell takes care that the upper cell remains in the array. But your code currently does only work if the left table border has been moved. But the whole scenario might take place somewhere in the middle of a table.
Created attachment 50636 [details] patch 2 for this issue
lijian->fme: Please review this patch. Basically I still want the first box in the last line of an upper cell taking care that the upper cell remains in the array.I have changed the condition on removing the upper cell from the array. It seems OK now. Waiting for your suggestions.
Created attachment 50756 [details] patch III for this issue
lijian->fme: For this patch, I have tested some more nesting examples. For the moment, I found everything OK. Please review this patch, and give me your reply :-)
fme->lijian: Thank you for your patch. Looks good but please test if thoroughly. One more remark: I think the code adjusting nDiff at the end of the while loop can be moved to the 'else' case.
lijian->fme: Yes,the code adjusting nDiff should be moved to the 'else' case. And then ? I just need to test this code more and more, until we can confirm that all is OK ? Actually I have debugged this as many as possible today.
fme->lijian: Ok, in this case I think we can consider this 'fixed'. I'll integrate this in a OOo 3.0 cws. Thank you for your help.
Created attachment 50775 [details] patch 4
lijian->fme: I have put the code adjusting nDiff into the else case. Thank you for your help. :-)
Fixed in swtable.cxx, cws sw30bf01. Thank you for your help.
Created attachment 51373 [details] bugdoc
Only reproducible with the 'old' table model. Attached new bugdoc. Ready for QA.
Verified in CWS sw30bf01.
*** Issue 65927 has been marked as a duplicate of this issue. ***
*** Issue 55794 has been marked as a duplicate of this issue. ***
Works fine in DEV300m14.