Re: [1/2] explorer: Do not set the caption for "Shell_TrayWnd", some applications depend on it being empty.

[Alexandre Julliard]

Dmitry Timoshkov  writes:
> The bug #20129 lists one of such applications.
>
> It does FindWindow("Shell_TrayWnd", "") and expects that it returns a valid
> tray window handle. Currently FindWindow() is broken, but my next patch makes
> the above call succeed.

This will cause the stand-alone tray window to not have a title, that
doesn't look nice.

[1/2] explorer: Do not set the caption for "Shell_TrayWnd", some applications depend on it being empty.

[Dmitry Timoshkov]

Alexandre Julliard  wrote:
> > The bug #20129 lists one of such applications.
> >
> > It does FindWindow("Shell_TrayWnd", "") and expects that it returns a valid
> > tray window handle. Currently FindWindow() is broken, but my next patch makes
> > the above call succeed.
>
> This will cause the stand-alone tray window to not have a title, that
> doesn't look nice.

What would you suggest?

[1/2] explorer: Do not set the caption for "Shell_TrayWnd", some applications depend on it being empty.

[Alexandre Julliard]

Dmitry Timoshkov  writes:
> > This will cause the stand-alone tray window to not have a title, that
> > doesn't look nice.
>
> What would you suggest?

Use a separate window for the stand-alone tray.

[1/2] explorer: Do not set the caption for "Shell_TrayWnd", some applications depend on it being empty.

[Dmitry Timoshkov]

Alexandre Julliard  wrote:
> > > This will cause the stand-alone tray window to not have a title, that
> > > doesn't look nice.
> >
> > What would you suggest?
>
> Use a separate window for the stand-alone tray.

Shouldn't that be a separate thing not related to the problem fixed by
my patch?

[1/2] explorer: Do not set the caption for "Shell_TrayWnd", some applications depend on it being empty.

[Alexandre Julliard]

Dmitry Timoshkov  writes:
> Shouldn't that be a separate thing not related to the problem fixed by
> my patch?

Sure it should be a separate change.