More KDevelop patches

[W. Tasin]

Hi,

Falk Brettschneider wrote:

> Hi,
>
>> 3) I don't know much about the MDI-part, but comparing it with other 
>> code-elements showed me some differences.
>> Isn't there a setFocus missing? Falk could you check this (when you 
>> will come back from cebit)? 
>
>
> What do you mean? (I don't see an MDI problem mentioned in the 
> original mail of Nir.)


Example of actual KDevelop:
....
void CKDevelop::slotActivateTView_VAR()
{
  if (dbgController != 0L) {  // when debugging, only
    
dockManager->findWidgetParentDock(var_viewer->parentWidget())->makeDockVisible();
    var_viewer->setFocus();
  }
}
.....

------- and in the patch:

>+/** No descriptions */
>+void CKDevelop::showVarViewer(){
>+  KDockWidget* pDockWdg = dockManager->findWidgetParentDock(var_viewer->parentWidget());
>+  pDockWdg->makeDockVisible();
> }
>
-------

So shouldn't it give the focus after the makeDockVisible()? Is it 
intended to have one function w/o and one with the setFocus()-call.

>
> Ciao,F at lk
>
Ciao

Walter

-- 
The KDevelop project: tasin at kdevelop.de [www.kdevelop.org]
--
oohhh sveglia.... il mondo e' ammalato, ma x colpa di chi.........
(Zucchero)
:-------W. Tasin, FB 04,FHM-------------------PGP-KeyID:0x7961A645----------:
<Key-Fingerprint: 1610 835F 0080 32F4 6140  6CF7 A7D0 44CD 7961A645>