TODO list (again)

August Hörandl august.hoerandl at gmx.at
Sat Mar 2 10:44:02 UTC 2002


August Hörandl wrote:
... 
> 13 Bug     tab page mode, use crtl-t to remove treeview
>            then try one of these
>            * crtl-t   shows treeview (ok)
>            * ansicht/arbeistansichten/baumansicht    treeview is ok
>            * ansicht/arbeitsansichten/klassen   treeview in a new window

as nobody else likes this bug i am replying to myself ;-)

i took a look at the code and came up with this little patch which fixes
the problem for me - and doesn't seem to harm anything else

diff -u -3 -p -c -r1.158.2.114.2.140.2.31 ckdevelop.cpp
*** kdevelop/ckdevelop.cpp      2002/03/01 15:27:12    
1.158.2.114.2.140.2.31
--- kdevelop/ckdevelop.cpp      2002/03/02 09:23:10
*************** void CKDevelop::fillToggleTreeViewsMenu(
*** 4178,4183 ****
--- 4178,4184 ----

  void CKDevelop::slotViewTClassesView()
  {
+       slotViewTTreeView();
    KDockWidget* pDock =
(KDockWidget*)class_tree->parentWidget()->parentWidget();
    if
(toggletreeviews_popup->isItemChecked(toggletreeviews_popup->idAt(2)))
      pDock->undock();

actually this should be added also to the slots of the other views (5
times) and 
slotViewTOutputView() should be added at the slots of all output views
(7 times)

questions:
- is this a bad idea ? maybe it would be better to disable the entry for
  class menu if the whole treeview window isn't visible

- any reasons not commit the whole changes ?

Gustl

-- 
August Hörandl                                   august.hoerandl at gmx.at
Protection Fault: GAME OVER. Do you wish to play another game of WIN95?




More information about the KDevelop-devel mailing list