Bug 130319 - Hiding the javaScript Console also hides plot windows
Summary: Hiding the javaScript Console also hides plot windows
Status: RESOLVED FIXED
Alias: None
Product: kst
Classification: Applications
Component: general (show other bugs)
Version: 1.x
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: kst
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-07-05 21:14 UTC by Andrew Walker
Modified: 2006-07-06 01:21 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Walker 2006-07-05 21:14:29 UTC
Version:           1.2.1 (using KDE KDE 3.5.0)
Installed from:    Compiled From Sources
OS:                Linux

PROBLEM:
When the user is using the JavaScript Console and selects Tools... Hide JavaScript Console all plots windows are also hidden.

STEPS TO REPRODUCE:
Start Kst
If necessary select Settings...Extensions... and enable the JavaScript console
Select Tools... Show JavaScript Console
Create a plot in the default window
Select Tools... Hide JavaScript Console

RESULTS:
The JavaScript Console and the plot window are hidden

EXPECTED RESULTS:
Only the JavaScript Console is hidden
Comment 1 George Staikos 2006-07-05 23:33:47 UTC
Known, documented bug.  It's in KMDI.
Comment 2 Andrew Walker 2006-07-05 23:46:03 UTC
Wouldn't removing the call to

_splitter->hide();

in KstJS::hideConsole() in js.cpp

fix this?
Comment 3 George Staikos 2006-07-06 00:53:22 UTC
Odd, yes, that is the problem.  I'm sure I had found a related bug in KMDI in 
the past though.  The container was disappearing completely when the splitter 
was deleted.  Anyhow, please commit.
Comment 4 Andrew Walker 2006-07-06 01:21:36 UTC
SVN commit 558759 by arwalker:

BUG:130319 Do not hide the entire contents of the Kst frame when hiding the JavaScript Console

 M  +0 -1      js.cpp  


--- trunk/extragear/graphics/kst/src/extensions/js/js.cpp #558758:558759
@@ -287,7 +287,6 @@
 #ifdef KST_HAVE_READLINE
   _showAction->setText(i18n("Show &JavaScript Console"));
   if (_konsolePart) {
-    _splitter->hide();
     _konsolePart->widget()->hide();
   }
 #endif