Re: fvwm 2.6.x title vs icon title bug

2017-03-01 Thread Dominik Vogt
On Thu, Mar 02, 2017 at 12:00:08AM +0700, ?? wrote:
> On 01.03.2017 23:38, Dominik Vogt wrote:
> >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=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?
> >
> 
> I removed temporaries related to this issue from my .fvwm2rc and
> tested a patch with 2.6.7. I can say the issue is gone fully. I will
> notify if there will be regressions related to this.

Good.  I'll push the patch onto the stable branch.

One thing I'm not quite sure about; before that patch that added
TitleFormat and IconTitleFormat in 2008, what did fvwm show as the
icon title?  Was it the icon name or the window title?  Now it's
the icon name, but was it the same before that patch?

Ciao

Dominik ^_^  ^_^

-- 

Dominik Vogt



Re: fvwm 2.6.x title vs icon title bug

2017-03-01 Thread ЭлектроРысь

On 01.03.2017 23:38, Dominik Vogt wrote:

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=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?



I removed temporaries related to this issue from my .fvwm2rc and tested 
a patch with 2.6.7. I can say the issue is gone fully. I will notify if 
there will be regressions related to this.


Thank you!


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 ^_^  ^_^



--
494 lives.