dcaliste added a comment.
Well, after reading the difference between the logical and physical DPI values: https://stackoverflow.com/questions/16561879/what-is-the-difference-between-logicaldpix-and-physicaldpix-in-qt (see the first answer), using the logical value is consistent with the hard coded "arial 10" font metric some line above. Maybe adding a comment near the modified line explaining that logicalDPI should be used because of the "Arial 10" font metric being nice also on high DPI screen would be nice. If not too much burden… Good catch @davidllewellynjones ! REPOSITORY R8 Calligra REVISION DETAIL https://phabricator.kde.org/D24761 To: davidllewellynjones, #calligra:_3.0, pvuorela Cc: dcaliste, davidllewellynjones, Calligra-Devel-list, cochise, vandenoever
