Skip to content

Commit

Permalink
Added clear button to the editor for color properties
Browse files Browse the repository at this point in the history
  • Loading branch information
bjorn committed Nov 15, 2024
1 parent c033d94 commit 2568cb4
Showing 1 changed file with 32 additions and 8 deletions.
40 changes: 32 additions & 8 deletions src/tiled/varianteditor.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -430,22 +430,46 @@ QWidget *RectFProperty::createEditor(QWidget *parent)
return editor;
}

// todo: needs to allow setting color back to invalid (unset)
QWidget *ColorProperty::createEditor(QWidget *parent)
{
auto editor = new ColorButton(parent);
editor->setShowAlphaChannel(m_alpha);
auto editor = new QWidget(parent);
auto layout = new QHBoxLayout(editor);
layout->setContentsMargins(QMargins());
layout->setSpacing(0);

auto colorButton = new ColorButton(editor);
colorButton->setShowAlphaChannel(m_alpha);
colorButton->setSizePolicy(QSizePolicy::Preferred, QSizePolicy::Fixed);

QIcon resetIcon(QStringLiteral(":/images/16/edit-clear.png"));
resetIcon.addFile(QStringLiteral(":/images/24/edit-clear.png"));

auto resetButton = new QToolButton(editor);
resetButton->setIcon(resetIcon);
resetButton->setToolTip(tr("Clear Color"));
Utils::setThemeIcon(resetButton, "edit-clear");

layout->addWidget(colorButton);
layout->addWidget(resetButton);

auto syncEditor = [=] {
const QSignalBlocker blocker(editor);
editor->setColor(value());
const QSignalBlocker blocker(colorButton);
auto v = value();
colorButton->setColor(v);
resetButton->setEnabled(v.isValid());
};
syncEditor();

connect(this, &Property::valueChanged, editor, syncEditor);
connect(editor, &ColorButton::colorChanged, this, [this, editor] {
setValue(editor->color());
connect(resetButton, &QToolButton::clicked, colorButton, [colorButton] {
colorButton->setColor(QColor());
});
connect(this, &Property::valueChanged, colorButton, syncEditor);
connect(colorButton, &ColorButton::colorChanged, this, [=] {
resetButton->setEnabled(colorButton->color().isValid());
setValue(colorButton->color());
});

editor->setFocusProxy(colorButton);

return editor;
}
Expand Down

0 comments on commit 2568cb4

Please sign in to comment.