Skip to content

Commit

Permalink
QLayout::setMargin -> setContentsMargins
Browse files Browse the repository at this point in the history
Was apparently obsolete since at least Qt 5.6 already, but was never
marked with a warning or exclusion macro...
  • Loading branch information
bjorn committed Oct 12, 2020
1 parent 45bd6df commit 35d21cd
Show file tree
Hide file tree
Showing 21 changed files with 26 additions and 25 deletions.
2 changes: 1 addition & 1 deletion src/tiled/consoledock.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ ConsoleDock::ConsoleDock(QWidget *parent)

QWidget *widget = new QWidget(this);
QVBoxLayout *layout = new QVBoxLayout(widget);
layout->setMargin(0);
layout->setContentsMargins(0, 0, 0, 0);
layout->setSpacing(0);

mPlainTextEdit->setReadOnly(true);
Expand Down
2 changes: 1 addition & 1 deletion src/tiled/documentmanager.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,7 @@ DocumentManager::DocumentManager(QObject *parent)
vertical->addWidget(mTabBar);
vertical->addWidget(mFileChangedWarning);
vertical->addWidget(mBrokenLinksWidget);
vertical->setMargin(0);
vertical->setContentsMargins(0, 0, 0, 0);
vertical->setSpacing(0);

mEditorStack = new QStackedLayout;
Expand Down
2 changes: 1 addition & 1 deletion src/tiled/fileedit.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ FileEdit::FileEdit(QWidget *parent)
, mErrorTextColor(Qt::red)
{
QHBoxLayout *layout = new QHBoxLayout(this);
layout->setMargin(0);
layout->setContentsMargins(0, 0, 0, 0);
layout->setSpacing(0);

mLineEdit = new QLineEdit(this);
Expand Down
3 changes: 2 additions & 1 deletion src/tiled/issuescounter.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,8 @@ IssuesCounter::IssuesCounter(QWidget *parent)
, mWarningCount(new QLabel)
{
auto layout = new QHBoxLayout;
layout->setMargin(Utils::dpiScaled(2));
const int margin = Utils::dpiScaled(2);
layout->setContentsMargins(margin, margin, margin, margin);

int spacing = Utils::dpiScaled(5);
layout->addSpacing(spacing);
Expand Down
2 changes: 1 addition & 1 deletion src/tiled/issuesdock.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -190,7 +190,7 @@ IssuesDock::IssuesDock(QWidget *parent)
auto widget = new QWidget(this);
auto layout = new QVBoxLayout(widget);

layout->setMargin(0);
layout->setContentsMargins(0, 0, 0, 0);
layout->setSpacing(0);
layout->addLayout(toolBarLayout);
layout->addWidget(mIssuesView);
Expand Down
2 changes: 1 addition & 1 deletion src/tiled/layerdock.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ LayerDock::LayerDock(QWidget *parent):

QWidget *widget = new QWidget(this);
QVBoxLayout *layout = new QVBoxLayout(widget);
layout->setMargin(0);
layout->setContentsMargins(0, 0, 0, 0);

QHBoxLayout *opacityLayout = new QHBoxLayout;
mOpacitySlider->setRange(0, 100);
Expand Down
2 changes: 1 addition & 1 deletion src/tiled/locatorwidget.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -333,7 +333,7 @@ LocatorWidget::LocatorWidget(QWidget *parent)

auto margin = Utils::dpiScaled(4);
auto verticalLayout = new QVBoxLayout;
verticalLayout->setMargin(margin);
verticalLayout->setContentsMargins(margin, margin, margin, margin);
verticalLayout->setSpacing(margin);
verticalLayout->addWidget(mFilterEdit);
verticalLayout->addWidget(mResultsView);
Expand Down
2 changes: 1 addition & 1 deletion src/tiled/objectrefedit.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ ObjectRefEdit::ObjectRefEdit(QWidget *parent)
mPickObjectButton->setIcon(QIcon(QStringLiteral("://images/scalable/select-object.svg")));
mPickObjectButton->setToolTip(tr("Select Object on Map"));

layout->setMargin(0);
layout->setContentsMargins(0, 0, 0, 0);
layout->setSpacing(0);
layout->addWidget(mLineEdit);
layout->addWidget(mObjectDialogButton);
Expand Down
2 changes: 1 addition & 1 deletion src/tiled/objectsdock.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ ObjectsDock::ObjectsDock(QWidget *parent)

QWidget *widget = new QWidget(this);
QVBoxLayout *layout = new QVBoxLayout(widget);
layout->setMargin(0);
layout->setContentsMargins(0, 0, 0, 0);
layout->setSpacing(0);
layout->addWidget(mFilterEdit);
layout->addWidget(mObjectsView);
Expand Down
2 changes: 1 addition & 1 deletion src/tiled/projectdock.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ ProjectDock::ProjectDock(QWidget *parent)

auto widget = new QWidget(this);
auto layout = new QVBoxLayout(widget);
layout->setMargin(0);
layout->setContentsMargins(0, 0, 0, 0);
layout->setSpacing(0);

layout->addWidget(mProjectView);
Expand Down
2 changes: 1 addition & 1 deletion src/tiled/propertiesdock.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ PropertiesDock::PropertiesDock(QWidget *parent)

QWidget *widget = new QWidget(this);
QVBoxLayout *layout = new QVBoxLayout(widget);
layout->setMargin(0);
layout->setContentsMargins(0, 0, 0, 0);
layout->setSpacing(0);
layout->addWidget(mPropertyBrowser);
layout->addWidget(toolBar);
Expand Down
2 changes: 1 addition & 1 deletion src/tiled/shortcutsettingspage.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -386,7 +386,7 @@ ShortcutEditor::ShortcutEditor(QWidget *parent)
mResetButton->setIcon(QIcon(QLatin1String("://images/scalable/edit-undo-symbolic.svg")));

auto layout = new QHBoxLayout(this);
layout->setMargin(0);
layout->setContentsMargins(0, 0, 0, 0);
layout->setSpacing(0);
layout->addWidget(mKeySequenceEdit);
layout->addWidget(clearButton);
Expand Down
2 changes: 1 addition & 1 deletion src/tiled/templatesdock.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,7 @@ TemplatesDock::TemplatesDock(QWidget *parent)
editorLayout->addLayout(toolsLayout);
editorLayout->addWidget(mDescriptionLabel);
editorLayout->addWidget(mMapView);
editorLayout->setMargin(0);
editorLayout->setContentsMargins(0, 0, 0, 0);
editorLayout->setSpacing(0);

auto *widget = new QWidget;
Expand Down
2 changes: 1 addition & 1 deletion src/tiled/textpropertyedit.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,7 @@ TextPropertyEdit::TextPropertyEdit(QWidget *parent)
QStringLiteral("\n"),
QStringLiteral("\\n")));

