After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 144452 - should show page title in window title
should show page title in window title
Status: RESOLVED FIXED
Product: devhelp
Classification: Applications
Component: General
0.9
Other Linux
: Normal enhancement
: ---
Assigned To: Mikael Hallendal
Mikael Hallendal
Depends on:
Blocks:
 
 
Reported: 2004-06-16 09:13 UTC by Tommi Komulainen
Modified: 2005-09-26 21:08 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
document name in window title (767 bytes, patch)
2005-05-12 21:15 UTC, Luca Cavalli
none Details | Review
use the HTML title as window title (1.81 KB, patch)
2005-05-15 10:47 UTC, Tommi Komulainen
none Details | Review

Description Tommi Komulainen 2004-06-16 09:13:24 UTC
Whenever one scrolls down from the top of the page, the current page isn't
really indicated anywhere.  It would be nice if the window title would include
the page title always.
Comment 1 Luca Cavalli 2005-05-12 21:13:14 UTC
I like this feature too and also gtk+ documentation suggests to add document
name to window title.

"...A good title might include the application name and current document
filename, for example..."
<from gtk+2 docs>
Comment 2 Luca Cavalli 2005-05-12 21:15:19 UTC
Created attachment 46381 [details] [review]
document name in window title

This patch modify window title to include document name too ("%s - Devhelp",
where %s is the opened document name).
Comment 3 Mikael Hallendal 2005-05-13 06:42:16 UTC
Setting the PATCH keyword.
Comment 4 Tommi Komulainen 2005-05-15 10:23:23 UTC
Could use some improvements, IMHO.  Selecting links from Contents tab works
quite nicely, however the rest not so well.

Selecting functions from the Search tab gives you the clicked link as the window
title (gtk_pattern_tag_new() - Devhelp) so as soon as you scroll the page a bit
the title is no longer accurate.  The title should be taken from the page title
instead (which in this would've been GtkSourceTag)

Furthermore clicking links in the content area is not updating the window title
at all.
Comment 5 Tommi Komulainen 2005-05-15 10:47:51 UTC
Created attachment 46447 [details] [review]
use the HTML title as window title

Different approach.  Gecko signals when the page title changes, so use that to
construct the window title.  In this case it's important to signal every page
change, not only when it's not empty, let the receiver decide how to handle
NULL / empty title appropriately.

I still kept the "%s - Devhelp" format even though including the application
name in the window title is discouraged in the HIG.
Comment 6 Richard Hult 2005-09-26 21:08:22 UTC
Committed, thanks.