[GitHub] [openoffice] cbmarcum commented on pull request #164: Update EditEngine code to use 32 bit paragraph storage
cbmarcum commented on PR #164: URL: https://github.com/apache/openoffice/pull/164#issuecomment-1374548388 > > Copy/Paste from Firefox to Calc trunk and this PR all 2 rows for both > > There is no way all 2 pasted successfully over HTML on trunk. Either the transfer format wasn't HTML, or you tested something other than trunk. Hi Damjan, You are correct. I found out what was wrong with my test. I had copied from a Firefox outside of the VM and pasted to AOO inside the VM. This opened a Text Import dialog and did import all 2 rows in both trunk and the PR. I can confirm that copying from Firefox and pasting into AOO within the same VM results in: trunk getting to row 13106 and inserting a lot (or all) of the rest into column D of the next row. The PR build takes all 2 rows. Both without using the Text import dialog. Sorry for the confusion. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org For additional commands, e-mail: dev-h...@openoffice.apache.org
[GitHub] [openoffice] cbmarcum commented on pull request #164: Update EditEngine code to use 32 bit paragraph storage
cbmarcum commented on PR #164: URL: https://github.com/apache/openoffice/pull/164#issuecomment-1374528032 My latest testing: File > Open set file type filter to HTML Document (OpenOffice Calc) (*.html;*.htm) trunk got to row 13106 with PR 164 got all 2 rows --- Copy/Paste from Firefox to Calc trunk and this PR all 2 rows for both --- Copy/Paste from Firefox to Writer trunk and this PR all 2 rows as tab separated values per row. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org For additional commands, e-mail: dev-h...@openoffice.apache.org
[GitHub] [openoffice] cbmarcum commented on pull request #164: Update EditEngine code to use 32 bit paragraph storage
cbmarcum commented on PR #164: URL: https://github.com/apache/openoffice/pull/164#issuecomment-1374199770 H Damjan, Thanks for your work on this. I made a slight change to the html test file from i117635 by removing the first row so that cell A1 was in the first row. See link [1]. the file is a simple html table with 2 rows and 5 columns. When I open it with my build of this PR on CentOS 7 it still only opens into a Writer/HTML document to the row 12801 before starting to just produce text beyond that. Same as with trunk. [1] https://home.apache.org/~cmarcum/test-files/2-row-html-table.html -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org - To unsubscribe, e-mail: dev-unsubscr...@openoffice.apache.org For additional commands, e-mail: dev-h...@openoffice.apache.org