[GitHub] [openoffice] DamjanJovanovic commented on pull request #164: Update EditEngine code to use 32 bit paragraph storage

2023-01-07 Thread GitBox 


DamjanJovanovic commented on PR #164:
URL: https://github.com/apache/openoffice/pull/164#issuecomment-137456

   > 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.


-- 
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] DamjanJovanovic commented on pull request #164: Update EditEngine code to use 32 bit paragraph storage

2023-01-06 Thread GitBox 


DamjanJovanovic commented on PR #164:
URL: https://github.com/apache/openoffice/pull/164#issuecomment-1374407466

   I believe the Writer table limit of 64K cells is unrelated to this issue. 
   
   For example in main/sw/source/filter/html/htmltab.cxx:
   
   ```
   class HTMLTable
   {
   ...
   sal_uInt16 nRows;   // Anzahl Zeilen
   sal_uInt16 nCols;   // Anzahl Spalten
   ```
   
   That's not EditEngine related, that's a separate issue specific to Writer.
   
   Please test only copying from a web browser and pasting into Calc, and 
opening through the "File type" of "HTML document (**OpenOffice Calc**) (*html, 
*.htm)".


-- 
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] DamjanJovanovic commented on pull request #164: Update EditEngine code to use 32 bit paragraph storage

2023-01-06 Thread GitBox 


DamjanJovanovic commented on PR #164:
URL: https://github.com/apache/openoffice/pull/164#issuecomment-1374378198

   > 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
   
   Nice catch. Yes, apparently there is still a problem somewhere in Writer. I 
only tested copying from a web browser and pasting to Calc.


-- 
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