1 files changed, 38 insertions, 17 deletions

diff --git a/libopie/big-screen/osplitter.cpp b/libopie/big-screen/osplitter.cpp
index d06568f..47f856d 100644
--- a/libopie/big-screen/osplitter.cpp
+++ b/libopie/big-screen/osplitter.cpp
@@ -77,4 +77,5 @@ OSplitter::OSplitter( Orientation orient, QWidget* parent, const char* name, WFl
  */
 OSplitter::~OSplitter() {
+    qWarning("Deleted Splitter");
     m_splitter.setAutoDelete( true );
     m_splitter.clear();
@@ -155,5 +156,5 @@ void OSplitter::addWidget( OSplitter* split ) {
      */
     if (m_tabWidget )
-        split->setTabWidget( m_tabWidget );
+        setTabWidget( m_parentTab );
     else{
         Opie::OSplitterContainer con;
@@ -307,5 +308,5 @@ QWidget* OSplitter::currentWidget() const{
 }
 
-#if 0
+
 /**
  * @reimplented for internal reasons
@@ -313,5 +314,6 @@ QWidget* OSplitter::currentWidget() const{
  */
 QSize OSplitter::sizeHint()const {
-    return QSize(10, 10);
+    if (m_parentTab )
+        return QFrame::sizeHint();
 
     if (m_hbox )
@@ -322,7 +324,12 @@ QSize OSplitter::sizeHint()const {
 
 QSize OSplitter::minimumSizeHint()const {
-    return QSize(10, 10 );
+    if (m_parentTab )
+        return QFrame::minimumSizeHint();
+    if (m_hbox)
+        return m_hbox->sizeHint();
+    else
+        return m_tabWidget->sizeHint();
 }
-#endif
+
 
 /**
@@ -336,5 +343,5 @@ void OSplitter::resizeEvent( QResizeEvent* res ) {
 //    qWarning("Old size was width = %d height = %d", res->oldSize().width(), res->oldSize().height() );
     bool mode = true;
-    qWarning("New size is  width = %d height = %d", res->size().width(), res->size().height() );
+    qWarning("New size is  width = %d height = %d  %s", res->size().width(), res->size().height(), name() );
     if ( res->size().width() > m_size_policy &&
          m_orient == Horizontal ) {
@@ -347,5 +354,6 @@ void OSplitter::resizeEvent( QResizeEvent* res ) {
         changeTab();
     }else if ( res->size().height() > m_size_policy &&
-               m_size_policy == Vertical ) {
+               m_orient == Vertical ) {
+        qWarning("Changng to vbox %s", name() );
         changeVBox();
         mode = false;
@@ -399,5 +407,5 @@ void OSplitter::changeTab() {
     }
 
-    qWarning(" New Tab Widget ");
+    qWarning(" New Tab Widget %s", name() );
     /*
      * and add all widgets this will reparent them
@@ -408,5 +416,10 @@ void OSplitter::changeTab() {
     if ( m_parentTab ) {
         tab = m_parentTab;
-        tab->removePage( this );
+        /* expensive but needed cause we're called from setTabWidget and resizeEvent*/
+        if (!m_container.isEmpty() ) {
+            ContainerList::Iterator it = m_container.begin();
+            for (; it != m_container.end(); ++it )
+                m_parentTab->removePage( (*it).widget );
+        }
     }else
         tab = m_tabWidget = new OTabWidget( this );
@@ -420,6 +433,8 @@ void OSplitter::changeTab() {
     }
 
-    for ( OSplitter* split = m_splitter.first(); split; split = m_splitter.next() )
+    for ( OSplitter* split = m_splitter.first(); split; split = m_splitter.next() ) {
+        split->reparent(this, 0, QPoint(0, 0) );
         split->setTabWidget( tab );
+    }
 
 
@@ -444,5 +459,5 @@ void OSplitter::changeHBox() {
     }
 
-    qWarning("new HBox");
+    qWarning("new HBox %s", name() );
     m_hbox = new QHBox( this );
     commonChangeBox();
@@ -455,5 +470,5 @@ void OSplitter::changeVBox() {
     }
 
-    qWarning("New VBOX");
+    qWarning("New VBOX %s", name() );
     m_hbox = new QVBox( this );
 
@@ -469,4 +484,5 @@ void OSplitter::changeVBox() {
  */
 void OSplitter::commonChangeBox() {
+    qWarning(" Name of Splitters is %s", name() );
 
     for (ContainerList::Iterator it = m_container.begin(); it != m_container.end(); ++it ) {
@@ -482,4 +498,6 @@ void OSplitter::commonChangeBox() {
         Opie::OSplitterContainer con;
         con.widget = split;
+//        con.widget = split->m_tabWidget ? static_cast<QWidget*>(split->m_tabWidget)
+//                     : static_cast<QWidget*>(split->m_hbox);
         addToBox( con );
     }
@@ -488,6 +506,7 @@ void OSplitter::commonChangeBox() {
 
     if (m_parentTab )
-        m_parentTab->addTab(this, iconName(), label() );
+        m_parentTab->addTab(m_hbox, iconName(), label() );
     else {
+        qWarning(" setting Box geometry for %s", name() );
         m_hbox->setGeometry( frameRect() );
         m_hbox->show();
@@ -504,5 +523,5 @@ void OSplitter::setTabWidget( OTabWidget* wid) {
     if ( m_parentTab ) {
         if (m_hbox )
-            m_parentTab->removePage( this );
+            m_parentTab->removePage( m_hbox );
         else if (!m_container.isEmpty() ){
             ContainerList::Iterator it = m_container.begin();
@@ -527,6 +546,8 @@ void OSplitter::setTabWidget( OTabWidget* wid) {
 
     /* our own crap is added and children from change* */
-    delete tab;
-    delete box;
+    if (m_parentTab ) {
+        delete tab;
+        delete box;
+    }
 }
 
@@ -553,5 +574,5 @@ bool OSplitter::layoutMode()const {
         return false;
     }else if ( size().height() > m_size_policy &&
-               m_size_policy == Vertical ) {
+               m_orient == Vertical ) {
         return false;
     }