better layout

11 files changed, 108 insertions, 64 deletions

diff --git a/noncore/apps/confedit/editwidget.cpp b/noncore/apps/confedit/editwidget.cpp
index 64f968a..f7dc408 100644
--- a/noncore/apps/confedit/editwidget.cpp
+++ b/noncore/apps/confedit/editwidget.cpp
@@ -17,16 +17,17 @@ EditWidget::EditWidget( QWidget* parent,  const char* name, WFlags fl )
 {
 
     EditWidgetLayout = new QGridLayout( this );
-    EditWidgetLayout->setSpacing( 3 );
-    EditWidgetLayout->setMargin( 3 );
+    EditWidgetLayout->setSpacing( 0 );
+    EditWidgetLayout->setMargin( 0 );
+//    setSizePolicy( QSizePolicy( QSizePolicy::Minimum, QSizePolicy::Maximum));//, sizePolicy().hasHeightForWidth() ) );
 
-    QLabel *TextLabelFileName = new QLabel( this, "TextLabelFileName" );
+    TextLabelFileName = new QLabel( this, "TextLabelFileName" );
     TextLabelFileName->setText( tr( "File Name:" ) );
     EditWidgetLayout->addWidget( TextLabelFileName, 0, 0 );
     TextFileName = new QLabel( this, "TextFileName" );
     EditWidgetLayout->addWidget( TextFileName, 0, 1 );
 
-    QLabel *TextLabelGroup = new QLabel( this, "TextLabelGroup" );
+    TextLabelGroup = new QLabel( this, "TextLabelGroup" );
     TextLabelGroup->setText( tr( "Group:" ) );
     EditWidgetLayout->addWidget( TextLabelGroup, 1, 0 );
     LineEditGroup = new QLineEdit( this, "LineEditGroup" );
@@ -44,6 +45,7 @@ EditWidget::EditWidget( QWidget* parent,  const char* name, WFlags fl )
     LineEditValue = new QLineEdit( this, "LineEditValue" );
     EditWidgetLayout->addWidget( LineEditValue, 3, 1 );
 
+  //  layoutType(Nothing);
 }
 
 
@@ -52,22 +54,44 @@ EditWidget::~EditWidget()
 }
 
 
-void EditWidget::isKey(bool h)
+void EditWidget::layoutType(int t)
 {
-	if (h)
+	switch (t)
  	{
-    LineEditGroup->setEnabled(false);
-    TextLabelKey->show();
-    LineEditKey->show();
-    TextLabelValue->show();
-    LineEditValue->show();
-  }else{
-    LineEditGroup->setEnabled(true);
-    TextLabelKey->hide();
-    LineEditKey->hide();
-    TextLabelValue->hide();
-    LineEditValue->hide();
+	 	case Entry:
+  	  LineEditGroup->setEnabled(false);
+    	TextLabelKey->show();
+	    LineEditKey->show();
+  	  TextLabelValue->show();
+    	LineEditValue->show();
+     	TextLabelGroup->show();
+    	LineEditGroup->show();
+     	TextLabelFileName->show();
+      TextFileName->show();
+	    break;
+  	case Group:
+    	LineEditGroup->setEnabled(true);
+	    TextLabelKey->hide();
+  	  LineEditKey->hide();
+    	TextLabelValue->hide();
+	    LineEditValue->hide();
+     	TextLabelGroup->show();
+    	LineEditGroup->show();
+     	TextLabelFileName->show();
+      TextFileName->show();
+  	  break;
+   	default:
+	    TextLabelKey->hide();
+  	  LineEditKey->hide();
+    	TextLabelValue->hide();
+	    LineEditValue->hide();
+     	TextLabelGroup->hide();
+    	LineEditGroup->hide();
+     	TextLabelFileName->hide();
+      TextFileName->hide();
   }
-  update();
+//  update();
+// updateGeometry();
+//  adjustSize();
 }
 
diff --git a/noncore/apps/confedit/editwidget.h b/noncore/apps/confedit/editwidget.h
index 673166f..8261c85 100644
--- a/noncore/apps/confedit/editwidget.h
+++ b/noncore/apps/confedit/editwidget.h
@@ -18,16 +18,20 @@ class EditWidget : public QWidget
     Q_OBJECT
 
 public:
