diff options
| author | Miklos Vajna <vmiklos@collabora.co.uk> | 2015-11-02 15:18:09 +0100 |
|---|---|---|
| committer | Miklos Vajna <vmiklos@collabora.co.uk> | 2015-11-03 16:50:22 +0100 |
| commit | a900c9c8ee048c052f5e637a67c9f18a91d575f5 (patch) | |
| tree | c2bf18100ad620de66126d1c546825b46051251c | |
| parent | 99e1227e96f61c8ab82f7ff67ccb486b0386577e (diff) | |
sc lok: fix rounding errors with non-100% zoom
There were two problems here:
1) ScTabView::getRowColumnHeaders() did not expose twip values directly,
but used ScRow/ColBar::GetEntrySize(), which does a twip -> pixel
conversion, and then converted it back to twip. Avoid this unnecessary
roundtrip.
2) ScViewData::ToPixel() trunaces the resulting float to an integer, so
if the result is e.g. 67.7 pixels, then Calc handled that as 67, but
gtktiledviewer rounded that up to 68, resulting in non-matching headers
for the rendered tiles.
(cherry picked from commit 861b28b88909ec39fc83fccc0ab23d288128aa0e)
Conflicts:
libreofficekit/qa/gtktiledviewer/gtktiledviewer.cxx
Change-Id: Ie6ed1ea923a423d1526eeb235b7b87106fd2f20b
| -rw-r--r-- | sc/source/ui/view/tabview.cxx | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/sc/source/ui/view/tabview.cxx b/sc/source/ui/view/tabview.cxx index f2f6179eaaf9..2d886bca6003 100644 --- a/sc/source/ui/view/tabview.cxx +++ b/sc/source/ui/view/tabview.cxx @@ -2313,15 +2313,12 @@ OUString ScTabView::getRowColumnHeaders() SCROW nEndRow = 0; pDoc->GetTiledRenderingArea(aViewData.GetTabNo(), nEndCol, nEndRow); - double nPPTX = aViewData.GetPPTX(); - double nPPTY = aViewData.GetPPTY(); - boost::property_tree::ptree aRows; for (SCROW nRow = 0; nRow <= nEndRow; ++nRow) { boost::property_tree::ptree aRow; - sal_uInt16 nSize = pRowBar[SC_SPLIT_BOTTOM]->GetEntrySize(nRow); - aRow.put("size", OString::number(nSize / nPPTY).getStr()); + sal_uInt16 nSize = pDoc->GetOriginalHeight(nRow, aViewData.GetTabNo()); + aRow.put("size", OString::number(nSize).getStr()); OUString aText = pRowBar[SC_SPLIT_BOTTOM]->GetEntryText(nRow); aRow.put("text", aText.toUtf8().getStr()); aRows.push_back(std::make_pair("", aRow)); @@ -2331,8 +2328,8 @@ OUString ScTabView::getRowColumnHeaders() for (SCCOL nCol = 0; nCol <= nEndCol; ++nCol) { boost::property_tree::ptree aCol; - sal_uInt16 nSize = pColBar[SC_SPLIT_LEFT]->GetEntrySize(nCol); - aCol.put("size", OString::number(nSize / nPPTX).getStr()); + sal_uInt16 nSize = pDoc->GetColWidth(nCol, aViewData.GetTabNo()); + aCol.put("size", OString::number(nSize).getStr()); OUString aText = pColBar[SC_SPLIT_LEFT]->GetEntryText(nCol); aCol.put("text", aText.toUtf8().getStr()); aCols.push_back(std::make_pair("", aCol)); |