fixed bug woth my eyes no seeing the path combo very well, also fixed bug with popupmenu

2 files changed, 155 insertions, 106 deletions

diff --git a/noncore/apps/advancedfm/advancedfm.cpp b/noncore/apps/advancedfm/advancedfm.cpp
index f32031b..869a433 100644
--- a/noncore/apps/advancedfm/advancedfm.cpp
+++ b/noncore/apps/advancedfm/advancedfm.cpp
@@ -70,8 +70,9 @@
 AdvancedFm::AdvancedFm( )
         : QMainWindow( )
 {
     setCaption( tr( "AdvancedFm" ) );
+//      menuTimer( this );
 
     QGridLayout *layout = new QGridLayout( this );
     layout->setSpacing( 2);
     layout->setMargin( 2);
@@ -143,10 +144,27 @@ AdvancedFm::AdvancedFm( )
     viewMenu->insertSeparator();
     viewMenu->insertItem( tr( "About" ), this, SLOT( doAbout() ));
     viewMenu->setCheckable(TRUE);
 
+
+    currentPathCombo = new QComboBox( FALSE, this, "currentPathCombo" );
+    currentPathCombo->setEditable(TRUE);
+    layout->addMultiCellWidget( currentPathCombo, 1, 1, 0, 7);
+    currentPathCombo->lineEdit()->setText( currentDir.canonicalPath());
+
+    connect( currentPathCombo, SIGNAL( activated( const QString & ) ),
+              this, SLOT(  currentPathComboActivated( const QString & ) ) );
+
+    connect( currentPathCombo->lineEdit(),SIGNAL(returnPressed()),
+             this,SLOT(currentPathComboChanged()));
+
+    currentPathCombo->lineEdit()->setText( currentDir.canonicalPath());
+
+    layout->addMultiCellWidget( currentPathCombo, 1, 1, 0, 7);
+
+
     TabWidget = new QTabWidget( this, "TabWidget" );
-    layout->addMultiCellWidget( TabWidget, 1, 1, 0, 7);
+    layout->addMultiCellWidget( TabWidget, 2, 2, 0, 7);
 
     tab = new QWidget( TabWidget, "tab" );
     tabLayout = new QGridLayout( tab );
     tabLayout->setSpacing( 2);
@@ -164,14 +182,17 @@ AdvancedFm::AdvancedFm( )
      Local_View->setSelectionMode(QListView::Extended);
 
      QPEApplication::setStylusOperation( Local_View->viewport(),QPEApplication::RightOnHold);
 
+
+
     tabLayout->addWidget( Local_View, 0, 0 );
 
     connect( Local_View, SIGNAL( clicked( QListViewItem*)),
              this,SLOT( localListClicked(QListViewItem *)) );
     connect( Local_View, SIGNAL( mouseButtonPressed( int, QListViewItem *, const QPoint&, int)),
              this,SLOT( localListPressed(int, QListViewItem *, const QPoint&, int)) );
+    connect( Local_View, SIGNAL( selectionChanged() ), SLOT( cancelMenuTimer() ) );
 
     TabWidget->insertTab( tab, tr("1"));
 
     tab_2 = new QWidget( TabWidget, "tab_2" );
@@ -194,8 +215,9 @@ AdvancedFm::AdvancedFm( )
     connect( Remote_View, SIGNAL( clicked( QListViewItem*)),
              this,SLOT( remoteListClicked(QListViewItem *)) );
     connect( Remote_View, SIGNAL( mouseButtonPressed( int, QListViewItem *, const QPoint&, int)),
              this,SLOT( remoteListPressed(int, QListViewItem *, const QPoint&, int)) );
+    connect( Remote_View, SIGNAL( selectionChanged() ), SLOT( cancelMenuTimer() ) );
 
     tabLayout_2->addWidget( Remote_View, 0, 0 );
 
     TabWidget->insertTab( tab_2, tr( "2"));
@@ -220,25 +242,14 @@ AdvancedFm::AdvancedFm( )
     currentRemoteDir.setFilter( QDir::Files | QDir::Dirs | QDir::Hidden | QDir::All);
     currentRemoteDir.setPath( QDir::currentDirPath());
 
     b = TRUE;
-    currentPathCombo = new QComboBox( FALSE, this, "currentPathCombo" );
-    currentPathCombo->setEditable(TRUE);
-    layout->addMultiCellWidget( currentPathCombo, 3, 3, 0, 7);
-    currentPathCombo->lineEdit()->setText( currentDir.canonicalPath());
-
-    connect( currentPathCombo, SIGNAL( activated( const QString & ) ),
-              this, SLOT(  currentPathComboActivated( const QString & ) ) );
-
-    connect( currentPathCombo->lineEdit(),SIGNAL(returnPressed()),
-             this,SLOT(currentPathComboChanged()));
-
-    currentPathCombo->lineEdit()->setText( currentDir.canonicalPath());
-
-    layout->addMultiCellWidget( currentPathCombo, 3, 3, 0, 7);
 
     filterStr="*";
     b=FALSE;
+
+    connect( &menuTimer, SIGNAL( timeout() ), SLOT( showFileMenu() ) );
+    
     populateLocalView();
     populateRemoteView();
 }
 