+		enum{ Entry, Group, File, Nothing};
+
     EditWidget( QWidget* parent = 0, const char* name = 0, WFlags fl = 0 );
     ~EditWidget();
 
 		QLabel *TextLabelKey;
     QLabel *TextLabelValue;
+    QLabel *TextLabelGroup;
+    QLabel *TextLabelFileName;
     QLabel *TextFileName;
     QLineEdit *LineEditGroup;
     QLineEdit *LineEditKey;
     QLineEdit *LineEditValue;
-    void isKey(bool h);
+    void layoutType(int t);
 protected:
     QGridLayout* EditWidgetLayout;
 
diff --git a/noncore/apps/confedit/listviewconfdir.cpp b/noncore/apps/confedit/listviewconfdir.cpp
index f466f06..0c2feeb 100644
--- a/noncore/apps/confedit/listviewconfdir.cpp
+++ b/noncore/apps/confedit/listviewconfdir.cpp
@@ -24,6 +24,8 @@ ListViewConfDir::ListViewConfDir(QString settingsPath, QWidget *parent, const ch
 	if (!confDir.isReadable())
  		QMessageBox::critical(this,tr("Could not open"),tr("The directory ")+settingsPath+tr(" could not be opened."),1,0);
 	readConfFiles();
+
+ 	connect( this, SIGNAL(expanded(QListViewItem*)), SLOT(expand(QListViewItem*)));
 }
 
 
@@ -47,7 +49,13 @@ void ListViewConfDir::readConfFiles()
  	{
     qDebug( "opening: >%s<", fi->fileName().data() );
   	fileEntry = new ListViewItemConfFile( fi, this );
-   	
+    QListViewItem *dummy = new QListViewItem(fileEntry, "dummy");
   	++it;
   }
