D24761: Asign correct column width when importing XLS file

David Llewellyn-Jones noreply at phabricator.kde.org
Fri Oct 18 16:53:07 BST 2019


davidllewellynjones created this revision.
davidllewellynjones added a project: Calligra: 3.0.
Herald added a subscriber: Calligra-Devel-list.
davidllewellynjones requested review of this revision.

REVISION SUMMARY
  When importing an Excel 97-2003 .xls file (using the excel importer) the column widths don't match those from the file exported from Excel (or LibreOffice). Moreover, the widths of the columns depend on the dpi of the device in use, even though this doesn't affect the contents of the cells.
  
  For any screen that is higher than 96 dpi the columns will be too narrow and cell contents are likely to be truncated.
  
  This patch fixes the issue so that the correct column widths are assigned. As far as I can tell, this is the only situation where QWidget.physicalDpiX() is being explicitly used in the code.
  
  http://www.flypig.co.uk/dnload/dnload/other/calligra-column-widths.zip

TEST PLAN
  1. Download the following archive and extract the test-widths.xls and test-widths.xlsx files.
  
  http://www.flypig.co.uk/dnload/dnload/other/calligra-column-widths.zip
  
  2. Open the files. The column widths should be the same.
  
  The archive also contains screenshots of the patched and unpatched versions of Callibra using two different spi screens.

REPOSITORY
  R8 Calligra

REVISION DETAIL
  https://phabricator.kde.org/D24761

AFFECTED FILES
  filters/sheets/excel/sidewinder/sheet.cpp

To: davidllewellynjones
Cc: Calligra-Devel-list, dcaliste, cochise, vandenoever
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20191018/4d280e87/attachment.htm>


More information about the calligra-devel mailing list