From 3cc491f45a80fce397c9440397f1190968dcbadf Mon Sep 17 00:00:00 2001 From: Thomas Hartmann Date: Thu, 28 Apr 2011 17:12:45 +0200 Subject: [PATCH] QmlDesigner.navigator: add tool buttons for sliding/reparenting This patch adds for tool buttons that allow sliding and reparenting in the navigator. --- .../components/navigator/arrowdown.png | Bin 0 -> 390 bytes .../components/navigator/arrowleft.png | Bin 0 -> 471 bytes .../components/navigator/arrowright.png | Bin 0 -> 455 bytes .../components/navigator/arrowup.png | Bin 0 -> 376 bytes .../components/navigator/navigator.pri | 2 + .../components/navigator/navigator.qrc | 8 ++ .../components/navigator/navigatorview.cpp | 79 +++++++++++++++++- .../components/navigator/navigatorview.h | 7 +- .../components/navigator/navigatorwidget.cpp | 58 ++++++++++++- .../components/navigator/navigatorwidget.h | 14 +++- src/plugins/qmldesigner/designmodewidget.cpp | 21 +++++ 11 files changed, 182 insertions(+), 7 deletions(-) create mode 100644 src/plugins/qmldesigner/components/navigator/arrowdown.png create mode 100644 src/plugins/qmldesigner/components/navigator/arrowleft.png create mode 100644 src/plugins/qmldesigner/components/navigator/arrowright.png create mode 100644 src/plugins/qmldesigner/components/navigator/arrowup.png create mode 100644 src/plugins/qmldesigner/components/navigator/navigator.qrc diff --git a/src/plugins/qmldesigner/components/navigator/arrowdown.png b/src/plugins/qmldesigner/components/navigator/arrowdown.png new file mode 100644 index 0000000000000000000000000000000000000000..0efd6b4de362219d87d547f13eb6fc26f28d2717 GIT binary patch literal 390 zcmeAS@N?(olHy`uVBq!ia0y~yU=RRd4mJh`2Kmqb6B!s77>k44ofy`glX=O&z`&N| z?e4+=20xv5*E29Ma29w(7Bet#3xhBt!>l@z1_}LrU-R2qcB;Wuze-2npY2WkduO($ z37^fm?lI+VMe!oz&5S!N&KqZHo|3#{nfz_<63HD_58j-RoDu(R{e6|Qb0^oc9Xs?n z!Y8>bWs(VFi7UU`gEwj$dUOk>mQ;OR@QJ_gz(viDJiWY0e7@3?x2!Y@3HDs4wPK31 kX7$_jsD|Ka|9HQ!>|EpeRZwqpBPa+xUHx3vIVCg!0GcC?0RR91 literal 0 HcmV?d00001 diff --git a/src/plugins/qmldesigner/components/navigator/arrowleft.png b/src/plugins/qmldesigner/components/navigator/arrowleft.png new file mode 100644 index 0000000000000000000000000000000000000000..c9bf785829741d962e10adbcc901b317e1654887 GIT binary patch literal 471 zcmeAS@N?(olHy`uVBq!ia0y~yU=RRd4mJh`2Kmqb6B!s77>k44ofy`glX=O&z`&N| z?e4+=20xv5*E29Ma29w(7Bet#3xhBt!>l`Ki;O^-g5Z=fq&cMKE9&2L@zdjS|P4mbAYifi8tKh{3hj|=Q}&)x3*YkJPt0aYnj}#tZ>#( z(^o$xE3K_^$x->EYOViB`dPuuY&*}!ym`+H8>{v%s%u+raV+Q1p$jL`!8h|pLxIH!s$PbeqT3aB}vP^QNMG*iD|Lu#MLJCLie-(+dMYfRxo3+ z-LcIPYNLrbU{?ao%Usb0C-Ah(Ko_FVEyvEWs%jNgqQCCXJ3^*Hq|8mNk T%dfKjg5t!})z4*}Q$iB}>P5f< literal 0 HcmV?d00001 diff --git a/src/plugins/qmldesigner/components/navigator/arrowright.png b/src/plugins/qmldesigner/components/navigator/arrowright.png new file mode 100644 index 0000000000000000000000000000000000000000..06f50ebae8d1dfe6b3ac5ef64499a036fa60672a GIT binary patch literal 455 zcmeAS@N?(olHy`uVBq!ia0y~yU=RRd4mJh`2Kmqb6B!s77>k44ofy`glX=O&z`&N| z?e4+=20xv5*E29Ma29w(7Bet#3xhBt!>lOvOb}&fziRo>mW)-2nn~y%Y=OXW5>VKiYHJ zuk2jv+^yekJ(sC+aXC|dSi0(XrpufE?hkmo1vh>TZ{(LrR#h_luVm0uJtx6b$?R1^ zyvvHz`=|4OaM`}_e}yhba?Jm5GiCWs>^!bl4vGU$S3j3^P6k44ofy`glX=O&z`&N| z?e4+=20xv5*E29Ma29w(7Bet#3xhBt!>lbCYGe8D3oWGWGJ|M`UZqI@`*DrFkJR@aSX9IJ$Le1-ev=y*8L0g zCiG31{64l(^aV!=i%Ce{?UdHI9+61SzZ+XzgzWEq=Q-fG^3{@6{fp-n zD9AO&?&&zO_S*GYf%6BLcyuQP9A;A!PQUGUfZ>c%QNGZO&nfG6THMOX|Dv+6Y3~dM zgCP&aF9)=gt}j^CZ0UY|#}3Ia?`oLerM9Rj&Az+8Vfo?ru21gN z^zPnqLhi2B^-pu8EH}r#JZ$qyev+ly_GjTuFVy}Av%H`4zw<+A>D2m%L7f-Ym>W&; U-ze(zvj7xmp00i_>zopr01EJsr2qf` literal 0 HcmV?d00001 diff --git a/src/plugins/qmldesigner/components/navigator/navigator.pri b/src/plugins/qmldesigner/components/navigator/navigator.pri index 75e5333d791..3ce8bc68776 100644 --- a/src/plugins/qmldesigner/components/navigator/navigator.pri +++ b/src/plugins/qmldesigner/components/navigator/navigator.pri @@ -8,3 +8,5 @@ HEADERS += navigatorview.h \ navigatortreemodel.h \ navigatorwidget.h \ navigatortreeview.h + +RESOURCES += navigator.qrc diff --git a/src/plugins/qmldesigner/components/navigator/navigator.qrc b/src/plugins/qmldesigner/components/navigator/navigator.qrc new file mode 100644 index 00000000000..4b911febf9a --- /dev/null +++ b/src/plugins/qmldesigner/components/navigator/navigator.qrc @@ -0,0 +1,8 @@ + + + arrowdown.png + arrowleft.png + arrowright.png + arrowup.png + + diff --git a/src/plugins/qmldesigner/components/navigator/navigatorview.cpp b/src/plugins/qmldesigner/components/navigator/navigatorview.cpp index 408d299aa9c..c15d04c2e70 100644 --- a/src/plugins/qmldesigner/components/navigator/navigatorview.cpp +++ b/src/plugins/qmldesigner/components/navigator/navigatorview.cpp @@ -46,12 +46,18 @@ namespace QmlDesigner { NavigatorView::NavigatorView(QObject* parent) : AbstractView(parent), m_blockSelectionChangedSignal(false), - m_widget(new NavigatorWidget), + m_widget(new NavigatorWidget(this)), m_treeModel(new NavigatorTreeModel(this)) { m_widget->setTreeModel(m_treeModel.data()); connect(treeWidget()->selectionModel(), SIGNAL(selectionChanged(QItemSelection,QItemSelection)), this, SLOT(changeSelection(QItemSelection,QItemSelection))); + + connect(m_widget.data(), SIGNAL(leftButtonClicked()), this, SLOT(leftButtonClicked())); + connect(m_widget.data(), SIGNAL(rightButtonClicked()), this, SLOT(rightButtonClicked())); + connect(m_widget.data(), SIGNAL(downButtonClicked()), this, SLOT(downButtonClicked())); + connect(m_widget.data(), SIGNAL(upButtonClicked()), this, SLOT(upButtonClicked())); + treeWidget()->setIndentation(treeWidget()->indentation() * 0.5); NameItemDelegate *idDelegate = new NameItemDelegate(this,m_treeModel.data()); @@ -80,7 +86,7 @@ NavigatorView::~NavigatorView() delete m_widget.data(); } -QWidget *NavigatorView::widget() +NavigatorWidget *NavigatorView::widget() { return m_widget.data(); } @@ -256,6 +262,75 @@ void NavigatorView::changeToComponent(const QModelIndex &index) } } +void NavigatorView::leftButtonClicked() +{ + if (selectedModelNodes().count() > 1) + return; //Semantics are unclear for multi selection. + + bool blocked = blockSelectionChangedSignal(true); + + foreach (const ModelNode &node, selectedModelNodes()) { + if (!node.isRootNode() && !node.parentProperty().parentModelNode().isRootNode()) + node.parentProperty().parentModelNode().parentProperty().reparentHere(node); + } + updateItemSelection(); + blockSelectionChangedSignal(blocked); +} + +void NavigatorView::rightButtonClicked() +{ + if (selectedModelNodes().count() > 1) + return; //Semantics are unclear for multi selection. + + bool blocked = blockSelectionChangedSignal(true); + foreach (const ModelNode &node, selectedModelNodes()) { + if (!node.isRootNode() && node.parentProperty().isNodeListProperty() && node.parentProperty().toNodeListProperty().count() > 1) { + int index = node.parentProperty().toNodeListProperty().indexOf(node); + index--; + if (index >= 0) { //for the first node the semantics are not clear enough. Wrapping would be irritating. + ModelNode newParent = node.parentProperty().toNodeListProperty().at(index); + newParent.nodeAbstractProperty(newParent.metaInfo().defaultPropertyName()).reparentHere(node); + } + } + } + updateItemSelection(); + blockSelectionChangedSignal(blocked); +} + +void NavigatorView::upButtonClicked() +{ + bool blocked = blockSelectionChangedSignal(true); + foreach (const ModelNode &node, selectedModelNodes()) { + if (!node.isRootNode() && node.parentProperty().isNodeListProperty()) { + int oldIndex = node.parentProperty().toNodeListProperty().indexOf(node); + int index = oldIndex; + index--; + if (index < 0) + index = node.parentProperty().toNodeListProperty().count() - 1; //wrap around + node.parentProperty().toNodeListProperty().slide(oldIndex, index); + } + } + updateItemSelection(); + blockSelectionChangedSignal(blocked); +} + +void NavigatorView::downButtonClicked() +{ + bool blocked = blockSelectionChangedSignal(true); + foreach (const ModelNode &node, selectedModelNodes()) { + if (!node.isRootNode() && node.parentProperty().isNodeListProperty()) { + int oldIndex = node.parentProperty().toNodeListProperty().indexOf(node); + int index = oldIndex; + index++; + if (index >= node.parentProperty().toNodeListProperty().count()) + index = 0; //wrap around + node.parentProperty().toNodeListProperty().slide(oldIndex, index); + } + } + updateItemSelection(); + blockSelectionChangedSignal(blocked); +} + void NavigatorView::changeSelection(const QItemSelection & /*newSelection*/, const QItemSelection &/*deselected*/) { if (m_blockSelectionChangedSignal) diff --git a/src/plugins/qmldesigner/components/navigator/navigatorview.h b/src/plugins/qmldesigner/components/navigator/navigatorview.h index c51524eac52..825eae0e679 100644 --- a/src/plugins/qmldesigner/components/navigator/navigatorview.h +++ b/src/plugins/qmldesigner/components/navigator/navigatorview.h @@ -59,7 +59,7 @@ public: NavigatorView(QObject* parent = 0); ~NavigatorView(); - QWidget *widget(); + NavigatorWidget *widget(); // AbstractView void modelAttached(Model *model); @@ -104,6 +104,11 @@ private slots: void updateItemSelection(); void changeToComponent(const QModelIndex &index); + void leftButtonClicked(); + void rightButtonClicked(); + void upButtonClicked(); + void downButtonClicked(); + protected: //functions QTreeView *treeWidget(); NavigatorTreeModel *treeModel(); diff --git a/src/plugins/qmldesigner/components/navigator/navigatorwidget.cpp b/src/plugins/qmldesigner/components/navigator/navigatorwidget.cpp index 59d6297c664..d81e1e741c5 100644 --- a/src/plugins/qmldesigner/components/navigator/navigatorwidget.cpp +++ b/src/plugins/qmldesigner/components/navigator/navigatorwidget.cpp @@ -36,15 +36,17 @@ #include #include "navigatorwidget.h" +#include "navigatorview.h" #include namespace QmlDesigner { -NavigatorWidget::NavigatorWidget(QWidget* parent) : - QFrame(parent), - m_treeView(new NavigatorTreeView) +NavigatorWidget::NavigatorWidget(NavigatorView *view) : + QFrame(), + m_treeView(new NavigatorTreeView), + m_navigatorView(view) { m_treeView->setDragEnabled(true); m_treeView->setAcceptDrops(true); @@ -79,4 +81,54 @@ void NavigatorWidget::setTreeModel(QAbstractItemModel* model) m_treeView->setModel(model); } +QList NavigatorWidget::createToolBarWidgets() +{ + QList buttons; + + buttons << new QToolButton(); + buttons.last()->setIcon(QIcon(":/navigator/icon/arrowleft.png")); + buttons.last()->setToolTip(tr("Become first sibling of parent (CTRL + Left)")); + buttons.last()->setShortcut(QKeySequence(Qt::Key_Left | Qt::CTRL)); + connect(buttons.last(), SIGNAL(clicked()), this, SIGNAL(leftButtonClicked())); + buttons << new QToolButton(); + buttons.last()->setIcon(QIcon(":/navigator/icon/arrowright.png")); + buttons.last()->setToolTip(tr("Become child of first silbing (CTRL + Right)")); + buttons.last()->setShortcut(QKeySequence(Qt::Key_Right | Qt::CTRL)); + connect(buttons.last(), SIGNAL(clicked()), this, SIGNAL(rightButtonClicked())); + + buttons << new QToolButton(); + buttons.last()->setIcon(QIcon(":/navigator/icon/arrowdown.png")); + buttons.last()->setToolTip(tr("Move down (CTRL + Down)")); + buttons.last()->setShortcut(QKeySequence(Qt::Key_Down | Qt::CTRL)); + connect(buttons.last(), SIGNAL(clicked()), this, SIGNAL(downButtonClicked())); + + buttons << new QToolButton(); + buttons.last()->setIcon(QIcon(":/navigator/icon/arrowup.png")); + buttons.last()->setToolTip(tr("Move up (CTRL + Up)")); + buttons.last()->setShortcut(QKeySequence(Qt::Key_Up | Qt::CTRL)); + connect(buttons.last(), SIGNAL(clicked()), this, SIGNAL(upButtonClicked())); + + return buttons; +} + +QString NavigatorWidget::contextHelpId() const +{ + if (!navigatorView()) + return QString(); + + QList nodes = navigatorView()->selectedModelNodes(); + QString helpId; + if (!nodes.isEmpty()) { + helpId = nodes.first().type(); + helpId.replace("QtQuick", "QML"); + } + + return helpId; +} + +NavigatorView *NavigatorWidget::navigatorView() const +{ + return m_navigatorView.data(); +} + } diff --git a/src/plugins/qmldesigner/components/navigator/navigatorwidget.h b/src/plugins/qmldesigner/components/navigator/navigatorwidget.h index 2d78b6382f8..c09d2c6de1a 100644 --- a/src/plugins/qmldesigner/components/navigator/navigatorwidget.h +++ b/src/plugins/qmldesigner/components/navigator/navigatorwidget.h @@ -35,25 +35,37 @@ #include #include +#include #include "navigatortreeview.h" namespace QmlDesigner { class Model; +class NavigatorView; class NavigatorWidget: public QFrame { Q_OBJECT public: - NavigatorWidget(QWidget* parent = 0); + NavigatorWidget(NavigatorView *view); virtual ~NavigatorWidget(); void setTreeModel(QAbstractItemModel *model); QTreeView *treeView() const { return m_treeView; } + QList createToolBarWidgets(); + QString contextHelpId() const; + +signals: + void leftButtonClicked(); + void rightButtonClicked(); + void upButtonClicked(); + void downButtonClicked(); private: + NavigatorView *navigatorView() const; NavigatorTreeView *m_treeView; + QWeakPointer m_navigatorView; }; } diff --git a/src/plugins/qmldesigner/designmodewidget.cpp b/src/plugins/qmldesigner/designmodewidget.cpp index 7bd0faefa56..bbaa948d1bc 100644 --- a/src/plugins/qmldesigner/designmodewidget.cpp +++ b/src/plugins/qmldesigner/designmodewidget.cpp @@ -153,6 +153,27 @@ QList ItemLibrarySideBarItem::createToolBarWidgets() return qobject_cast(widget())->createToolBarWidgets(); } +class NavigatorSideBarItem : public Core::SideBarItem +{ +public: + explicit NavigatorSideBarItem(NavigatorWidget *widget, const QString &id); + virtual ~NavigatorSideBarItem(); + + virtual QList createToolBarWidgets(); +}; + +NavigatorSideBarItem::NavigatorSideBarItem(NavigatorWidget *widget, const QString &id) : Core::SideBarItem(widget, id) {} + +NavigatorSideBarItem::~NavigatorSideBarItem() +{ + +} + +QList NavigatorSideBarItem::createToolBarWidgets() +{ + return qobject_cast(widget())->createToolBarWidgets(); +} + void DocumentWarningWidget::goToError() { m_designModeWidget->textEditor()->gotoLine(m_error.line(), m_error.column() - 1);