-}
\ No newline at end of file
+}
+
+void ListViewConfDir::expand(QListViewItem *item)
+{
+ 	((ListViewItemConf*)item)->expand();
+}
+
diff --git a/noncore/apps/confedit/listviewconfdir.h b/noncore/apps/confedit/listviewconfdir.h
index 3131972..978007b 100644
--- a/noncore/apps/confedit/listviewconfdir.h
+++ b/noncore/apps/confedit/listviewconfdir.h
@@ -23,7 +23,8 @@ class ListViewConfDir : public QListView  {
 public: 
 	ListViewConfDir(QString settingsPath, QWidget *parent=0, const char *name=0);
 	~ListViewConfDir();
-
+protected slots:
+	void expand(QListViewItem*);
 private:
 	QDir confDir;
 
diff --git a/noncore/apps/confedit/listviewitemconf.h b/noncore/apps/confedit/listviewitemconf.h
index 3c504ed..c005475 100644
--- a/noncore/apps/confedit/listviewitemconf.h
+++ b/noncore/apps/confedit/listviewitemconf.h
@@ -27,6 +27,7 @@ public:
   virtual void revert() = 0;
   virtual void remove() = 0;
   virtual void displayText() = 0;
+  virtual void expand() = 0;
   virtual void changed();
   bool isChanged() {return _changed;};
   virtual void unchanged();
diff --git a/noncore/apps/confedit/listviewitemconffile.cpp b/noncore/apps/confedit/listviewitemconffile.cpp
index 228421b..1f85095 100644
--- a/noncore/apps/confedit/listviewitemconffile.cpp
+++ b/noncore/apps/confedit/listviewitemconffile.cpp
@@ -20,7 +20,7 @@ ListViewItemConfFile::ListViewItemConfFile(QFileInfo *file, QListView *parent)
    : ListViewItemConf(parent), _valid(false)
 {
   confFileInfo = file;
-  parseFile();
+//  parseFile();
   displayText();
 }
 
@@ -41,6 +41,7 @@ QString ListViewItemConfFile::fileName()
 
 void ListViewItemConfFile::parseFile()
 {
+	//qDebug("ListViewItemConfFile::parseFile BEGIN");
   QFile confFile(confFileInfo->absFilePath());
 	if(! confFile.open(IO_ReadOnly))
  		QMessageBox::critical(0,tr("Could not open"),tr("The file ")+confFileInfo->fileName()+tr(" could not be opened."),1,0);	
@@ -52,7 +53,7 @@ void ListViewItemConfFile::parseFile()
   while ( !t.atEnd() )
   {
   	s = t.readLine().stripWhiteSpace();
-	//	qDebug( "line: >%s<\n", s.latin1() );
+		//qDebug( "line: >%s<\n", s.latin1() );
   	if (s.contains("<?xml"))
    	{
     	_valid = false;
@@ -62,19 +63,21 @@ void ListViewItemConfFile::parseFile()
    	{
   //    qDebug("got group"+s);
       group = s.mid(1,s.length()-2);
+      if (!groupItem) groupItem = new ListViewItemConfigEntry(this, tr("no group") );
       groupItem = new ListViewItemConfigEntry(this, group );
       insertItem( groupItem );
     } else
     if ( int pos = s.find('=') )
     {
- //     qDebug("got key"+s);
+//      qDebug("got key"+s);
+     	if (!groupItem) qDebug("PANIK NO GROUP! >%s<",group.latin1());
     	item = new ListViewItemConfigEntry(this, group, s );
       groupItem->insertItem( item );
     }
   }	
 	confFile.close();
- 	unchanged();
 	setExpandable( _valid );
+//	qDebug("ListViewItemConfFile::parseFile END");
 }
 
 
@@ -140,3 +143,9 @@ QString ListViewItemConfFile::backupFileName()
 {
 	return confFileInfo->absFilePath()+"~";
 }
+
+
+void ListViewItemConfFile::expand()
+{
+	parseFile();
+}
diff --git a/noncore/apps/confedit/listviewitemconffile.h b/noncore/apps/confedit/listviewitemconffile.h
index 679a2ed..18937af 100644
--- a/noncore/apps/confedit/listviewitemconffile.h
+++ b/noncore/apps/confedit/listviewitemconffile.h
@@ -22,14 +22,15 @@ class ListViewItemConfFile : public ListViewItemConf  {
 public:
 	ListViewItemConfFile(QFileInfo *file, QListView *parent=0);
 	~ListViewItemConfFile();
-  void parseFile();
 	QString fileName();
   void save();
   virtual void displayText();
   virtual bool revertable();
   virtual void revert();
   virtual void remove();
+  virtual void expand();
   QString backupFileName();
+  void parseFile();
 protected:
 private:
 	bool _valid;
diff --git a/noncore/apps/confedit/listviewitemconfigentry.cpp b/noncore/apps/confedit/listviewitemconfigentry.cpp
index b947514..5ef9f61 100644
--- a/noncore/apps/confedit/listviewitemconfigentry.cpp
+++ b/noncore/apps/confedit/listviewitemconfigentry.cpp
@@ -22,7 +22,6 @@ ListViewItemConfigEntry::ListViewItemConfigEntry(ListViewItemConfFile *parent, Q
  	_groupOrig = group;
   _keyOrig = _key;
   _valueOrig = _value;
- 	_fileItem->unchanged();
 }
 
 ListViewItemConfigEntry::~ListViewItemConfigEntry()
@@ -143,4 +142,10 @@ void ListViewItemConfigEntry::revert()
 	_value = _valueOrig;	
 	_changed=false;
  	displayText();
+}
+
+
+void ListViewItemConfigEntry::expand()
+{
+
 }
\ No newline at end of file
diff --git a/noncore/apps/confedit/listviewitemconfigentry.h b/noncore/apps/confedit/listviewitemconfigentry.h
index 6b651a6..c5f8737 100644
--- a/noncore/apps/confedit/listviewitemconfigentry.h
+++ b/noncore/apps/confedit/listviewitemconfigentry.h
@@ -35,6 +35,7 @@ public:
   virtual void displayText();
   virtual void changed();
   virtual void remove();
+  virtual void expand();
   void save(QTextStream*);
   virtual void revert();
 private:
diff --git a/noncore/apps/confedit/mainwindow.cpp b/noncore/apps/confedit/mainwindow.cpp
index fc403d2..6de8741 100644
--- a/noncore/apps/confedit/mainwindow.cpp
+++ b/noncore/apps/confedit/mainwindow.cpp
@@ -17,6 +17,7 @@
 #include <qpe/resource.h>
 #include <qpe/config.h>
 #include <qpe/qpetoolbar.h>
+#include <qpe/qpeapplication.h>
 #include <qaction.h>
 #include <qmessagebox.h>
 #include <qpopupmenu.h>
@@ -34,34 +35,35 @@
 
 
 MainWindow::MainWindow( QWidget *parent, const char *name, WFlags f ) :
-  QMainWindow( parent, name, f ), _currentItem(0), _fileItem(0)
+  QDialog( parent, name, f ), _currentItem(0), _fileItem(0)
 {	
   setCaption( tr("Conf File Editor") );
 
-  QWidget *mainWidget = new  QWidget(this);
-  setCentralWidget( mainWidget);
-  QGridLayout *mainLayout = new QGridLayout( mainWidget );
-	mainLayout->setSpacing( 3 );
-  mainLayout->setMargin( 3 );
+//	setBaseSize(  qApp->globalStrut() );
+  setSizePolicy( QSizePolicy( QSizePolicy::MinimumExpanding, QSizePolicy::MinimumExpanding));//, sizePolicy().hasHeightForWidth() ) );
 
+  mainLayout = new QVBoxLayout( this );
+	mainLayout->setSpacing( 0 );
+  mainLayout->setMargin( 0 );
 
-  qDebug("settingList");
+
+  qDebug("creating settingList");
   settingList = new ListViewConfDir( "/root/Settings/", this, "settingslist");
-  settingList->setSizePolicy( QSizePolicy( QSizePolicy::Expanding, QSizePolicy::Expanding));//, sizePolicy().hasHeightForWidth() ) );
-  mainLayout->addWidget( settingList, 0, 0 );
+  settingList->setSizePolicy( QSizePolicy( QSizePolicy::MinimumExpanding, QSizePolicy::MinimumExpanding));//, sizePolicy().hasHeightForWidth() ) );
+  mainLayout->addWidget( settingList, 0);
 
-  qDebug("editor");
+  qDebug("creating editor");
   editor = new EditWidget(this);
-  editor->setSizePolicy( QSizePolicy( QSizePolicy::Minimum, QSizePolicy::Minimum));//, sizePolicy().hasHeightForWidth() ) );
-//  editor->setSizePolicy( QSizePolicy( (QSizePolicy::SizeType)3, (QSizePolicy::SizeType)3));//, sizePolicy().hasHeightForWidth() ) );
-  mainLayout->addWidget( editor, 1, 0 );
+  editor->setSizePolicy( QSizePolicy( QSizePolicy::Minimum, QSizePolicy::Maximum));//, sizePolicy().hasHeightForWidth() ) );
+  mainLayout->addWidget( editor, 1 );
+  editor->layoutType( ListViewItemConf::File );
 
   makeMenu();
 
-  qDebug("connect");
   connect(settingList, SIGNAL( pressed(QListViewItem*) ),
 						this, SLOT(setCurrent(QListViewItem*)));
-
+	connect( settingList, SIGNAL( clicked( QListViewItem* ) ),
+					   this, SLOT( stopTimer( QListViewItem* ) ) );	
 
  	connect( editor->LineEditGroup, SIGNAL( textChanged(const QString&) ),
 	           SLOT( groupChanged(const QString&) ) );
@@ -69,47 +71,36 @@ MainWindow::MainWindow( QWidget *parent, const char *name, WFlags f ) :
 	           SLOT( keyChanged(const QString&) ) );
  	connect( editor->LineEditValue, SIGNAL( textChanged(const QString&) ),
 	           SLOT( valueChanged(const QString&) ) );
-//  qDebug("editor->hide()");
-// 	editor->hide();
-  qDebug("connect");
-	connect( settingList, SIGNAL( clicked( QListViewItem* ) ),
-					   this, SLOT( stopTimer( QListViewItem* ) ) );	
+
  	setCurrent(0);
 }
 
 void MainWindow::makeMenu()
 {
-  qDebug("MainWindow::makeMenu()");
-
   popupTimer = new QTimer(this);
   popupMenuFile = new QPopupMenu(this);
   popupMenuEntry = new QPopupMenu(this);
 
-  qDebug("Save");
 	popupActionSave = new QAction( tr("Save"),QString::null,  0, this, 0 );
 	popupActionSave->addTo( popupMenuFile );
  // popupActionSave->addTo( popupMenuEntry );
   connect( popupActionSave, SIGNAL( activated() ),
 		  	 	this , SLOT( saveConfFile() ) );
 
-  qDebug("Revert");
 	popupActionRevert = new QAction( tr("Revert"),QString::null,  0, this, 0 );
 	popupActionRevert->addTo( popupMenuFile );
   popupActionRevert->addTo( popupMenuEntry );
   connect( popupActionRevert, SIGNAL( activated() ),
 		  	 	this , SLOT( revertConfFile() ) );
 
-  qDebug("Delete");
   popupActionDelete = new QAction( tr("Delete"),QString::null,  0, this, 0 );
 	popupActionDelete->addTo( popupMenuFile );
   popupActionDelete->addTo( popupMenuEntry );
   connect( popupActionDelete, SIGNAL( activated() ),
 		  	 	this , SLOT( removeConfFile() ) );
 
-  qDebug("connect");
   connect( popupTimer, SIGNAL(timeout()),
 		  			this, SLOT(showPopup()) );
-  qDebug("connect");
 }
 
 MainWindow::~MainWindow()
@@ -120,19 +111,18 @@ MainWindow::~MainWindow()
 
 void MainWindow::setCurrent(QListViewItem *item)
 {
-  editor->hide();
+//	qDebug("MainWindow::setCurrent");
 	if (!item) return;
  	_item = (ListViewItemConf*) item;
   if (!_item) return;
   popupTimer->start( 750, true );
   if (_item->getType() == ListViewItemConf::File)
   {
-   	updateGeometry();
+	  editor->layoutType(EditWidget::File);
     _currentItem=0;
     _fileItem = (ListViewItemConfFile*)item;
    	return;
   }
-  editor->show();
   _fileItem = 0;
  	_currentItem = (ListViewItemConfigEntry*)item;
   if (!_currentItem) return;
@@ -144,15 +134,12 @@ void MainWindow::setCurrent(QListViewItem *item)
   editor->LineEditGroup->setText(group);
   if (!key.isEmpty())
   {
-   	editor->isKey(true);
+   	editor->layoutType(EditWidget::Entry);
     editor->LineEditKey->setText(key);
     editor->LineEditValue->setText(val);
   }else{
-    editor->isKey(false);
+    editor->layoutType(EditWidget::Group);
   }
-  updateGeometry();
-  editor->updateGeometry();
-  settingList->updateGeometry();
 }
 
 
@@ -197,6 +184,7 @@ void MainWindow::removeConfFile()
 	if (!_item) return;	
  	_item->remove();
 }
+
 void MainWindow::showPopup()
 {
 qDebug("showPopup");
diff --git a/noncore/apps/confedit/mainwindow.h b/noncore/apps/confedit/mainwindow.h
index 886b829..5be4a86 100644
--- a/noncore/apps/confedit/mainwindow.h
+++ b/noncore/apps/confedit/mainwindow.h
@@ -13,6 +13,7 @@
 #define MAINWINDOW_H
 
 #include <qmainwindow.h>
+#include <qdialog.h>
 #include <qaction.h>
 #include <qtimer.h>
 #include <qpopupmenu.h>
@@ -23,7 +24,7 @@ class ListViewItemConfFile;
 class ListViewConfDir;
 class ListViewItemConf;
 
-class MainWindow : public QMainWindow
+class MainWindow : public QDialog//QMainWindow
 {
     Q_OBJECT
 
@@ -47,6 +48,7 @@ public slots:
 private:
 	ListViewConfDir *settingList;
  	EditWidget *editor;
+  QVBoxLayout *mainLayout;
   ListViewItemConf *_item;
   ListViewItemConfigEntry *_currentItem;
   ListViewItemConfFile *_fileItem;