|
Apache OpenOffice (AOO) Bugzilla – Full Text Issue Listing |
| Summary: | [From Symphony]White block( on Mac and Black block on Windows) appears if click somewhere else in presentation | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | QA | Reporter: | Yan Ji <yanji.yj> | ||||||
| Component: | tcm | Assignee: | Armin Le Grand <Armin.Le.Grand> | ||||||
| Status: | CLOSED FIXED | QA Contact: | |||||||
| Severity: | Major | ||||||||
| Priority: | P3 | CC: | Armin.Le.Grand, issues, orw, zhangjf | ||||||
| Version: | current | ||||||||
| Target Milestone: | 4.0.0 | ||||||||
| Hardware: | All | ||||||||
| OS: | All | ||||||||
| Issue Type: | DEFECT | Latest Confirmation in: | --- | ||||||
| Developer Difficulty: | --- | ||||||||
| Attachments: |
|
||||||||
ALG: Seems to always be around the mouse position, so may have to do with pointer refresh I will try to migrate fix code to 3.4. ALG -> zhangjf: Thanks. Do You have some information about the technical background and what is happening? Below is the analysis of the problem, Root Cause: The overlaybuffermanager in svx remember the invalidate range and expand it with antialiane pixel, but it's use old range to save background. That will cause that some background was not saved, and refresh to window with initialized GDI color( Windows is balck and Mac is white). Resolution: Remember the offset when the invalidate range would be expand. Use the offset when save background. I still need a few time to prepare the code and verify it in new code base when I have time. Created attachment 77253 [details]
fix code patch
The code doesn't match what in 3.1 exactly, I made some changes and do more test. Below are some more additional comments to the fix code, When switch from not use buffer overlay manager(text edit mode) to use buffer overlay manager(normal mode), save background with the anti-alias offset, it will draw correct from text edit mode to normal mode. And other drawing process will not use the offset. SdrPaintView::EndCompleteRedraw will check whether text edit mode. SdrPaintWindow::DrawOverlay->impCreateOverlayManager will set or reset the flag to overlayBufferManager. Please help to have a review before I commit the code to 3.4, thanks. ALG: The reason is interesting, I removed the need to switch OverlayManagers during TextEdit in branch aw080, but it's not ready yet. Switching OverlayManagers was a fix itself, necessary because EditView (with EditEngine) can only work with a Window, not a VirualDevice as is used for overlay. I made EditView work on VirtualDevices and comlteletely removed the switching code, so this should also remove the problem. If you are interested in this solution I could try to extract the needed parts from aw080 (or you may take a look, it's at https://svn.apache.org/repos/asf/incubator/ooo/branches/alg/aw080). Created attachment 77254 [details]
Removes the necessity of switching OverlayManagers
ALG: Added patch. It removes the necessity of switching OverlayManagers at all (and removes some unnecessary code doing this) and adds painting OutlinerView to OutputDevice (not only Window). This removes the reason for the issue and thus fixes it. It's also the right step since swithcing the OverlayManagers at all was a compromize bugfix itself and it will be good to get rid of it again, too.
ALG: Built mac and win version with suggested patch (can be tested at http://people.apache.org/~alg/trunk/unxmacxi/OOo_3.4.0_MacOS_x86_install_en-US.dmg resp. http://people.apache.org/~alg/trunk/wntmsci12/setup.zip). Works well, just needs checks if the WhiteBlock blinking is gone. I have not seen any WitheBlick vlinking anymore, so i suggest to apply that patch. @ALG, I am a little confused. It seems you have a better solution. Do you still suggest I commit my patch? If you mean to apply your patch, I will reassign the defect to you then. @zhangjf: As Armin is on vacation he asked me to have a look at this issue. Armin told me that his patch should be integrated, if you have reviewed it, it is working and it is ok for you. Thus, he told me that I should wait for your feedback. If it is positive, I would integrate Armin's patch. @Oliver. Yes, Armin's fix is much better than mine, please commit to 3.4. I reassign this bug to you. Thanks. As we are currently working on the last steps for our AOO 3.4 release and we are somehow in a show-stopper-mode(*) I would like to postpone the commit until AOO 3.4 has been finished respectively a separate svn branch has been opened for AOO 3.4 (*) show-stopper-mode means for me that we are working towards a stable application only fixing show stoppers on which the community had agreed on. ALG: Back to me, I'll take care after 3.4. ALG: New trunk, added patch, test build, works as expected. Checked in, done. This also removes the old workaround to be able to switch between buffered and nonbuffered overlays in the edit view (thus avoiding the flicker). Next step will be to use the standard text selection color and to always paint EditEngine using primitives. Problem is fixed |
Steps: 1. New Impress doc. 2. Click into text box, and input some text. 3. Click outside of the text box. Defect: on Mac, there is a whilte block appears. on Winodws, there is a black block appears.