@@ -640,109 +651,31 @@ void AdvancedFm::localListPressed( int mouse, QListViewItem *item, const QPoint
     switch (mouse) {
       case 1:
           break;
       case 2:
-          showLocalMenu(item);
-          Local_View->clearSelection();
+           menuTimer.start( 500, TRUE );
     break;
     };
 }
 
 void AdvancedFm::remoteListPressed( int mouse, QListViewItem *item, const QPoint &point, int i)
 {
+
  switch (mouse) {
       case 1:
           break;
       case 2:
-          showRemoteMenu(item);
-          Remote_View->clearSelection();
+           menuTimer.start( 500, TRUE );
       break;
     };
 }
 
-void AdvancedFm::showLocalMenu(QListViewItem * item)
-{
-    if(item) {
-        QPopupMenu  m;
-        m.insertItem(  tr( "Show Hidden Files" ), this,  SLOT( showHidden() ));
-        m.insertSeparator();
-        if( /*item->text(0).right(1) == "/" ||*/ item->text(0).find("/",0,TRUE) !=-1)
-            m.insertItem( tr( "Change Directory" ), this, SLOT( doLocalCd() ));
-        else
-            m.insertItem( tr( "Open / Execute" ), this, SLOT( runThis() ));
-        m.insertItem( tr( "Open as Text" ), this, SLOT( runText() ));
-        m.insertItem( tr( "Make Directory" ), this, SLOT( localMakDir() ));
-        m.insertItem( tr( "Make Symlink" ), this, SLOT( mkSym() ));
-        m.insertSeparator();
-        m.insertItem( tr( "Rename" ), this, SLOT( localRename() ));
-        m.insertItem( tr( "Copy" ), this, SLOT( copy() ));
-        m.insertItem( tr( "Copy As" ), this, SLOT( copyAs() ));
-        m.insertItem( tr( "Move" ), this, SLOT( move() ));
-        m.insertSeparator();
-        m.insertItem( tr( "Rescan" ), this, SLOT( populateLocalView() ));
-        m.insertItem( tr( "Run Command" ), this, SLOT( runCommand() ));
-        m.insertItem( tr( "File Info" ), this, SLOT( fileStatus() ));
-        m.insertSeparator();
-        m.insertItem( tr( "Delete" ), this, SLOT( localDelete() ));
-        m.insertSeparator();
-        m.insertItem( tr( "Set Permissions" ), this, SLOT( filePerms() ));
-        if( QFile(QPEApplication::qpeDir()+"lib/libopie.so").exists() )  //bad hack for Sharp zaurus failings
-        m.insertItem( tr( "Properties" ), this, SLOT( doProperties() ));
-        m.setCheckable(TRUE);
-        if (!b)
-            m.setItemChecked(m.idAt(0),TRUE);
-        else
-            m.setItemChecked(m.idAt(0),FALSE);
-        if(Ir::supported())
-        m.insertItem( tr( "Beam File" ), this, SLOT( doBeam() ));
-        m.exec( QCursor::pos() );
-    }
-}
-
-void AdvancedFm::showRemoteMenu(QListViewItem * item)
-{
-    if(item) {
-        QPopupMenu  m;
-        m.insertItem(  tr( "Show Hidden Files" ), this,  SLOT( showRemoteHidden() ));
-        m.insertSeparator();
-        if( /*item->text(0).right(1) == "/" ||*/ item->text(0).find("/",0,TRUE) !=-1)
-            m.insertItem( tr( "Change Directory" ), this, SLOT( doRemoteCd() ));
-        else
-            m.insertItem( tr( "Open / Execute" ), this, SLOT( runThis() ));
-        m.insertItem( tr( "Open as Text" ), this, SLOT( runText() ));
-        m.insertItem( tr( "Make Directory" ), this, SLOT( remoteMakDir() ));
-        m.insertItem( tr( "Make Symlink" ), this, SLOT( mkSym() ));
-        m.insertSeparator();
-        m.insertItem( tr( "Rename" ), this, SLOT( remoteRename() ));
-        m.insertItem( tr( "Copy" ), this, SLOT( copy() ));
-        m.insertItem( tr( "Copy As" ), this, SLOT( copyAs() ));
-        m.insertItem( tr( "Move" ), this, SLOT( move() ));
-        m.insertSeparator();
-        m.insertItem( tr( "Rescan" ), this, SLOT( populateRemoteView() ));
-        m.insertItem( tr( "Run Command" ), this, SLOT( runCommand() ));
-        m.insertItem( tr( "File Info" ), this, SLOT( fileStatus() ));
-        m.insertSeparator();
-        m.insertItem( tr( "Delete" ), this, SLOT( remoteDelete() ));
-        m.insertSeparator();
-        m.insertItem( tr( "Set Permissions" ), this, SLOT( filePerms() ));
-    if( QFile(QPEApplication::qpeDir()+"lib/libopie.so").exists() )  //bad hack for Sharp zaurus failings
-        m.insertItem( tr( "Properties" ), this, SLOT( doProperties() ));
-        m.setCheckable(TRUE);
-        if (!b)
-            m.setItemChecked(m.idAt(0),TRUE);
-        else
-            m.setItemChecked(m.idAt(0),FALSE);
-        if(Ir::supported())
-        m.insertItem( tr( "Beam File" ), this, SLOT( doBeam() ));
-        m.exec( QCursor::pos() );
-    }
-}
-
 void AdvancedFm::runThis() {
 //    QFileInfo *fi;
 QString fs;
     if (TabWidget->currentPageIndex() == 0) {
         QString curFile = Local_View->currentItem()->text(0);
+        if(curFile != "../") {
 
         fs= getFileSystemType((const QString &) currentDir.canonicalPath());
         QFileInfo fileInfo( currentDir.canonicalPath()+"/"+curFile);
         qDebug( fileInfo.owner());
@@ -762,11 +695,14 @@ QString fs;
             } else {
                 nf.execute();
             }
          }
+        }
 //         MimeType mt( curFile);
     } else {
         QString curFile = Remote_View->currentItem()->text(0);
+        if(curFile != "../") {
+
         fs= getFileSystemType((const QString &) currentRemoteDir.canonicalPath());
         qDebug("Filesystemtype is "+fs);
         QFileInfo fileInfo( currentRemoteDir.canonicalPath()+"/"+curFile);
         if( (fileInfo.permission( QFileInfo::ExeUser)
@@ -784,26 +720,31 @@ QString fs;
             } else {
                 nf.execute();
             }
          }
+        }
 //         MimeType mt( curFile);
     }
 }
 
 void AdvancedFm::runText() {
     if (TabWidget->currentPageIndex() == 0) {
         QString curFile = Local_View->currentItem()->text(0);
+        if(curFile != "../") {
         curFile =  currentDir.canonicalPath()+"/"+curFile;
         QCopEnvelope e("QPE/Application/textedit", "setDocument(QString)" );
         e << curFile;
+        }
     } else {
         QString curFile = Remote_View->currentItem()->text(0);
+        if(curFile != "../") {
         curFile =  currentRemoteDir.canonicalPath()+"/"+curFile;
         DocLnk nf(curFile);
         QCopEnvelope e("QPE/Application/textedit", "setDocument(QString)" );
         e << curFile;
     }
 }
+}
 
 void AdvancedFm::localMakDir()
 {
     InputDialog *fileDlg;
@@ -830,8 +771,9 @@ void AdvancedFm::remoteMakDir()
 
 void AdvancedFm::localDelete()
 {
     QStringList curFileList = getPath();
+    if(curFileList.count() > 0) {
     QString myFile;
     for ( QStringList::Iterator it = curFileList.begin(); it != curFileList.end(); ++it ) {
         myFile = (*it);
         if( myFile.find(" -> ",0,TRUE) != -1)
@@ -874,12 +816,14 @@ void AdvancedFm::localDelete()
             };
         }
     }
 }
