diff -r 1a77c6156db7 -r 988b6563d47d src/mainwindow.cpp --- a/src/mainwindow.cpp Tue Jan 01 22:30:10 2019 +0200 +++ b/src/mainwindow.cpp Thu Feb 07 13:45:07 2019 +0200 @@ -188,6 +188,9 @@ delete m_grid; delete &ui; + for (QItemSelectionModel* model : m_selectionModels) + delete model; + for (Toolset* toolset : m_toolsets) delete toolset; } @@ -840,7 +843,7 @@ m_renderers[document].append(canvas); QMdiSubWindow* const subWindow = ui.viewport->addSubWindow(canvas); m_subWindows[canvas] = subWindow; - connect(canvas, &QObject::destroyed, this, &MainWindow::canvasClosed); + connect(canvas, &gl::Renderer::closed, this, &MainWindow::canvasClosed); ui.viewport->setActiveSubWindow(subWindow); return canvas; } @@ -873,6 +876,7 @@ { Canvas* canvas = qobject_cast(sender()); + Q_ASSERT(canvas != nullptr); if (canvas != nullptr) { LDDocument* const document = canvas->document(); @@ -890,6 +894,10 @@ Canvas* canvas = getRendererForDocument(document); updateDocumentList(); connect(document, &LDDocument::windingChanged, canvas, &Canvas::fullUpdate); + + QItemSelectionModel* selection = new QItemSelectionModel; + m_selectionModels[document] = selection; + canvas->setSelectionModel(selection); } } @@ -927,19 +935,8 @@ canvas->fullUpdate(); QItemSelectionModel* selection = m_selectionModels.value(document); - - if (selection == nullptr) - { - selection = new QItemSelectionModel; - m_selectionModels[document] = selection; + ui.objectList->setSelectionModel(selection); - for (Canvas* canvas : m_renderers[document]) - canvas->setSelectionModel(m_selectionModels[document]); - } - else - { - ui.objectList->setSelectionModel(selection); - } } }