Re: [Adobe-source-devel] win32 group_window_proc

[Ralph T. <ra...@gm...>]

I just committed the fix for this to sandbox, as well as the fix for
another issue that Thomas found -- when there are many items in a
popup widget it will occupy the entire screen (and some items are
inaccessible).

Ralph

On 1/20/06, Ralph Thomas <ra...@gm...> wrote:
> Unfortunately my text editor (notepad!) changed the line endings from
> what is in CVS so I can't post a diff, but one "fix" is to change
> group_window_proc to this:
>
> // Forward any WM_COMMAND on to the widget that needs to receive it.
>
> LRESULT CALLBACK group_window_proc (HWND window, UINT message, WPARAM
> wParam, LPARAM lParam)
>
> {
>
>     if (message =3D=3D WM_COMMAND)
>     {
>         bool unused (false);
>         return event_dispatcher::event( message, wParam, lParam, unused )=
;
>     }
>
>
>    // do default button message processing
>
>    control_t*   control(reinterpret_cast<control_t*>(get_user_reference(w=
indow)));
>
>    return CallWindowProc (control->default_window_proc_m, window,
> message, wParam, lParam);
>
> }
>
> But then there's not much difference between group_window_proc and
> container_window_proc which doesn't seem to be being used anymore (?)
> so I think the best fix is to remove group_window_proc completely and
> just use container_window_proc in  place of it.
>
> The event_dispatcher (which container_window_proc) uses works by
> assigning each control a menu id when the control is created. All
> WM_COMMAND and WM_NOTIFYs contain the menu id as LOWORD(wParam), the
> event_dispatcher can look up the control_t* to call when it encounters
> a menu id that it assigned.
>
> After this change it looks like slider_suite still works, as does
> image_size (I put the buttons into a group), and the example that
> Thomas sent for unit_text now works too.
>
> Ralph
>
> On 1/20/06, Niki Spahiev <nik...@gm...> wrote:
> > ATL supports message reflection. IIRC it's source is available.
> >
> > http://msdn.microsoft.com/library/en-us/dnvc60/html/atlwindow.asp?frame=
=3Dtrue#atlwindow_topic11
> >
> > HTH
> > Niki Spahiev
> >
> >
> > -------------------------------------------------------
> > This SF.net email is sponsored by: Splunk Inc. Do you grep through log =
files
> > for problems?  Stop!  Download the new AJAX search engine that makes
> > searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
> > http://sel.as-us.falkag.net/sel?cmdlnk&kid=103432&bid#0486&dat=121642
> > _______________________________________________
> > Adobe-source-devel mailing list
> > Ado...@li...
> > https://lists.sourceforge.net/lists/listinfo/adobe-source-devel
> >
>