Move the "Next/Previous" card buttons to their own widget.

cockatrice/src/client/ui/widgets/printing_selector/printing_selector.cpp

@@ -3,6 +3,7 @@
 #include "../../../../settings/cache_settings.h"
 #include "../../../../utility/card_set_comparator.h"
 #include "printing_selector_card_display_widget.h"
+#include "printing_selector_card_selection_widget.h"
 
 #include <QComboBox>
 #include <QDebug>
@@ -77,18 +78,7 @@ PrintingSelector::PrintingSelector(QWidget *parent,
 
     layout->addWidget(cardSizeWidget);
 
-    cardSelectionBar = new QWidget(this);
+    cardSelectionBar = new PrintingSelectorCardSelectionWidget(this);
-    cardSelectionBarLayout = new QHBoxLayout(cardSelectionBar);
-    previousCardButton = new QPushButton(cardSelectionBar);
-    previousCardButton->setText(tr("Previous Card"));
-    connect(previousCardButton, &QPushButton::clicked, this, &PrintingSelector::selectPreviousCard);
-    nextCardButton = new QPushButton(cardSelectionBar);
-    nextCardButton->setText(tr("Next Card"));
-    connect(nextCardButton, &QPushButton::clicked, this, &PrintingSelector::selectNextCard);
-
-    cardSelectionBarLayout->addWidget(previousCardButton);
-    cardSelectionBarLayout->addWidget(nextCardButton);
-
     layout->addWidget(cardSelectionBar);
 
     connect(deckModel, &DeckListModel::dataChanged, this, [this]() {

cockatrice/src/client/ui/widgets/printing_selector/printing_selector.h

@@ -15,6 +15,7 @@
 
 #define BATCH_SIZE 10
 
+class PrintingSelectorCardSelectionWidget;
 class TabDeckEditor;
 class PrintingSelector : public QWidget
 {
@@ -54,10 +55,7 @@ private:
     QHBoxLayout *cardSizeLayout;
     QLabel *cardSizeLabel;
     QSlider *cardSizeSlider;
-    QWidget *cardSelectionBar;
+    PrintingSelectorCardSelectionWidget *cardSelectionBar;
-    QHBoxLayout *cardSelectionBarLayout;
-    QPushButton *previousCardButton;
-    QPushButton *nextCardButton;
     TabDeckEditor *deckEditor;
     DeckListModel *deckModel;
     QTreeView *deckView;

cockatrice/src/client/ui/widgets/printing_selector/printing_selector_card_selection_widget.cpp

@@ -0,0 +1,22 @@
+#include "printing_selector_card_selection_widget.h"
+
+PrintingSelectorCardSelectionWidget::PrintingSelectorCardSelectionWidget(PrintingSelector *parent) : parent(parent)
+{
+    cardSelectionBarLayout = new QHBoxLayout(this);
+    previousCardButton = new QPushButton(this);
+    previousCardButton->setText(tr("Previous Card"));
+
+    nextCardButton = new QPushButton(this);
+    nextCardButton->setText(tr("Next Card"));
+
+    connectSignals();
+
+    cardSelectionBarLayout->addWidget(previousCardButton);
+    cardSelectionBarLayout->addWidget(nextCardButton);
+}
+
+void PrintingSelectorCardSelectionWidget::connectSignals()
+{
+    connect(previousCardButton, &QPushButton::clicked, parent, &PrintingSelector::selectPreviousCard);
+    connect(nextCardButton, &QPushButton::clicked, parent, &PrintingSelector::selectNextCard);
+}

cockatrice/src/client/ui/widgets/printing_selector/printing_selector_card_selection_widget.h

@@ -0,0 +1,26 @@
+#ifndef PRINTING_SELECTOR_CARD_SELECTION_WIDGET_H
+#define PRINTING_SELECTOR_CARD_SELECTION_WIDGET_H
+
+#include "printing_selector.h"
+
+#include <QHBoxLayout>
+#include <QPushButton>
+#include <QWidget>
+
+class PrintingSelectorCardSelectionWidget : public QWidget
+{
+    Q_OBJECT
+
+public:
+    explicit PrintingSelectorCardSelectionWidget(PrintingSelector *parent);
+
+    void connectSignals();
+
+private:
+    PrintingSelector *parent;
+    QHBoxLayout *cardSelectionBarLayout;
+    QPushButton *previousCardButton;
+    QPushButton *nextCardButton;
+};
+
+#endif // PRINTING_SELECTOR_CARD_SELECTION_WIDGET_H