<table><tr><td style="">davidllewellynjones created this revision.<br />davidllewellynjones added a project: Calligra: 3.0.<br />Herald added a subscriber: Calligra-Devel-list.<br />davidllewellynjones requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D24761">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>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.</p>

<p>For any screen that is higher than 96 dpi the columns will be too narrow and cell contents are likely to be truncated.</p>

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

<p><a href="http://www.flypig.co.uk/dnload/dnload/other/calligra-column-widths.zip" class="remarkup-link" target="_blank" rel="noreferrer">http://www.flypig.co.uk/dnload/dnload/other/calligra-column-widths.zip</a></p></div></div><br /><div><strong>TEST PLAN</strong><div><ol class="remarkup-list">
<li class="remarkup-list-item">Download the following archive and extract the test-widths.xls and test-widths.xlsx files.</li>
</ol>

<p><a href="http://www.flypig.co.uk/dnload/dnload/other/calligra-column-widths.zip" class="remarkup-link" target="_blank" rel="noreferrer">http://www.flypig.co.uk/dnload/dnload/other/calligra-column-widths.zip</a></p>

<ol class="remarkup-list" start="2">
<li class="remarkup-list-item">Open the files. The column widths should be the same.</li>
</ol>

<p>The archive also contains screenshots of the patched and unpatched versions of Callibra using two different spi screens.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R8 Calligra</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D24761">https://phabricator.kde.org/D24761</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>filters/sheets/excel/sidewinder/sheet.cpp</div></div></div><br /><div><strong>To: </strong>davidllewellynjones<br /><strong>Cc: </strong>Calligra-Devel-list, dcaliste, cochise, vandenoever<br /></div>