Re: fvwm 2.6.x title vs icon title bug

2017-03-01 Thread Dominik Vogt
On Wed, Mar 01, 2017 at 04:20:29PM +0100, Dominik Vogt wrote:
> On Sat, Feb 18, 2017 at 11:35:48AM +0700, ?? wrote:
> > With both fvwm 2.6.5 and latest 2.6.7 I experience a bug when icon
> > of the window of some apps keeps a previously set title. For
> > example:
> > - Normal window has title AND icon title "title", which is correct
> > according to FvwmIdent,
> > - After Iconify, icon title appears as some default name of the
> > application, or "title".
> > - Deiconifying it again, and making app change title to "title
> > (new)". Both title and icon title are correct and shown as "title
> > (new)" according to FvwmIdent.
> > - Iconifying it again, and icon is named "title" (if there was some
> > default app name, it changes to this previously set name).
> > - Deiconifying it again, and making app change title to "new title".
> > Again, both title and icon title is correct per FvwmIdent.
> > - Iconifying it again, and I see icon name set to "title (new)".
> > 
> > This issue happens only with some apps. Namely, I think all FOX
> > toolkit apps are affected (for me they are Xfe, Xfw, Adie).
> > 
> > On forums http://fvwmforums.org/phpBB3/viewtopic.php?f=6&t=3204 I
> > was directed to mailing list thread
> > http://www.mail-archive.com/fvwm-workers@fvwm.org/msg03213.html,
> > which gave me an idea that setting IconTitleFormat to %i and
> > TitleFormat to %n separately may work. Indeed this worked and solved
> > my issue I described.
> 
> I'm working on the problem.

Can you please try the fix on the master branch in Git?

> > So, if someone is affected too by strangely behaving titles of
> > _some_ apps, here is a patch.
> > 
> > --- fvwm-2.6.7.o/fvwm/add_window.c
> > +++ fvwm-2.6.7/fvwm/add_window.c
> > @@ -586,7 +586,7 @@
> > {
> > format = (style->flags.has_icon_title_format_string) ?
> > SGET_ICON_TITLE_FORMAT_STRING(*style) :
> > -   DEFAULT_TITLE_FORMAT;
> > +   DEFAULT_ICON_TITLE_FORMAT;
> 
> There's more broken than just the default format ffor icon titles.
> 
> > } else {
> > format = (style->flags.has_title_format_string) ?
> > SGET_TITLE_FORMAT_STRING(*style) :
> > --- fvwm-2.6.7.o/fvwm/style.c
> > +++ fvwm-2.6.7/fvwm/style.c
> > @@ -2978,7 +2978,7 @@
> > 
> > if (fmt_string == NULL)
> > {
> > -   fmt_string = DEFAULT_TITLE_FORMAT;
> > +   fmt_string = DEFAULT_ICON_TITLE_FORMAT;
> > }
> > 
> > if (!__validate_titleformat_string(fmt_string))
> > --- fvwm-2.6.7.o/fvwm/style.h
> > +++ fvwm-2.6.7/fvwm/style.h
> > @@ -5,6 +5,7 @@
> > 
> >  /* The default title in case the user doesn't supply one. */
> >  #define DEFAULT_TITLE_FORMAT "%n"
> > +#define DEFAULT_ICON_TITLE_FORMAT "%i"
> > 
> >  /* access to the special flags of a style */
> >  /* call these with a pointer to a style_flags struct */
> > 
> > -- 
> > HTH, lynx.
> > 494 lives.
> > 
> > 
> > 
> 
> 
> Ciao
> 
> Dominik ^_^  ^_^
> 
> -- 
> 
> Dominik Vogt
> 
> 


Ciao

Dominik ^_^  ^_^

-- 

Dominik Vogt



Re: fvwm 2.6.x title vs icon title bug

2017-03-01 Thread Dominik Vogt
On Sat, Feb 18, 2017 at 11:35:48AM +0700, ?? wrote:
> Hello.
> 
> With both fvwm 2.6.5 and latest 2.6.7 I experience a bug when icon
> of the window of some apps keeps a previously set title. For
> example:
> - Normal window has title AND icon title "title", which is correct
> according to FvwmIdent,
> - After Iconify, icon title appears as some default name of the
> application, or "title".
> - Deiconifying it again, and making app change title to "title
> (new)". Both title and icon title are correct and shown as "title
> (new)" according to FvwmIdent.
> - Iconifying it again, and icon is named "title" (if there was some
> default app name, it changes to this previously set name).
> - Deiconifying it again, and making app change title to "new title".
> Again, both title and icon title is correct per FvwmIdent.
> - Iconifying it again, and I see icon name set to "title (new)".
> 
> This issue happens only with some apps. Namely, I think all FOX
> toolkit apps are affected (for me they are Xfe, Xfw, Adie).
> 
> On forums http://fvwmforums.org/phpBB3/viewtopic.php?f=6&t=3204 I
> was directed to mailing list thread
> http://www.mail-archive.com/fvwm-workers@fvwm.org/msg03213.html,
> which gave me an idea that setting IconTitleFormat to %i and
> TitleFormat to %n separately may work. Indeed this worked and solved
> my issue I described.

I'm working on the problem.

> So, if someone is affected too by strangely behaving titles of
> _some_ apps, here is a patch.
> 
> --- fvwm-2.6.7.o/fvwm/add_window.c
> +++ fvwm-2.6.7/fvwm/add_window.c
> @@ -586,7 +586,7 @@
>   {
>   format = (style->flags.has_icon_title_format_string) ?
>   SGET_ICON_TITLE_FORMAT_STRING(*style) :
> - DEFAULT_TITLE_FORMAT;
> + DEFAULT_ICON_TITLE_FORMAT;

There's more broken than just the default format ffor icon titles.

>   } else {
>   format = (style->flags.has_title_format_string) ?
>   SGET_TITLE_FORMAT_STRING(*style) :
> --- fvwm-2.6.7.o/fvwm/style.c
> +++ fvwm-2.6.7/fvwm/style.c
> @@ -2978,7 +2978,7 @@
> 
>   if (fmt_string == NULL)
>   {
> - fmt_string = DEFAULT_TITLE_FORMAT;
> + fmt_string = DEFAULT_ICON_TITLE_FORMAT;
>   }
> 
>   if (!__validate_titleformat_string(fmt_string))
> --- fvwm-2.6.7.o/fvwm/style.h
> +++ fvwm-2.6.7/fvwm/style.h
> @@ -5,6 +5,7 @@
> 
>  /* The default title in case the user doesn't supply one. */
>  #define DEFAULT_TITLE_FORMAT "%n"
> +#define DEFAULT_ICON_TITLE_FORMAT "%i"
> 
>  /* access to the special flags of a style */
>  /* call these with a pointer to a style_flags struct */
> 
> -- 
> HTH, lynx.
> 494 lives.
> 
> 
> 


Ciao

Dominik ^_^  ^_^

-- 

Dominik Vogt