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 134333 - Devhelp doesn't select book/chapter currently browsed anymore
Devhelp doesn't select book/chapter currently browsed anymore
Status: RESOLVED FIXED
Product: devhelp
Classification: Applications
Component: General
git master
Other All
: Normal normal
: ---
Assigned To: Mikael Hallendal
Mikael Hallendal
: 135534 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2004-02-13 17:47 UTC by Maciej Katafiasz
Modified: 2005-09-28 18:40 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Screenshot showing problem (99.57 KB, image/png)
2004-02-14 02:07 UTC, Maciej Katafiasz
  Details
Improved syncing (4.50 KB, patch)
2005-09-27 23:19 UTC, Richard Hult
none Details | Review

Description Maciej Katafiasz 2004-02-13 17:47:16 UTC
Devhelp doesn't select book/chapter currently browsed anymore, like it used
to do. That is of great help when having two version of docs (like stable
and dev ones) installed, as search result usually are indentical in that
case making it next to impossible to distinguish which one is selected.
Probably there should be more sophisticated way of finding out current book
anyway, but this one was good enough to be sorely missed :).
Comment 1 Mikael Hallendal 2004-02-14 01:33:09 UTC
Not sure I'm following you here, where doesn't it select? In what
version did it behave the way you liked it?
Comment 2 Maciej Katafiasz 2004-02-14 02:02:29 UTC
It doesn't select (ie. unfold and highlight) correct book/chapter in
side pane tab containing list of all books after you get to that
chapter by searching for given keyword. Can't really remember in what
version it got changed, but I think it was on gtk1->gtk2 transition
(IIRC, the one shipped with Debian Woody, which happens to be 0.3, was
the one with correct behavior)
The problem can be seen clearly with attached screenshot, what was
searched for was "gtk_block_comment_tag_new". There is no way to tell
which book currently browsed chapter belongs to.
Comment 3 Maciej Katafiasz 2004-02-14 02:07:10 UTC
Created attachment 24401 [details]
Screenshot showing problem
Comment 4 Mikael Hallendal 2004-03-01 15:10:52 UTC
*** Bug 135534 has been marked as a duplicate of this bug. ***
Comment 5 Sven Herzberg 2004-10-12 19:44:17 UTC
This bug is still in 0.8.1, please update the version
Comment 6 Luca Cavalli 2005-05-12 14:06:37 UTC
The code for this feature is already written, but seems it needs some fixation.
See dh_book_tree_show_uri in
http://cvs.gnome.org/viewcvs/devhelp/src/dh-book-tree.c?view=markup
Comment 7 Luca Cavalli 2005-05-13 21:38:51 UTC
I have written a patch. It works. Now the book tree selection follows the actual
page displayed on the right pane, but if I click on links from one of the last
items of my book collection (about 50 first level entries in the tree) it takes
more than half a second to display the html page, since the whole tree is parsed
(gtk_tree_model_foreach) to match the clicked/searched link. I will try to speed
up the patch a bit before posting it here.
Any idea is highly appreciated :)
Comment 8 Mikael Hallendal 2005-05-13 22:43:48 UTC
This is why it was disabled, ie, a change internally. We have started working on a new version of Devhelp 
though that will have this functionality.
Comment 9 Richard Hult 2005-09-27 22:38:50 UTC
How many entries do you have in your tree? I can't really see how going through,
say, a few thousand rows would take any noticable time...
Comment 10 Richard Hult 2005-09-27 23:19:07 UTC
Created attachment 52754 [details] [review]
Improved syncing

Could you try this patch and see if it's slow? It seems to work well for me,
but I don't have that many nodes.
Comment 11 Richard Hult 2005-09-27 23:21:54 UTC
I'll close here, please reopen if it's not fast enough.
Comment 12 Luca Cavalli 2005-09-28 08:46:20 UTC
Testing the patch right now. Much much better. Now the tree (about 50 top level
entries) is updated almost immediately.
Thanks.
Comment 13 Richard Hult 2005-09-28 18:40:26 UTC
I'm about to commit a new version that should feel even better, since it updates
the tree before updating the HTML view.