Gui: Apply clang-format

2024-11-09 00:29:22 +01:00
parent 6ac794b8e3
commit a9694be87b
1 changed files with 242 additions and 173 deletions
--- a/src/Gui/TaskElementColors.cpp
+++ b/src/Gui/TaskElementColors.cpp
@@ -23,9 +23,9 @@
 #include "PreCompiled.h"

 #ifndef _PreComp_
-# include <boost/algorithm/string/predicate.hpp>
-# include <QColorDialog>
-# include <sstream>
+#include <boost/algorithm/string/predicate.hpp>
+#include <QColorDialog>
+#include <sstream>
 #endif

 #include <App/ElementNamingUtils.h>
@@ -54,10 +54,10 @@ class ElementColors::Private: public Gui::SelectionGate
 public:
    using Connection = boost::signals2::connection;
    std::unique_ptr<Ui_TaskElementColors> ui;
-    ViewProviderDocumentObject *vp;
-    ViewProviderDocumentObject *vpParent;
-    Document *vpDoc;
-    std::map<std::string,QListWidgetItem*> elements;
+    ViewProviderDocumentObject* vp;
+    ViewProviderDocumentObject* vpParent;
+    Document* vpDoc;
+    std::map<std::string, QListWidgetItem*> elements;
    std::vector<QListWidgetItem*> items;
    std::string hiddenSub;
    Connection connectDelDoc;
@@ -72,22 +72,24 @@ public:
    std::string editSub;
    std::string editElement;

