From 9e12957c363a68543456dfe95ab88a11a69ae610 Mon Sep 17 00:00:00 2001 From: Arno Date: Thu, 5 Aug 2010 20:39:09 +0200 Subject: Fixed edit trigger in SeriesTreeView Until now doubleclicking an item with children in SeriesTreeWidget did 3 things: 1. Expand or collapse the item 2. Trigger editing 3. Start playing all child movies in the default player This is not really expected behavior. So I disabled all EditTriggers on SeriesTreeView and set expandsOnDoubleClick to false. To still retain the ability to edit series a new slot was introduced to SeriesTreeWidget triggering the edit event. It's available from the SeriesTreeWidget context menu. Now doubleclicking an item in SeriesTreeView just plays all child movies. Child items also have their name changed when changing the parent item. This bug was long present in renameSeries. mergeSeries always did the right thing (tm). --- seriestreewidget.cpp | 9 +++++++++ 1 file changed, 9 insertions(+) (limited to 'seriestreewidget.cpp') diff --git a/seriestreewidget.cpp b/seriestreewidget.cpp index b09d7f7..e717b13 100644 --- a/seriestreewidget.cpp +++ b/seriestreewidget.cpp @@ -58,6 +58,8 @@ SeriesTreeWidget::SeriesTreeWidget(QWidget *parent) : QWidget(parent){ } mView->resizeColumnToContents(0); mView->setSelectionMode(QAbstractItemView::ExtendedSelection); + mView->setEditTriggers(QAbstractItemView::NoEditTriggers); + mView->setExpandsOnDoubleClick(false); connect(mModel, SIGNAL(needResort()), this, SLOT(resort())); connect(mView, SIGNAL(expanded(QModelIndex)), this, SLOT(itemExpanded(QModelIndex))); connect(mView, SIGNAL(collapsed(QModelIndex)), this, SLOT(itemCollaped(QModelIndex))); @@ -268,6 +270,13 @@ void SeriesTreeWidget::expandItems(const QStringList &items){ } } +void SeriesTreeWidget::editItem(){ + QModelIndex current = mView->selectionModel()->currentIndex(); + if(current.isValid()){ + mView->edit(current); + } +} + void SeriesTreeWidget::addCover(){ QModelIndexList selected = mView->selectionModel()->selectedRows(); if(selected.isEmpty()){ -- cgit v1.2.3-70-g09d2