+}
 
 void AdvancedFm::remoteDelete()
 {
     QStringList curFileList = getPath();
+    if( curFileList.count() > 0) {
     QString myFile;
 
     for ( QStringList::Iterator it = curFileList.begin(); it != curFileList.end(); ++it ) {
         myFile = (*it);
@@ -922,12 +866,15 @@ void AdvancedFm::remoteDelete()
             };
         }
     }
 }
+}
 
 void AdvancedFm::localRename()
 {
     QString curFile = Local_View->currentItem()->text(0);
+    qDebug("currentItem "+curFile);
+    if( curFile !="../") {
     InputDialog *fileDlg;
     fileDlg = new InputDialog(this,tr("Rename"),TRUE, 0);
     fileDlg->setInputText((const QString &)curFile);
     fileDlg->exec();
@@ -938,12 +885,14 @@ void AdvancedFm::localRename()
             QMessageBox::message(tr("Note"),tr("Could not rename"));
     }
     populateLocalView();
 }
+}
 
 void AdvancedFm::remoteRename()
 {
-    QString curFile = Local_View->currentItem()->text(0);
+        QString curFile = Remote_View->currentItem()->text(0);
+    if( curFile !="../") {
     InputDialog *fileDlg;
     fileDlg = new InputDialog(this,tr("Rename"),TRUE, 0);
     fileDlg->setInputText((const QString &)curFile);
     fileDlg->exec();
@@ -954,8 +903,9 @@ void AdvancedFm::remoteRename()
             QMessageBox::message(tr("Note"),tr("Could not rename"));
     }
     populateRemoteView();
 }
