https://bugs.documentfoundation.org/show_bug.cgi?id=156781
--- Comment #8 from Hossein <[email protected]> --- (In reply to Mike Kaganski from comment #7) > (In reply to Hossein from comment #6) > > But in my opinion, as the code is wrong > > on both screen and print, the first step would be to fix the problem here, > > make the code not dependent on DPI and zoom level, and then go for splitting > > the code if necessary. > > But is it decided that it should *not* be DPI-dependent and zoom-dependent > on screen? OK, then let me explain more: 0. By "DPI and zoom independent" I mean that the the box size should not change relative to the size of the text in the document and the comments. 1. The comment text IS actually DPI and zoom independent. 2. The yellow box is not DPI and zoom independent, and it seems to be a miscalculation, and not an intentional adjustment. 1 & 2 => To fix the issue, one need to make the box also DPI and zoom independent. In the next paragraph I discuss why. > Isn't current behavior *on screen* reasonable - except for a > problem of wrong height not fitting all the text? This also needs explanation. One need to fix the yellow box width and height so that it fits all the comment text. But, there is an important point here: One may try to fix the problem by making the box only fit the comment text, without making it DPI and zoom independent. But, as the text is DPI and zoom independent, a fix would require the width/height calculation for the yellow box to be also DPI and zoom independent. Agree? One may try to do a better adjustment for both text and the box, but IMO it should be (possibly) considered to be a further enhancement. I would prefer not to go with this path, because IMO it is meaningful to keep the size of the comment text and box size constant relative to the size of the text in the document. I can compare it to the older decision of using LO UI elements for content controls, and the later decision to draw the elements inside the document, and let the users interact with those elements inside the document. -- You are receiving this mail because: You are the assignee for the bug.