-    explicit Private(ViewProviderDocumentObject* vp, const char *element="")
-        : ui(new Ui_TaskElementColors()), vp(vp),editElement(element)
+    explicit Private(ViewProviderDocumentObject* vp, const char* element = "")
+        : ui(new Ui_TaskElementColors())
+        , vp(vp)
+        , editElement(element)
    {
        vpDoc = vp->getDocument();
        vpParent = vp;
        auto doc = Application::Instance->editDocument();
-        if(doc) {
-            auto editVp = doc->getInEdit(&vpParent,&editSub);
-            if(editVp == vp) {
+        if (doc) {
+            auto editVp = doc->getInEdit(&vpParent, &editSub);
+            if (editVp == vp) {
                auto obj = vpParent->getObject();
                editDoc = obj->getDocument()->getName();
                editObj = obj->getNameInDocument();
                editSub = Data::noElementName(editSub.c_str());
            }
        }
-        if(editDoc.empty()) {
+        if (editDoc.empty()) {
            vpParent = vp;
            editDoc = vp->getObject()->getDocument()->getName();
            editObj = vp->getObject()->getNameInDocument();
@@ -97,10 +99,11 @@ public:
        busy = false;
        touched = false;
        int w = QApplication::style()->standardPixmap(QStyle::SP_DirClosedIcon).width();
-        px = QPixmap(w,w);
+        px = QPixmap(w, w);
    }

-    ~Private() override {
+    ~Private() override
+    {
        try {
            vpParent->OnTopWhenSelected.setValue(onTopMode);
        }
@@ -109,94 +112,111 @@ public:
        }
    }

-    bool allow(App::Document *doc, App::DocumentObject *obj, const char *subname) override {
-        if(editDoc!=doc->getName() ||
-           editObj!=obj->getNameInDocument() ||
-           !boost::starts_with(subname,editSub))
+    bool allow(App::Document* doc, App::DocumentObject* obj, const char* subname) override
+    {
+        if (editDoc != doc->getName() || editObj != obj->getNameInDocument()
+            || !boost::starts_with(subname, editSub)) {
            return false;
-        if(editElement.empty())
+        }
+        if (editElement.empty()) {
            return true;
-        const char *dot = strrchr(subname,'.');
-        if(!dot)
+        }
+        const char* dot = strrchr(subname, '.');
+        if (!dot) {
            dot = subname;
-        else
+        }
+        else {
            ++dot;
-        return *dot==0 || boost::starts_with(dot,editElement);
+        }
+        return *dot == 0 || boost::starts_with(dot, editElement);
    }

-    void populate() {
-        int i=0;
-        for(auto &v : vp->getElementColors())
-            addItem(i++,v.first.c_str());
+    void populate()
+    {
+        int i = 0;
+        for (auto& v : vp->getElementColors()) {
+            addItem(i++, v.first.c_str());
+        }
        apply();
    }

-    void addItem(int i,const char *sub, bool push=false) {
+    void addItem(int i, const char* sub, bool push = false)
+    {
        auto itE = elements.find(sub);
-        if(i<0 && itE!=elements.end()) {
-            if(push && !ViewProvider::hasHiddenMarker(sub))
+        if (i < 0 && itE != elements.end()) {
+            if (push && !ViewProvider::hasHiddenMarker(sub)) {
                items.push_back(itE->second);
+            }
            return;
        }

-        const char *marker = ViewProvider::hasHiddenMarker(sub);
-        if(marker) {
+        const char* marker = ViewProvider::hasHiddenMarker(sub);
+        if (marker) {
            auto icon = BitmapFactory().pixmap("Invisible");
-            auto item = new QListWidgetItem(icon,
-                    QString::fromLatin1(std::string(sub,marker-sub).c_str()), ui->elementList);
-            item->setData(Qt::UserRole,QColor());
-            item->setData(Qt::UserRole+1,QString::fromLatin1(sub));
-            elements.emplace(sub,item);
+            auto item =
+                new QListWidgetItem(icon,
+                                    QString::fromLatin1(std::string(sub, marker - sub).c_str()),
+                                    ui->elementList);
+            item->setData(Qt::UserRole, QColor());
+            item->setData(Qt::UserRole + 1, QString::fromLatin1(sub));
+            elements.emplace(sub, item);
            return;
        }

-        for(auto &v : vp->getElementColors(sub)) {
+        for (auto& v : vp->getElementColors(sub)) {
            auto it = elements.find(v.first.c_str());
-            if(it!=elements.end()) {
-                if(push)
+            if (it != elements.end()) {
+                if (push) {
                    items.push_back(it->second);
+                }
                continue;
            }
            auto color = v.second;
            QColor c;
-            c.setRgbF(color.r,color.g,color.b,1.0-color.a);
+            c.setRgbF(color.r, color.g, color.b, 1.0 - color.a);
            px.fill(c);
-            auto item = new QListWidgetItem(QIcon(px),
-                    QString::fromLatin1(Data::oldElementName(v.first.c_str()).c_str()),
-                    ui->elementList);
-            item->setData(Qt::UserRole,c);
-            item->setData(Qt::UserRole+1,QString::fromLatin1(v.first.c_str()));
-            if(push)
+            auto item = new QListWidgetItem(
+                QIcon(px),
+                QString::fromLatin1(Data::oldElementName(v.first.c_str()).c_str()),
+                ui->elementList);
+            item->setData(Qt::UserRole, c);
+            item->setData(Qt::UserRole + 1, QString::fromLatin1(v.first.c_str()));
+            if (push) {
                items.push_back(item);
-            elements.emplace(v.first,item);
+            }
+            elements.emplace(v.first, item);
        }
    }

-    void apply() {
-        std::map<std::string,App::Color> info;
+    void apply()
+    {
+        std::map<std::string, App::Color> info;
        int count = ui->elementList->count();
-        for(int i=0;i<count;++i) {
+        for (int i = 0; i < count; ++i) {
            auto item = ui->elementList->item(i);
            auto color = item->data(Qt::UserRole).value<QColor>();
-            std::string sub = qPrintable(item->data(Qt::UserRole+1).value<QString>());
-            info.emplace(qPrintable(item->data(Qt::UserRole+1).value<QString>()),
-                    App::Color(color.redF(), color.greenF(), color.blueF(), color.alphaF()));
+            info.emplace(
+                qPrintable(item->data(Qt::UserRole + 1).value<QString>()),
+                App::Color(color.redF(), color.greenF(), color.blueF(), color.alphaF()));
        }
-        if(!App::GetApplication().getActiveTransaction())
+        if (!App::GetApplication().getActiveTransaction()) {
            App::GetApplication().setActiveTransaction("Set colors");
+        }
        vp->setElementColors(info);
        touched = true;
        Selection().clearSelection();
    }

-    void reset() {
+    void reset()
+    {
        touched = false;
        App::GetApplication().closeActiveTransaction(true);
        Selection().clearSelection();
    }

-    void accept() {
-        if(touched && ui->recompute->isChecked()) {
+    void accept()
+    {
+        if (touched && ui->recompute->isChecked()) {
            auto obj = vp->getObject();
            obj->touch();
            obj->getDocument()->recompute(obj->getInListRecursive());
@@ -205,8 +225,9 @@ public:
        App::GetApplication().closeActiveTransaction();
    }

-    void removeAll() {
-        if(!elements.empty()) {
+    void removeAll()
+    {
+        if (!elements.empty()) {
            hiddenSub.clear();
            ui->elementList->clear();
            elements.clear();
@@ -214,93 +235,111 @@ public:
        }
    }

-    void removeItems() {
+    void removeItems()
+    {
        const auto items = ui->elementList->selectedItems();
-        for(auto item : items) {
-            std::string sub = qPrintable(item->data(Qt::UserRole+1).value<QString>());
-            if(sub == hiddenSub)
+        for (auto item : items) {
+            std::string sub = qPrintable(item->data(Qt::UserRole + 1).value<QString>());
+            if (sub == hiddenSub) {
                hiddenSub.clear();
+            }
            elements.erase(sub);
            delete item;
        }
        apply();
    }

-    void editItem(QWidget *parent, QListWidgetItem *item) {
-        std::string sub = qPrintable(item->data(Qt::UserRole+1).value<QString>());
-        if(ViewProvider::hasHiddenMarker(sub.c_str()))
+    void editItem(QWidget* parent, QListWidgetItem* item)
+    {
+        std::string sub = qPrintable(item->data(Qt::UserRole + 1).value<QString>());
+        if (ViewProvider::hasHiddenMarker(sub.c_str())) {
            return;
+        }
        auto color = item->data(Qt::UserRole).value<QColor>();
        QColorDialog cd(color, parent);
        cd.setOption(QColorDialog::ShowAlphaChannel);
-        if (DialogOptions::dontUseNativeColorDialog())
+        if (DialogOptions::dontUseNativeColorDialog()) {
            cd.setOption(QColorDialog::DontUseNativeDialog);
-        if (cd.exec()!=QDialog::Accepted || color==cd.selectedColor())
+        }
+        if (cd.exec() != QDialog::Accepted || color == cd.selectedColor()) {
            return;
+        }
        color = cd.selectedColor();
-        item->setData(Qt::UserRole,color);
+        item->setData(Qt::UserRole, color);
        px.fill(color);
        item->setIcon(QIcon(px));
        apply();
    }

-    void onSelectionChanged(const SelectionChanges &msg) {
+    void onSelectionChanged(const SelectionChanges& msg)
+    {
        // no object selected in the combobox or no sub-element was selected
-        if (busy)
+        if (busy) {
            return;
+        }
        busy = true;
-        switch(msg.Type) {
-        case SelectionChanges::ClrSelection:
-            ui->elementList->clearSelection();
-            break;
-        case SelectionChanges::AddSelection:
-        case SelectionChanges::RmvSelection:
-            if(msg.pDocName && msg.pObjectName && msg.pSubName && msg.pSubName[0]) {
-                if(editDoc == msg.pDocName &&
-                   editObj == msg.pObjectName &&
-                   boost::starts_with(msg.pSubName,editSub))
-                {
-                    const auto items = ui->elementList->findItems(QString::fromLatin1(msg.pSubName-editSub.size()), Qt::MatchExactly);
-                    for(auto item : items)
-                        item->setSelected(msg.Type==SelectionChanges::AddSelection);
+        switch (msg.Type) {
+            case SelectionChanges::ClrSelection:
+                ui->elementList->clearSelection();
+                break;
+            case SelectionChanges::AddSelection:
+            case SelectionChanges::RmvSelection:
+                if (msg.pDocName && msg.pObjectName && msg.pSubName && msg.pSubName[0]) {
+                    if (editDoc == msg.pDocName && editObj == msg.pObjectName
+                        && boost::starts_with(msg.pSubName, editSub)) {
+                        const auto items = ui->elementList->findItems(
+                            QString::fromLatin1(msg.pSubName - editSub.size()),
+                            Qt::MatchExactly);
+                        for (auto item : items) {
+                            item->setSelected(msg.Type == SelectionChanges::AddSelection);
+                        }
+                    }
                }
-            }
-        default:
-            break;
+            default:
+                break;
        }
        busy = false;
    }

-    void onSelectionChanged() {
-        if(busy)
+    void onSelectionChanged()
+    {
+        if (busy) {
            return;
+        }
        busy = true;
-        std::map<std::string,int> sels;
-        for(auto &sel : Selection().getSelectionEx(
-                    editDoc.c_str(),App::DocumentObject::getClassTypeId(), ResolveMode::NoResolve))
-        {
-            if(sel.getFeatName()!=editObj) continue;
-            for(auto &sub : sel.getSubNames()) {
-                if(boost::starts_with(sub,editSub))
-                    sels[sub.c_str()+editSub.size()] = 1;
+        std::map<std::string, int> sels;
+        for (auto& sel : Selection().getSelectionEx(editDoc.c_str(),
+                                                    App::DocumentObject::getClassTypeId(),
+                                                    ResolveMode::NoResolve)) {
+            if (sel.getFeatName() != editObj) {
+                continue;
+            }
+            for (auto& sub : sel.getSubNames()) {
+                if (boost::starts_with(sub, editSub)) {
+                    sels[sub.c_str() + editSub.size()] = 1;
+                }
            }
            break;
        }
        const auto items = ui->elementList->selectedItems();
-        for(auto item : items) {
-            std::string name(qPrintable(item->data(Qt::UserRole+1).value<QString>()));
-            if(ViewProvider::hasHiddenMarker(name.c_str()))
+        for (auto item : items) {
+            std::string name(qPrintable(item->data(Qt::UserRole + 1).value<QString>()));
+            if (ViewProvider::hasHiddenMarker(name.c_str())) {
                continue;
-            auto &v = sels[name];
-            if(!v)
+            }
+            auto& v = sels[name];
+            if (!v) {
                Selection().addSelection(editDoc.c_str(),
-                        editObj.c_str(), (editSub+name).c_str());
+                                         editObj.c_str(),
+                                         (editSub + name).c_str());
+            }
            v = 2;
        }
-        for(auto &v : sels) {
-            if(v.second!=2) {
+        for (auto& v : sels) {
+            if (v.second != 2) {
                Selection().rmvSelection(editDoc.c_str(),
-                        editObj.c_str(), (editSub+v.first).c_str());
+                                         editObj.c_str(),
+                                         (editSub + v.first).c_str());
            }
        }
        busy = false;
@@ -310,34 +349,36 @@ public:
 /* TRANSLATOR Gui::TaskElementColors */

 ElementColors::ElementColors(ViewProviderDocumentObject* vp, bool noHide)
-    :d(new Private(vp))
+    : d(new Private(vp))
 {
    d->ui->setupUi(this);
    setupConnections();

    d->ui->objectLabel->setText(QString::fromUtf8(vp->getObject()->Label.getValue()));
-    d->ui->elementList->setMouseTracking(true); // needed for itemEntered() to work
+    d->ui->elementList->setMouseTracking(true);  // needed for itemEntered() to work

    if (noHide) {
        d->ui->hideSelection->setVisible(false);
    }

-    ParameterGrp::handle hPart = App::GetApplication().GetParameterGroupByPath
-        ("User parameter:BaseApp/Preferences/View");
-    d->ui->recompute->setChecked(hPart->GetBool("ColorRecompute",true));
-    d->ui->onTop->setChecked(hPart->GetBool("ColorOnTop",true));
+    ParameterGrp::handle hPart =
+        App::GetApplication().GetParameterGroupByPath("User parameter:BaseApp/Preferences/View");
+    d->ui->recompute->setChecked(hPart->GetBool("ColorRecompute", true));
+    d->ui->onTop->setChecked(hPart->GetBool("ColorOnTop", true));
    if (d->ui->onTop->isChecked()) {
        d->vpParent->OnTopWhenSelected.setValue(3);
    }

    Selection().addSelectionGate(d, ResolveMode::NoResolve);

-    //NOLINTBEGIN
+    // NOLINTBEGIN
+    // clang-format off
    d->connectDelDoc = Application::Instance->signalDeleteDocument.connect(std::bind
        (&ElementColors::slotDeleteDocument, this, sp::_1));
    d->connectDelObj = Application::Instance->signalDeletedObject.connect(std::bind
        (&ElementColors::slotDeleteObject, this, sp::_1));
-    //NOLINTEND
+    // clang-format on
+    // NOLINTEND

    d->populate();
 }
@@ -351,6 +392,7 @@ ElementColors::~ElementColors()

 void ElementColors::setupConnections()
 {
+    // clang-format off
    connect(d->ui->removeSelection, &QPushButton::clicked,
            this, &ElementColors::onRemoveSelectionClicked);
    connect(d->ui->addSelection, &QPushButton::clicked,
@@ -371,33 +413,36 @@ void ElementColors::setupConnections()
            this, &ElementColors::onHideSelectionClicked);
    connect(d->ui->boxSelect, &QPushButton::clicked,
            this, &ElementColors::onBoxSelectClicked);
+    // clang-format on
 }

 void ElementColors::onRecomputeClicked(bool checked)
 {
-    ParameterGrp::handle hPart = App::GetApplication().GetParameterGroupByPath
-        ("User parameter:BaseApp/Preferences/View");
-    hPart->SetBool("ColorRecompute",checked);
+    ParameterGrp::handle hPart =
+        App::GetApplication().GetParameterGroupByPath("User parameter:BaseApp/Preferences/View");
+    hPart->SetBool("ColorRecompute", checked);
 }

 void ElementColors::onTopClicked(bool checked)
 {
-    ParameterGrp::handle hPart = App::GetApplication().GetParameterGroupByPath
-        ("User parameter:BaseApp/Preferences/View");
-    hPart->SetBool("ColorOnTop",checked);
-    d->vpParent->OnTopWhenSelected.setValue(checked?3:d->onTopMode);
+    ParameterGrp::handle hPart =
+        App::GetApplication().GetParameterGroupByPath("User parameter:BaseApp/Preferences/View");
+    hPart->SetBool("ColorOnTop", checked);
+    d->vpParent->OnTopWhenSelected.setValue(checked ? 3 : d->onTopMode);
 }

 void ElementColors::slotDeleteDocument(const Document& Doc)
 {
-    if (d->vpDoc==&Doc || d->editDoc==Doc.getDocument()->getName())
+    if (d->vpDoc == &Doc || d->editDoc == Doc.getDocument()->getName()) {
        Control().closeDialog();
+    }
 }

 void ElementColors::slotDeleteObject(const ViewProvider& obj)
 {
-    if (d->vp==&obj)
+    if (d->vp == &obj) {
        Control().closeDialog();
+    }
 }

 void ElementColors::onRemoveSelectionClicked()
@@ -408,22 +453,27 @@ void ElementColors::onRemoveSelectionClicked()
 void ElementColors::onBoxSelectClicked()
 {
    auto cmd = Application::Instance->commandManager().getCommandByName("Std_BoxElementSelection");
-    if(cmd)
+    if (cmd) {
        cmd->invoke(0);
+    }
 }

-void ElementColors::onHideSelectionClicked() {
-    auto sels = Selection().getSelectionEx(d->editDoc.c_str(), App::DocumentObject::getClassTypeId(), ResolveMode::NoResolve);
-    for(auto &sel : sels) {
-        if(d->editObj!=sel.getFeatName())
+void ElementColors::onHideSelectionClicked()
+{
+    auto sels = Selection().getSelectionEx(d->editDoc.c_str(),
+                                           App::DocumentObject::getClassTypeId(),
+                                           ResolveMode::NoResolve);
+    for (auto& sel : sels) {
+        if (d->editObj != sel.getFeatName()) {
            continue;
-        const auto &subs = sel.getSubNames();
-        if(!subs.empty()) {
-            for(auto &sub : subs) {
-                if(boost::starts_with(sub,d->editSub)) {
-                    auto name = Data::noElementName(sub.c_str()+d->editSub.size());
+        }
+        const auto& subs = sel.getSubNames();
+        if (!subs.empty()) {
+            for (auto& sub : subs) {
+                if (boost::starts_with(sub, d->editSub)) {
+                    auto name = Data::noElementName(sub.c_str() + d->editSub.size());
                    name += ViewProvider::hiddenMarker();
-                    d->addItem(-1,name.c_str());
+                    d->addItem(-1, name.c_str());
                }
            }
            d->apply();
@@ -434,37 +484,45 @@ void ElementColors::onHideSelectionClicked() {

 void ElementColors::onAddSelectionClicked()
 {
-    auto sels = Selection().getSelectionEx(d->editDoc.c_str(), App::DocumentObject::getClassTypeId(), ResolveMode::NoResolve);
+    auto sels = Selection().getSelectionEx(d->editDoc.c_str(),
+                                           App::DocumentObject::getClassTypeId(),
+                                           ResolveMode::NoResolve);
    d->items.clear();
-    if(sels.empty())
-        d->addItem(-1,"Face",true);
+    if (sels.empty()) {
+        d->addItem(-1, "Face", true);
+    }
    else {
-        for(auto &sel : sels) {
-            if(d->editObj!=sel.getFeatName())
+        for (auto& sel : sels) {
+            if (d->editObj != sel.getFeatName()) {
                continue;
-            const auto &subs = sel.getSubNames();
-            if(subs.empty()) {
-                d->addItem(-1,"Face",true);
+            }
+            const auto& subs = sel.getSubNames();
+            if (subs.empty()) {
+                d->addItem(-1, "Face", true);
                break;
            }
-            for(auto &sub : subs) {
-                if(boost::starts_with(sub,d->editSub))
-                    d->addItem(-1,sub.c_str()+d->editSub.size(),true);
+            for (auto& sub : subs) {
+                if (boost::starts_with(sub, d->editSub)) {
+                    d->addItem(-1, sub.c_str() + d->editSub.size(), true);
+                }
            }
            break;
        }
    }
-    if(!d->items.empty()) {
+
+    if (!d->items.empty()) {
        auto color = d->items.front()->data(Qt::UserRole).value<QColor>();
        QColorDialog cd(color, this);
        cd.setOption(QColorDialog::ShowAlphaChannel);
-        if (DialogOptions::dontUseNativeColorDialog())
+        if (DialogOptions::dontUseNativeColorDialog()) {
            cd.setOption(QColorDialog::DontUseNativeDialog);
-        if (cd.exec()!=QDialog::Accepted)
+        }
+        if (cd.exec() != QDialog::Accepted) {
            return;
+        }
        color = cd.selectedColor();
-        for(auto item : d->items) {
-            item->setData(Qt::UserRole,color);
+        for (auto item : d->items) {
+            item->setData(Qt::UserRole, color);
            d->px.fill(color);
            item->setIcon(QIcon(d->px));
        }
@@ -491,7 +549,7 @@ bool ElementColors::reject()
    return true;
 }

-void ElementColors::changeEvent(QEvent *e)
+void ElementColors::changeEvent(QEvent* e)
 {
    QWidget::changeEvent(e);
    if (e->type() == QEvent::LanguageChange) {
@@ -499,33 +557,44 @@ void ElementColors::changeEvent(QEvent *e)
    }
 }

-void ElementColors::leaveEvent(QEvent *e) {
+void ElementColors::leaveEvent(QEvent* e)
+{
    QWidget::leaveEvent(e);
    Selection().rmvPreselect();
-    if(!d->hiddenSub.empty()) {
-        d->vp->partialRender({d->hiddenSub},false);
+
+    if (!d->hiddenSub.empty()) {
+        d->vp->partialRender({d->hiddenSub}, false);
        d->hiddenSub.clear();
    }
 }

-void ElementColors::onElementListItemEntered(QListWidgetItem *item) {
-    std::string name(qPrintable(item->data(Qt::UserRole+1).value<QString>()));
-    if(!d->hiddenSub.empty()) {
-        d->vp->partialRender({d->hiddenSub},false);
+void ElementColors::onElementListItemEntered(QListWidgetItem* item)
+{
+    std::string name(qPrintable(item->data(Qt::UserRole + 1).value<QString>()));
+    if (!d->hiddenSub.empty()) {
+        d->vp->partialRender({d->hiddenSub}, false);
        d->hiddenSub.clear();
    }
-    if(ViewProvider::hasHiddenMarker(name.c_str())) {
+
+    if (ViewProvider::hasHiddenMarker(name.c_str())) {
        d->hiddenSub = name;
-        d->vp->partialRender({name},true);
-        name.resize(name.size()-ViewProvider::hiddenMarker().size());
+        d->vp->partialRender({name}, true);
+        name.resize(name.size() - ViewProvider::hiddenMarker().size());
    }
+
    Selection().setPreselect(d->editDoc.c_str(),
-            d->editObj.c_str(), (d->editSub+name).c_str(),0,0,0,
-            d->ui->onTop->isChecked() ? Gui::SelectionChanges::MsgSource::TreeView
-                                      : Gui::SelectionChanges::MsgSource::Internal);
+                             d->editObj.c_str(),
+                             (d->editSub + name).c_str(),
+                             0,
+                             0,
+                             0,
+                             d->ui->onTop->isChecked()
+                                 ? Gui::SelectionChanges::MsgSource::TreeView
+                                 : Gui::SelectionChanges::MsgSource::Internal);
 }

-void ElementColors::onElementListItemSelectionChanged() {
+void ElementColors::onElementListItemSelectionChanged()
+{
    d->onSelectionChanged();
 }

@@ -534,23 +603,23 @@ void ElementColors::onSelectionChanged(const SelectionChanges& msg)
    d->onSelectionChanged(msg);
 }

-void ElementColors::onElementListItemDoubleClicked(QListWidgetItem *item) {
-    d->editItem(this,item);
+void ElementColors::onElementListItemDoubleClicked(QListWidgetItem* item)
+{
+    d->editItem(this, item);
 }

 /* TRANSLATOR Gui::TaskElementColors */

 TaskElementColors::TaskElementColors(ViewProviderDocumentObject* vp, bool noHide)
 {
-    widget = new ElementColors(vp,noHide);
+    widget = new ElementColors(vp, noHide);
    addTaskBox(widget);
 }

 TaskElementColors::~TaskElementColors() = default;

 void TaskElementColors::open()
-{
-}
+{}

 void TaskElementColors::clicked(int)
 {