+}
 
 void AdvancedFm::switchToLocalTab()
 {
     TabWidget->setCurrentPage(0);
@@ -1054,17 +1004,14 @@ void AdvancedFm::filePerms() {
         populateLocalView();
     } else {
         populateRemoteView();
     }
-
-
 }
 
 void AdvancedFm::doProperties() {
     QStringList curFileList = getPath();
     QString filePath;
     if (TabWidget->currentPageIndex() == 0) {
-
         filePath = currentDir.canonicalPath()+"/";
     } else {
         filePath= currentRemoteDir.canonicalPath()+"/";
     }
@@ -1183,8 +1130,9 @@ void AdvancedFm::upDir()
 
 void AdvancedFm::copy()
 {
     QStringList curFileList = getPath();
+    if( curFileList.count() > 0) {
     QString curFile;
     if (TabWidget->currentPageIndex() == 0) {
         for ( QStringList::Iterator it = curFileList.begin(); it != curFileList.end(); ++it ) {
 
@@ -1227,12 +1175,14 @@ void AdvancedFm::copy()
         populateLocalView();
         TabWidget->setCurrentPage(0);
     }
 }
+}
 
 void AdvancedFm::copyAs()
 {
     QStringList curFileList = getPath();
+    if( curFileList.count() > 0) {
     QString curFile;
     InputDialog *fileDlg;
     fileDlg = new InputDialog(this,tr("Copy As"),TRUE, 0);
 
@@ -1285,12 +1235,14 @@ void AdvancedFm::copyAs()
             TabWidget->setCurrentPage(0);
         }
     }
 }
+}
 
 void AdvancedFm::move() {
 
     QStringList curFileList = getPath();
+    if( curFileList.count() > 0) {
     QString curFile;
 //    qDebug(curFile);
     QString destFile;
 
@@ -1341,8 +1293,9 @@ void AdvancedFm::move() {
     }
         populateRemoteView();
         populateLocalView();
  }
+ }
 
  bool AdvancedFm::copyFile( const QString & dest, const QString & src )
 {
     char bf[ 50000 ];
@@ -1573,8 +1526,9 @@ void AdvancedFm::keyReleaseEvent( QKeyEvent *e)
 
 void AdvancedFm::mkSym() {
     QString cmd;
     QStringList curFileList = getPath();
+    if( curFileList.count() > 0) {
 
     if (TabWidget->currentPageIndex() == 0) {
         for ( QStringList::Iterator it = curFileList.begin(); it != curFileList.end(); ++it ) {
 
@@ -1603,8 +1557,9 @@ void AdvancedFm::mkSym() {
             populateLocalView();
             TabWidget->setCurrentPage(0);
     }
 }
+}
 
 void AdvancedFm::QPEButtonPushed() {
     QString current = QPEApplication::qpeDir();
     chdir( current.latin1() );
@@ -1667,8 +1622,9 @@ void AdvancedFm::doBeam() {
   if(!ir.supported()){
   } else {
 
       QStringList curFileList = getPath();
+        if( curFileList.count() > 0) {
 
       if (TabWidget->currentPageIndex() == 0) {
           for ( QStringList::Iterator it = curFileList.begin(); it != curFileList.end(); ++it ) {
 
@@ -1692,8 +1648,10 @@ void AdvancedFm::doBeam() {
       }
   }
 }
 
+}
+
 void AdvancedFm::fileBeamFinished( Ir *ir) {
   QMessageBox::message( tr("Advancedfm Beam out"), tr("Ir sent.") ,tr("Ok") );
 
 }
@@ -1707,4 +1665,89 @@ void AdvancedFm::fileBeamFinished( Ir *ir) {
 //             QString localFile = currentDir.canonicalPath()+"/"+strItem;
 //             QFileInfo fi(localFile);
 //         }
 //     }
+
+void AdvancedFm::showFileMenu() {
+  QString curApp;
+  bool isLocalView = false;
+  if (TabWidget->currentPageIndex() == 0) {
+    isLocalView = TRUE;
+    curApp = Local_View->currentItem()->text(0);
+  }   else {
+    curApp = Remote_View->currentItem()->text(0);
+  }
+
+  MimeType mt( curApp );
+  const AppLnk* app = mt.application();
+  QFile fi(curApp);
+
+  QPopupMenu m;
+  m.insertItem(  tr( "Show Hidden Files" ), this,  SLOT( showHidden() ));
+  m.insertSeparator();
+  if ( QFileInfo(fi).isDir() ) {
+    m.insertItem( tr( "Change Directory" ), this, SLOT( doLocalCd() ));
+  } else {
+
+    if ( app )
+      m.insertItem( app->pixmap(), tr( "Open in " + app->name() ), this, SLOT( run() ) );
+    else if( QFileInfo(fi).isExecutable() )
+      m.insertItem(  tr( "Execute" ), this, SLOT( run() ) );
+
+    m.insertItem( /*Resource::loadPixmap( "txt" ),*/ tr( "Open as text" ),this, SLOT( runText() ) );
+  }
+    m.insertSeparator();
+
+
+    if(isLocalView)
+      m.insertItem( tr( "Make Directory" ), this, SLOT( localMakDir() ));
+    else
+      m.insertItem( tr( "Make Directory" ), this, SLOT( remoteMakDir() ));
+
+    m.insertItem( tr( "Make Symlink" ), this, SLOT( mkSym() ));
+    m.insertSeparator();
+
+    if(isLocalView)
+      m.insertItem( tr( "Rename" ), this, SLOT( localRename() ));
+    else
+      m.insertItem( tr( "Rename" ), this, SLOT( remoteRename() ));
+
+    m.insertItem( tr( "Copy" ), this, SLOT( copy() ));
+    m.insertItem( tr( "Copy As" ), this, SLOT( copyAs() ));
+    m.insertItem( tr( "Move" ), this, SLOT( move() ));
+    m.insertSeparator();
+
+    if(isLocalView)
+      m.insertItem( tr( "Rescan" ), this, SLOT( populateLocalView() ));
+    else
+      m.insertItem( tr( "Rescan" ), this, SLOT( populateRemoteView() ));
+
+    m.insertItem( tr( "Run Command" ), this, SLOT( runCommand() ));
+    m.insertItem( tr( "File Info" ), this, SLOT( fileStatus() ));
+    m.insertSeparator();
+
+    if(isLocalView)
+      m.insertItem( tr( "Delete" ), this, SLOT( localDelete() ));
+    else
+      m.insertItem( tr( "Delete" ), this, SLOT( remoteDelete() ));
+
+    m.insertSeparator();
+    m.insertItem( tr( "Set Permissions" ), this, SLOT( filePerms() ));
+    if( QFile(QPEApplication::qpeDir()+"lib/libopie.so").exists() )  //bad hack for Sharp zaurus failings
+      m.insertItem( tr( "Properties" ), this, SLOT( doProperties() ));
+    m.setCheckable(TRUE);
+    if (!b)
+      m.setItemChecked(m.idAt(0),TRUE);
+    else
+      m.setItemChecked(m.idAt(0),FALSE);
+    if(Ir::supported())
+      m.insertItem( tr( "Beam File" ), this, SLOT( doBeam() ));
+    m.exec( QCursor::pos() );
+}
+
+
+void AdvancedFm::cancelMenuTimer()
+{
+qDebug("cancel menu timer");
+    if( menuTimer.isActive() )
+    menuTimer.stop();
+}
diff --git a/noncore/apps/advancedfm/advancedfm.h b/noncore/apps/advancedfm/advancedfm.h
index 5817fca..0cf94c6 100644
--- a/noncore/apps/advancedfm/advancedfm.h
+++ b/noncore/apps/advancedfm/advancedfm.h
@@ -11,16 +11,18 @@
  ***************************************************************************/
 #ifndef ADVANCEDFM_H
 #define ADVANCEDFM_H
 
+#include <qpe/ir.h>
+
 #include <qvariant.h>
 #include <qdialog.h>
 #include <qmainwindow.h>
+#include <qstringlist.h>
 #include <qdir.h>
 #include <qstring.h>
 #include <qpoint.h>
-#include <qstringlist.h>
-#include <qpe/ir.h>
+#include <qtimer.h>
 
 class QVBoxLayout;
 class QHBoxLayout;
 class QGridLayout;
@@ -38,8 +40,9 @@ class QPopupMenu;
 class QFile;
 class QListViewItem;
 class QLineEdit;
 class QPushButton;
+class Ir;
 
 class AdvancedFm : public QMainWindow
 {
     Q_OBJECT
@@ -61,10 +64,8 @@ public:
     bool b;
     QStringList fileSystemTypeList, fsList;
     int currentServerConfig;
 protected slots:
-    void showLocalMenu( QListViewItem *);
-    void showRemoteMenu( QListViewItem *);
     void doLocalCd();
     void doRemoteCd();
 //    void copy();
     void mkDir();
@@ -103,13 +104,16 @@ protected slots:
 
 protected:
   QGridLayout *tabLayout, *tabLayout_2, *tabLayout_3;
   QStringList remoteDirPathStringList, localDirPathStringList;
+
   void keyReleaseEvent( QKeyEvent *);
   QString getFileSystemType(const QString &);
   void parsetab(const QString &fileName);
   
 protected slots:
+  void showFileMenu();
+  void cancelMenuTimer();
   void homeButtonPushed();
   void docButtonPushed();
   void SDButtonPushed();
   void CFButtonPushed();
@@ -124,8 +128,10 @@ protected slots:
   void move();
   void fileStatus();
   void doAbout();
   void doBeam();
-  void fileBeamFinished( Ir *ir);
+  void fileBeamFinished( Ir *);
+private:
+      QTimer menuTimer;
 };
 
 #endif // ADVANCEDFM_H