https://bugs.documentfoundation.org/show_bug.cgi?id=166525
Saburo <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords|bibisectRequest |bibisected, bisected OS|Windows (All) |All --- Comment #9 from Saburo <[email protected]> --- (In reply to nobu from comment #8) > For the most obvious explanation, I have explained it in formulas and > results, but there is a similar problem with text and numbers. > This problem must be more important. > And it may have been created with the same commit as the bug below. > > https://bugs.documentfoundation.org/show_bug.cgi?id=166520 > https://bugs.documentfoundation.org/show_bug.cgi?id=166559 right. bibisected with linux-64-25.8 author Armin Le Grand (Collabora) commit f678ec9a65c4d7ae2d4dda4599a06fc4b66a27d8 CEOO: CellEditOnOverlay Instead of using EditView's Paint directly to Window I changed this now to using the Overlay that is available in all applications. This allows to have less repaints of the Calc View since the Overlay has a copy of the area it is working overlayed. It also allows to get away from one of the last areas where XOR selection was used since EditView/EditEngine have no better way to do this. Selection is now visualized using the OS'es choosen SelectionColor, as in all other apps. I could also get rid of the flush() of the OverlayManager that was needed before and caused some problems (see tdf#165621 and associated tasks). A failing UnitTest showed that OverlayManagerBuffered needs to be aware when no CompleteRedraw was done yet and thus no background data exists and refresh makes no sense. This may happen when there is no redraw/UI e.g. UnitTests, but still an OverlayManager is used and together with a Reschedule (many possible reasons) that refresh might be triggered. Made access to EditView in OverlayObject no longer dependent on ActiveView, that is not always the one with the EditView. Now using a combination of ScTabView and ScSplitPos for accesses. Asserting now if no EditView -> is mandatory. Continuing going through CppunitTest_sc_tiledrendering UTs, one more identified. Have to re-activate EditViewInvalidate rect forwarding since the Invalidate is needed for sc tiled rendering tests, but not for sd tiled rendering tests. Added that to the impl of EditViewCallbacks using LibreOfficeKit::isActive(). Change-Id: I41f5448bfbabfaae4858c7617478771053c2cd77 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/184830 -- You are receiving this mail because: You are the assignee for the bug.
