Community
Participate
Working Groups
I20100513-1500 * In the LCD, plug-ins tab, feature launch mode * Deselect all, check "only show selected" * Press "select features..." * drag the feature selection dialog around and notice that the tree no longer paints behind the dialog. Looks like redraw is being turned off before the dialog is opened, but it should only be toggled off/on when the actual updates are performed (after the dialog selection is complete - and not required if cancel is pressed).
Created attachment 168558 [details] screen shot Notice the trail as one drags the dialog around over the tree.
Created attachment 168879 [details] Patch
+1
I will need one more +1. Curtis, plz review
Created attachment 169007 [details] Simpler Patch Solves the reported problem by moving the setredraw calls.
Ankur, was your patch trying to accomplish more than fixing the redraw problem? There are a lot of changes in your patch that I didn't understand. I feel much better putting in a small change to fix the problem over a larger patch that could affect the behaviour of other buttons in the tab.
The simpler patch doesn't solves all the problems. 1. It works only for Select Feature dialog. The validate and Add plug-ins buttons too open a dialog. They need to be handled too. 2. Also, when the show selected is checked, and then any of these dialogs is opened, we can see the tree getting unfiltered and then refiltered in background with scrollbar dancing. The previous patch addresses these issues too. All these are redraw related problems only.
Created attachment 169031 [details] Updated Patch This patch fixes the problems mentioned in comment #7. We don't need to disable drawing for anything except adding new content to the tree. I see that your original patch did this using a boolean, but it is better to have the redraw cover the smallest possible area. If there is a mistake in the code or an exception occurs while redraw is off, the UI will look very broken. This patch could definitely use some more testing. I think that it is a safer solution than your original patch Ankur.
+1 This one is good.
+1. Applied/Fixed.
Verified.
