Setting keyword

Adding to Meta Issue.

With the idea of paveljanik, Added function

void AquaSalGraphics::RefreshRect(float lX, float lY, float lWidth, float lHeight)
{
	fprintf(stderr,"-->%s refresh %f - %f - %f - %f\n",__func__,lX, lY,lWidth,lHeight);
	// Refresh windows content
	Rect aRect;
    aRect.left = static_cast<float>(lX);
    aRect.top = static_cast<float>(lY);
    aRect.right = static_cast<float>(lWidth);
    aRect.bottom = static_cast<float>(lHeight);
	
	InvalWindowRect(mrWindow, &aRect);
} 

to salgdiutils.ccx, and get good refresh, adding it step by step to drawing
function that need it

With the refreshrect : i ve got all icons appearing at startup of writer, text 
being typed appears on screen (need still soem fixin) : in svdem8svtool the 
cursor run fine while typing, in writer the cursor is logically moved but 
still blinking at the beginning of the line...

ericb->plipli

did you progress since ?

plipli->ericb

I've discussed with ismael to inform him about what HIframework will bring to
enhance considerably :
the refresh management, "speed up" the implementation of native controls (GsoC
applicatio nof ismael).

All the helpers we ll register automatically will bring us native refresh
without many lines of code :-)
so cheers :-)

Created attachment 45031 [details]
fix for the initial show problem of writer

Attached patch fixes Writer's initial show problem for me.

The change changed refresh of FirstStart wizard window.

Before the change, the window got refreshed when the mouse moved over some button or text entity, not 
it is not refreshed at all, only the native buttons are shown when the mouse walks over them.

True. I wonder why this seems to be the only dialog that shows this ? I need to
investigate further.

I found another one, with base : same for Base wizard when you try to create a
new .odb document.
Only buttons are refreshed when moving the mouse , but the window is not.

Searching other ...

I think i found the new problem. It goes like this:

at first GetClientSize did not return (0,0) initially. Which it should do since
this is our only indicator for the visible/non visible state (historical
reasons, on windows that was the case automatically). The change form (0,0) to
(whatever,whatever) after Show(TRUE) triggers activation handling in writer.
This is why writer initially didn't paint, but toolbars and other widgets did
(they go a paint after all :-) ).

Changing that however triggered new dialog behaviour: the dialogs consider
themselves resized now and immediately paint (without waiting for a paint
event). This works on Windows and X11, but does not seem to with Carbon. It
would not matter, if there came a system paint event afterwards (which one would
expect after a Show), however occasionally, this does not come anymore (which I
found out with the stderr output). Why, I cannot say, I can only guess that the
immediate painting is the culprit and interferes with the system's event
handling (I could imagine the calling of HIViewSetNeedsDisplayInRect interferes
with the paint event hadling).

However one can force the issue by calling InvalidateWindowRect after
ShowWindow; this seems to trigger the expected system paint event regardless of
the immediate painting. Checked in according change.

Does anyone have a better solution ?

plipli -> pl

You are true about events of painting.
When the window is resized (ie maximized) events are fired and well handled .

SetHINeedsDisplay(HIViewid) only tells Carbon that this HIview has to be
refreshed next time the applicationeventloop/Quartz will do a refresh. It is
used when we call refreshrect (after something drawn ahs to be visible immediately).

I m not sure but I don t think the HI* code that disable paint/vcl event
handling was committed so it should handle windowdraw content event. but
salpaint event was disabled some times ago (because of slowing down issue).

Hope this will help the vcl guru you are :-)

*** Issue 77555 has been marked as a duplicate of this issue. ***

*** Issue 77558 has been marked as a duplicate of this issue. ***

issue just fixed by hdu 


Please verify 

Verified in aquavcl01. No need to resize initially.

Verified in aquavcl01. No need to resize initially.

Change target milestone to 2.3.

Verified Ok in SRC680_m220

Closing