D15111: [KoUnit] Let's show pixel units

ndavis added a comment. This does need DPI configuration for pixel units to be useful. Or maybe you could just set the DPI to 96 for pixel units since that's probably what most people want. REPOSITORY R8 Calligra REVISION DETAIL https://phabricator.kde.org/D15111 To: anthonyfieroni, bo

D15111: [KoUnit] Let's show pixel units

ndavis added inline comments. INLINE COMMENTS > anthonyfieroni wrote in KoUnit.h:59-60 > @ndavis, DPI is 96, it's not configurable for now. Was it already like that when I made the comment? When I export an SVG, it's proportioned like the DPI was 72 if I open it in Gwenview or Inkscape. 48x48

D15111: [KoUnit] Let's show pixel units

ndavis added a comment. In D15111#671492 , @boemann wrote: > And that is why this proposal should be rejected as is > > pixels is not a unit unless assigned a ppi on a case by case basis > > and thus it needs to be something requested and

D15111: [KoUnit] Let's show pixel units

ndavis added inline comments. INLINE COMMENTS > anthonyfieroni wrote in KoUnit.h:59-60 > Are you using HiDPI resolution? Scale factor, Calligra is not hidpi > compatible, i think I am using 96 font DPI (not forced) and 1x scaling at 1920x1080. REPOSITORY R8 Calligra REVISION DETAIL https:

D15111: [KoUnit] Let's show pixel units

ndavis added a comment. In D15111#671510 , @boemann wrote: > Nothing prevents Karbon from having a m_pixelUnit with ppi collected from a widget for user control > > and the use that unit like inkscape does. > > But what this patch does is