Fixed issue that Ctrl+A sometimes did not result in an update of the Information Panel. As a sideeffect some code simplification has been done, which was possible because of the column view refactorization some time ago.

svn path=/trunk/KDE/kdebase/apps/; revision=1070740
This commit is contained in:
Peter Penz 2010-01-06 16:56:01 +00:00
parent 5cf5a8ea4b
commit 4fdc3bad9c
5 changed files with 4 additions and 28 deletions

View file

@ -255,11 +255,6 @@ void DolphinController::emitViewportEntered()
emit viewportEntered();
}
void DolphinController::emitSelectionChanged()
{
emit selectionChanged();
}
void DolphinController::updateMouseButtonState()
{
m_mouseButtons = QApplication::mouseButtons();

View file

@ -62,7 +62,6 @@ class QPoint;
* - handleKeyPressEvent()
* - emitItemEntered()
* - emitViewportEntered()
* - emitSelectionChanged()
* - replaceUrlByClipboard()
* - hideToolTip()
* - setVersionControlActions()
@ -293,12 +292,6 @@ public slots:
*/
void emitViewportEntered();
/**
* Emits the signal selectionChanged(). The method should be invoked by
* the view implementation whenever the selection has been changed.
*/
void emitSelectionChanged();
signals:
/**
* Is emitted if the URL for the Dolphin controller has been changed
@ -407,13 +400,6 @@ signals:
*/
void viewportEntered();
/**
* Is emitted whenever the selection of the view implementation
* has been changed (see emitSelectionChanged()). The abstract
* Dolphin view connects to this signal.
*/
void selectionChanged();
/**
* Is emitted if the view should respect the name filter \a nameFilter. The view
* implementation must connect to this signal if it supports name filters.

View file

@ -1357,8 +1357,6 @@ void DolphinView::createView()
view->viewport()->installEventFilter(this);
m_controller->setItemView(view);
connect(m_controller, SIGNAL(selectionChanged()),
this, SLOT(emitDelayedSelectionChangedSignal()));
// When changing the view mode, the selection is lost due to reinstantiating
// a new item view with a custom selection model. Pass the ownership of the
@ -1369,6 +1367,8 @@ void DolphinView::createView()
m_selectionModel = view->selectionModel();
}
m_selectionModel->setParent(this);
connect(view->selectionModel(), SIGNAL(selectionChanged(const QItemSelection&, const QItemSelection&)),
this, SLOT(emitDelayedSelectionChangedSignal()));
connect(view->verticalScrollBar(), SIGNAL(valueChanged(int)),
this, SLOT(emitContentsMoved()));

View file

@ -171,9 +171,8 @@ void InformationPanel::showItemInfo()
} else {
// no item is hovered and no selection has been done: provide
// an item for the directory represented by m_shownUrl
m_fileItem = KFileItem(KFileItem::Unknown, KFileItem::Unknown, m_shownUrl);
m_fileItem.refresh();
item = m_fileItem;
item = KFileItem(KFileItem::Unknown, KFileItem::Unknown, m_shownUrl);
item.refresh();
}
m_content->showItem(item);

View file

@ -112,10 +112,6 @@ ViewExtensionsFactory::ViewExtensionsFactory(QAbstractItemView* view,
connect(dolphinView, SIGNAL(sortFoldersFirstChanged(bool)),
this, SLOT(slotSortFoldersFirstChanged(bool)));
// inform the controller about selection changes
connect(view->selectionModel(), SIGNAL(selectionChanged(const QItemSelection&, const QItemSelection&)),
controller, SLOT(emitSelectionChanged()));
// Give the view the ability to auto-expand its directories on hovering
// (the column view takes care about this itself). If the details view
// uses expandable folders, the auto-expanding should be used always.