QUndoStack across 2 cpps? QTreeView and QGraphicsScene

JonB

@StudentScripter said in QUndoStack across 2 cpps? QTreeView and QGraphicsScene:

but in reality there is no real checkbox but only the paint event of the delegate

You have to connect that to a model. Else you have nowhere to store the state for things which just happen in the UI. Look at how, say, checkbox state is linked to Qt::CheckState QStandardItem::checkState() const or QVariant QAbstractItemModel::data(const QModelIndex &index, int role = Qt::DisplayRole) const with enum Qt::ItemDataRole set to Qt::CheckStateRole. And the delegate deals with this. And you can have a common model which has some aspects reflected/not reflected in a tree view and other aspects in a graphics scene.

I have no idea what "a singleton class to exchange the model" is about, but it's a detail compared to the overall concept.

StudentScripter

@JonB Well what do you mean with common model at all?

I have a QStandardItemModel and i do already set the checkstate. However the problem i have is that the checkstate has to be set when clicking at the standard item at a certain position. Im currently doing this in the qtreeviews mousepressevent:

void ViewLayerList::mousePressEvent(QMouseEvent *event)
{  


    //Bekommt den Index welcher gerade unter der Mouse Position ist
    indexAtMouse = indexAt(event->position().toPoint());

    
    // Zugriff auf das SelectionModel, um ausgewählte Items zu erhalten
    QItemSelectionModel *selectionModel = this->selectionModel();

    // Überprüfen, ob ein SelectionModel vorhanden ist
    if (selectionModel) {
        selectedIndexes = selectionModel->selectedIndexes();
    }
    
    int columnWidth = this->columnWidth(0);
    int ClickPosX = event->position().x();


   
    if(ClickPosX >= columnWidth-40 && ClickPosX <= columnWidth -15){
        if(selectedIndexes.count()>1){
            for (const QModelIndex &index : selectedIndexes) {
                bool value = index.data(Qt::CheckStateRole).toBool();
                model->setData(index, !value, Qt::CheckStateRole);
           } 
      }
}

}

SGaist

That's the role of QStyledItemDelegate::editorEvent.

StudentScripter

@SGaist But the editor event is only triggert when the delegate is activated/created right? When the editor isn't currently active this wont work. I want it to set the checked even when the editor at this index isn't created/open.

JonB

@StudentScripter said in QUndoStack across 2 cpps? QTreeView and QGraphicsScene:

I want it to set the checked even when the editor at this index isn't created/open.

I don't understand this. Why can't you store that information in a model and when the delegate is shown the state is picked up from there and displayed then? Not store the state in a delegate instance which does not currently exist.

But I don't understand your situation, I don't know where a graphics scene comes into it, I don't know what it has to do with the title of QUndoStack across multiple .cpp files. And I doubt I will :) So will try to leave it to you/others who understand better.

SGaist

@StudentScripter No, it will also be called for other interactions. See here.

StudentScripter

@JonB Haha no problem. Your answers where quite helpful still. And i guess i will adapt a few things the way you suggested. :D Will try to give the scene access to the model.

@SGaist So even when the delegate isn't active in the standarditem clicked on it will still receive those clicks?

SGaist

@StudentScripter the delegate is always active once you set it. Or did you mean the editor ? If so, then yes, the events will be sent to the delegate.

StudentScripter

@SGaist Yeah i probably mean the editor cause my delegate is visible trough its paint event even if the editor is closed, right?

Now i click on one position of my delegate (the editor is still closed/or should stay closed) but i still want to change the Qt::CheckStateRole of that indexes delegate.

Is this understandable? Thanks for your help and time too. Can't say that often enough.

Also could i just add a setter function in my graphicsscene and set the model from the treeview? As said i have access from the treeview to the model object and the graphicsscene. Like this:

#include "ViewLayerList.h"
#include <QHBoxLayout>
#include <QCheckBox>
#include <QLabel>
#include "ViewLayerDropIndicatorStyle.h"
#include <QMouseEvent>
#include "resizablepixmapitem.h"
#include <QHeaderView>
#include <QPushButton>

#include <QTimer>
#include <QApplication>
#include <QDrag>
#include <QCursor>
#include <QMimeData>

#include <QApplication>
#include <QFile>
#include <QDataStream>

#include <QJsonDocument>
#include <QJsonObject>
#include <QJsonArray>

#include <QUndoCommand>
#include "CommandManager.h"





ViewLayerList::ViewLayerList(CustomGraphicsScene *scene, QWidget *parent)
    : QTreeView{parent}, scene_durchgereicht(scene)
{

                





 setStyle(new ViewLayerDropIndicatorStyle(style()));


 // Ändern Sie den Abstand zwischen den Items und der vertikalen Scrollbar
    setViewportMargins(0, 0, 50, 0); // Passen Sie den rechten Rand (20) an Ihre Anforderungen an
    
//Versteckt die sinnlose Kopfzeile
    setHeaderHidden(true);
    setRootIsDecorated(true);
    setMouseTracking(true);







mydelegate = new ViewLayerItemDelegate(this);



model = new ViewLayerStandartItemModel(0,1,this);


scene_durchgereicht->setModel(model); //Dieses exakte model
}

StudentScripter

bumping this

SGaist

@StudentScripter your tree view has no reason to be doing that setup with the scene. They share a model, they don't need to know about each other.

As for the delegate, your goal is clear, and again, the editorEvent is what you want to use to implement the behavior you want.

StudentScripter

@SGaist Well but as i understand it with

model = new ViewLayerStandartItemModel(0,1,this);

im creating a new instance of this class, a new object. So in order to have access to the same model i have to pass the object i create in the treeview trough to the scene.
Is this a mistake? How should i do it instead?

SGaist

@StudentScripter you should create the model in the class that manages the ViewLayerList object and set it on both objects there. In the absolute, with the code you show, ViewLayerList is not even needed unless there are some additional function you are implementing. If not, then just use QTreeView directly and configure it in the class managing it.

StudentScripter

@SGaist Yes im implementing additional functions in the qtreeview/ViewLayerList, thats why i subclassed it. Im creating the model object directly in ViewLayerLists constructor, cause i use that model mostly in this cpp.

So i guess my approach with giving the model from there to the scene is okish?

SGaist

@StudentScripter no, it makes that class know another one just for the purpose of sharing the model. That's tight coupling for the bad reasons.

JonB

@JonB said in QUndoStack across 2 cpps? QTreeView and QGraphicsScene:

The model code can be kept in its own/class/module/file, that is included into both the tree view & graphics scene classes, they should not need to see each other directly. This would be my inclination.

StudentScripter

@JonB Ok and from their i add an setter in the qtreeview class and the viewlayerlist to retrieve the model.

JonB

@StudentScripter Yes, a suitable public getter & setter for the model called from each of the QTreeView and QGraphicsScene modules.

SGaist

@StudentScripter QTreeView already has a setter for the model. You just need to add one to your QGraphicsScene subclass.

StudentScripter

@SGaist Yep done everything and works fine. :D