forked from qt-creator/qt-creator
QmlJSDebugger: Simplify selection handling code
Avoids multiple calls singleSelectionManipulator.select().
This commit is contained in:
@@ -107,51 +107,9 @@ void SelectionTool::mousePressEvent(QMouseEvent *event)
|
|||||||
if (m_rubberbandSelectionMode) {
|
if (m_rubberbandSelectionMode) {
|
||||||
m_rubberbandSelectionManipulator.begin(event->pos());
|
m_rubberbandSelectionManipulator.begin(event->pos());
|
||||||
} else {
|
} else {
|
||||||
|
|
||||||
if (itemList.isEmpty()) {
|
|
||||||
QDeclarativeViewObserverPrivate::get(observer())->setSelectedItems(itemList);
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
|
|
||||||
if ((selectionType == SingleSelectionManipulator::InvertSelection
|
|
||||||
|| selectionType == SingleSelectionManipulator::ReplaceSelection)
|
|
||||||
&& alreadySelected(itemList))
|
|
||||||
{
|
|
||||||
//view()->changeToMoveTool(event->pos());
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
|
|
||||||
QGraphicsItem* item = itemList.first();
|
|
||||||
|
|
||||||
if (item->children().isEmpty()) {
|
|
||||||
m_singleSelectionManipulator.begin(event->pos());
|
m_singleSelectionManipulator.begin(event->pos());
|
||||||
m_singleSelectionManipulator.select(selectionType, m_selectOnlyContentItems);
|
m_singleSelectionManipulator.select(selectionType, m_selectOnlyContentItems);
|
||||||
} else {
|
|
||||||
m_mousePressTimer.start();
|
|
||||||
|
|
||||||
if (itemList.isEmpty()) {
|
|
||||||
observer()->setSelectedItems(itemList);
|
|
||||||
return;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
if (item->children().isEmpty()) {
|
|
||||||
m_singleSelectionManipulator.begin(event->pos());
|
|
||||||
m_singleSelectionManipulator.select(selectionType, m_selectOnlyContentItems);
|
|
||||||
} else {
|
|
||||||
m_singleSelectionManipulator.begin(event->pos());
|
|
||||||
m_singleSelectionManipulator.select(selectionType, m_selectOnlyContentItems);
|
|
||||||
m_singleSelectionManipulator.end(event->pos());
|
|
||||||
//view()->changeToMoveTool(event->pos());
|
|
||||||
}
|
|
||||||
|
|
||||||
m_singleSelectionManipulator.begin(event->pos());
|
|
||||||
m_singleSelectionManipulator.select(selectionType, m_selectOnlyContentItems);
|
|
||||||
m_singleSelectionManipulator.end(event->pos());
|
|
||||||
//view()->changeToMoveTool(event->pos());
|
|
||||||
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
} else if (event->buttons() & Qt::RightButton) {
|
} else if (event->buttons() & Qt::RightButton) {
|
||||||
createContextMenu(itemList, event->globalPos());
|
createContextMenu(itemList, event->globalPos());
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user