layout->setMargin(0);
layout->setContentsMargins(0, 0, 0, 0);
layout->setSpacing(0);
layout->addWidget(mLineEdit);
layout->addWidget(button);
Expand Down
4 changes: 2 additions & 2 deletions src/tiled/tilecollisiondock.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -151,7 +151,7 @@ TileCollisionDock::TileCollisionDock(QWidget *parent)
mObjectsWidget->setVisible(false);
auto objectsVertical = new QVBoxLayout(mObjectsWidget);
objectsVertical->setSpacing(0);
objectsVertical->setMargin(0);
objectsVertical->setContentsMargins(0, 0, 0, 0);
objectsVertical->addWidget(mObjectsView);
objectsVertical->addWidget(objectsToolBar);

Expand Down Expand Up @@ -214,7 +214,7 @@ TileCollisionDock::TileCollisionDock(QWidget *parent)
auto widget = new QWidget(this);
auto vertical = new QVBoxLayout(widget);
vertical->setSpacing(0);
vertical->setMargin(0);
vertical->setContentsMargins(0, 0, 0, 0);
vertical->addLayout(horizontal);
vertical->addWidget(mObjectsViewSplitter);

Expand Down
2 changes: 1 addition & 1 deletion src/tiled/tilesetdock.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -209,7 +209,7 @@ TilesetDock::TilesetDock(QWidget *parent)

QVBoxLayout *vertical = new QVBoxLayout(w);
vertical->setSpacing(0);
vertical->setMargin(0);
vertical->setContentsMargins(0, 0, 0, 0);
vertical->addLayout(horizontal);
vertical->addWidget(mSuperViewStack);

Expand Down
2 changes: 1 addition & 1 deletion src/tiled/tilesetparametersedit.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ TilesetParametersEdit::TilesetParametersEdit(QWidget *parent)
: QWidget(parent)
{
QHBoxLayout *layout = new QHBoxLayout(this);
layout->setMargin(0);
layout->setContentsMargins(0, 0, 0, 0);
layout->setSpacing(0);

mLabel = new QLabel(this);
Expand Down
2 changes: 1 addition & 1 deletion src/tiled/tilestampsdock.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ TileStampsDock::TileStampsDock(TileStampManager *stampManager, QWidget *parent)

QWidget *widget = new QWidget(this);
QVBoxLayout *layout = new QVBoxLayout(widget);
layout->setMargin(0);
layout->setContentsMargins(0, 0, 0, 0);

QToolBar *buttonContainer = new QToolBar;
buttonContainer->setFloatable(false);
Expand Down
2 changes: 1 addition & 1 deletion src/tiled/undodock.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ UndoDock::UndoDock(QWidget *parent)

QWidget *widget = new QWidget(this);
QVBoxLayout *layout = new QVBoxLayout(widget);
layout->setMargin(0);
layout->setContentsMargins(0, 0, 0, 0);
layout->addWidget(mUndoView);

setWidget(widget);
Expand Down
2 changes: 1 addition & 1 deletion src/tiled/varianteditorfactory.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ ResetWidget::ResetWidget(QtProperty *property, QWidget *editor, QWidget *parent)
resetButton->setToolTip(tr("Reset"));
Utils::setThemeIcon(resetButton, "edit-clear");

layout->setMargin(0);
layout->setContentsMargins(0, 0, 0, 0);
layout->setSpacing(0);
layout->addWidget(editor);
layout->addWidget(resetButton);
Expand Down
8 changes: 4 additions & 4 deletions src/tiled/wangdock.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -199,7 +199,7 @@ WangDock::WangDock(QWidget *parent)
wangSetHorizontal->addWidget(mWangSetToolBar);

QVBoxLayout *wangSetVertical = new QVBoxLayout(wangSetWidget);
wangSetVertical->setMargin(0);
wangSetVertical->setContentsMargins(0, 0, 0, 0);
wangSetVertical->addWidget(mWangSetView);
wangSetVertical->addLayout(wangSetHorizontal);

Expand All @@ -211,7 +211,7 @@ WangDock::WangDock(QWidget *parent)
colorViewHorizontal->addWidget(mWangColorToolBar);

QVBoxLayout *colorViewVertical = new QVBoxLayout(mWangColorWidget);
colorViewVertical->setMargin(0);
colorViewVertical->setContentsMargins(0, 0, 0, 0);
colorViewVertical->addWidget(mWangColorView);
colorViewVertical->addLayout(colorViewHorizontal);

Expand All @@ -227,7 +227,7 @@ WangDock::WangDock(QWidget *parent)
templateAndColorHorizontal->addWidget(mEraseWangIdsButton);

QVBoxLayout *templateAndColorVertical = new QVBoxLayout(mTemplateAndColorWidget);
templateAndColorVertical->setMargin(0);
templateAndColorVertical->setContentsMargins(0, 0, 0, 0);
templateAndColorVertical->addWidget(mTemplateAndColorView);
templateAndColorVertical->addLayout(templateAndColorHorizontal);

Expand All @@ -238,7 +238,7 @@ WangDock::WangDock(QWidget *parent)
wangViews->addWidget(mTemplateAndColorWidget);

QVBoxLayout *vertical = new QVBoxLayout(w);
vertical->setMargin(0);
vertical->setContentsMargins(0, 0, 0, 0);
vertical->addWidget(wangViews);

hideTemplateColorView();
Expand Down

0 comments on commit 35d21cd

Please sign in to comment.