[TabArchidekt] Cleaner filters, infinite scrolling, and a "go back button" (#6545)

* [TabArchidekt] Cleaner filters, infinite scrolling, and a "go back button"

Took 46 minutes

Took 5 seconds

* Fix infinite scroll triggering in detail view.

Took 25 minutes

Took 3 seconds

* Use setLabelText() so it's white

Took 2 minutes

---------

Co-authored-by: Lukas Brübach <Bruebach.Lukas@bdosecurity.de>
This commit is contained in:
BruebachL 2026-01-24 11:21:12 +01:00 committed by GitHub
parent 3c48d92663
commit 12b5525a2d
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
7 changed files with 534 additions and 354 deletions

View file

@ -9,6 +9,9 @@
#include <QCompleter>
#include <QDebug>
#include <QFormLayout>
#include <QGridLayout>
#include <QGroupBox>
#include <QHBoxLayout>
#include <QJsonArray>
#include <QJsonDocument>
@ -18,129 +21,258 @@
#include <QPushButton>
#include <QRegularExpression>
#include <QResizeEvent>
#include <QScrollArea>
#include <QScrollBar>
#include <QUrlQuery>
#include <libcockatrice/card/database/card_database_manager.h>
#include <libcockatrice/models/database/card/card_completer_proxy_model.h>
#include <libcockatrice/models/database/card/card_search_model.h>
#include <version_string.h>
TabArchidekt::TabArchidekt(TabSupervisor *_tabSupervisor) : Tab(_tabSupervisor)
TabArchidekt::TabArchidekt(TabSupervisor *_tabSupervisor)
: Tab(_tabSupervisor), currentPage(1), isLoadingMore(false), isListMode(true)
{
// Initialize network
networkManager = new QNetworkAccessManager(this);
networkManager->setTransferTimeout(); // Use Qt's default timeout
networkManager->setTransferTimeout();
networkManager->setRedirectPolicy(QNetworkRequest::ManualRedirectPolicy);
connect(networkManager, SIGNAL(finished(QNetworkReply *)), this, SLOT(processApiJson(QNetworkReply *)));
connect(networkManager, &QNetworkAccessManager::finished, this, &TabArchidekt::processApiJson);
// Initialize debounce timer
searchDebounceTimer = new QTimer(this);
searchDebounceTimer->setSingleShot(true); // We only want it to fire once after inactivity
searchDebounceTimer->setInterval(300); // 300ms debounce
searchDebounceTimer->setSingleShot(true);
searchDebounceTimer->setInterval(300);
connect(searchDebounceTimer, &QTimer::timeout, this, &TabArchidekt::doSearchImmediate);
connect(searchDebounceTimer, &QTimer::timeout, this, [this]() { doSearchImmediate(); });
initializeUi();
setupFilterWidgets();
connectSignals();
retranslateUi();
getTopDecks();
}
void TabArchidekt::initializeUi()
{
// Main container
container = new QWidget(this);
mainLayout = new QVBoxLayout(container);
mainLayout->setContentsMargins(0, 0, 0, 0);
container->setLayout(mainLayout);
mainLayout->setSpacing(0);
navigationContainer = new QWidget(container);
navigationContainer->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Minimum);
navigationLayout = new QHBoxLayout(navigationContainer);
navigationLayout->setSpacing(3);
navigationContainer->setLayout(navigationLayout);
// Primary toolbar (most important filters)
primaryToolbar = new QWidget(container);
primaryToolbar->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Fixed);
primaryToolbarLayout = new QHBoxLayout(primaryToolbar);
primaryToolbarLayout->setContentsMargins(6, 6, 6, 6);
primaryToolbarLayout->setSpacing(6);
// Sort by
orderByCombo = new QComboBox(navigationContainer);
// Sort controls
sortByLabel = new QLabel(primaryToolbar);
orderByCombo = new QComboBox(primaryToolbar);
orderByCombo->addItems({"name", "updatedAt", "createdAt", "viewCount", "size", "edhBracket"});
orderByCombo->setCurrentText("updatedAt"); // Pre-select updatedAt
orderByCombo->setCurrentText("updatedAt");
orderByCombo->setSizeAdjustPolicy(QComboBox::AdjustToContents);
// Asc/Desc toggle
orderDirButton = new QPushButton(tr("Desc."), navigationContainer);
orderDirButton->setCheckable(true); // checked = DESC, unchecked = ASC
orderDirButton = new QPushButton(tr("Desc."), primaryToolbar);
orderDirButton->setCheckable(true);
orderDirButton->setChecked(true);
orderDirButton->setFixedWidth(60);
connect(orderByCombo, &QComboBox::currentTextChanged, this, &TabArchidekt::doSearch);
connect(orderDirButton, &QPushButton::clicked, this, [this](bool checked) {
orderDirButton->setText(checked ? tr("Desc.") : tr("Asc."));
doSearch();
});
// Colors
QHBoxLayout *colorLayout = new QHBoxLayout();
QString colorIdentity = "WUBRG"; // Optionally include "C" for colorless once we have a symbol for it
// Color filter (inline)
QWidget *colorWidget = new QWidget(primaryToolbar);
QHBoxLayout *colorLayout = new QHBoxLayout(colorWidget);
colorLayout->setContentsMargins(0, 0, 0, 0);
colorLayout->setSpacing(2);
QString colorIdentity = "WUBRG";
for (const QChar &color : colorIdentity) {
auto *manaSymbol = new ManaSymbolWidget(navigationContainer, color, false, true);
manaSymbol->setFixedWidth(25);
auto *manaSymbol = new ManaSymbolWidget(colorWidget, color, false, true);
manaSymbol->setFixedSize(28, 28);
colorSymbols.append(manaSymbol);
colorLayout->addWidget(manaSymbol);
connect(manaSymbol, &ManaSymbolWidget::colorToggled, this, [this](QChar c, bool active) {
if (active) {
if (active)
activeColors.insert(c);
} else {
else
activeColors.remove(c);
}
doSearch();
});
}
logicalAndCheck = new QCheckBox("Require ALL colors", navigationContainer);
logicalAndCheck = new QCheckBox(tr("AND"), primaryToolbar);
logicalAndCheck->setToolTip(tr("Require ALL selected colors"));
// Formats
// Common search fields
nameField = new QLineEdit(primaryToolbar);
nameField->setPlaceholderText(tr("Deck name..."));
nameField->setMinimumWidth(150);
formatLabel = new QLabel(this);
ownerField = new QLineEdit(primaryToolbar);
ownerField->setPlaceholderText(tr("Owner..."));
ownerField->setMinimumWidth(120);
formatSettingsWidget = new SettingsButtonWidget(this);
// Filter by label
filterByLabel = new QLabel(primaryToolbar);
// Package toggle
packagesCheck = new QCheckBox(tr("Packages"), primaryToolbar);
// Search button
searchButton = new QPushButton(tr("Search"), primaryToolbar);
searchButton->setDefault(true);
// Advanced filters toggle button
advancedFiltersButton = new QPushButton(tr("Advanced Filters"), primaryToolbar);
advancedFiltersButton->setCheckable(true);
advancedFiltersButton->setChecked(false);
// Settings
settingsButton = new SettingsButtonWidget(primaryToolbar);
cardSizeSlider = new CardSizeWidget(primaryToolbar, nullptr, SettingsCache::instance().getArchidektPreviewSize());
settingsButton->addSettingsWidget(cardSizeSlider);
// Assemble primary toolbar
primaryToolbarLayout->addWidget(sortByLabel);
primaryToolbarLayout->addWidget(orderByCombo);
primaryToolbarLayout->addWidget(orderDirButton);
// Add separator/spacing
primaryToolbarLayout->addSpacing(12);
primaryToolbarLayout->addWidget(filterByLabel);
primaryToolbarLayout->addWidget(colorWidget);
primaryToolbarLayout->addWidget(logicalAndCheck);
primaryToolbarLayout->addWidget(nameField, 1);
primaryToolbarLayout->addWidget(ownerField, 1);
primaryToolbarLayout->addWidget(packagesCheck);
primaryToolbarLayout->addWidget(searchButton, 1);
primaryToolbarLayout->addWidget(advancedFiltersButton);
primaryToolbarLayout->addWidget(settingsButton);
// Secondary toolbar (advanced filters - initially hidden)
secondaryToolbar = new QWidget(container);
secondaryToolbar->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Fixed);
secondaryToolbar->setVisible(false); // Start hidden
secondaryToolbarLayout = new QHBoxLayout(secondaryToolbar);
secondaryToolbarLayout->setContentsMargins(6, 3, 6, 6);
secondaryToolbarLayout->setSpacing(6);
// Scrollable results area
scrollArea = new QScrollArea(container);
scrollArea->setWidgetResizable(true);
scrollArea->setHorizontalScrollBarPolicy(Qt::ScrollBarAlwaysOff);
scrollArea->setVerticalScrollBarPolicy(Qt::ScrollBarAsNeeded);
resultsContainer = new QWidget();
resultsLayout = new QVBoxLayout(resultsContainer);
resultsLayout->setContentsMargins(0, 0, 0, 0);
resultsLayout->setSpacing(0);
scrollArea->setWidget(resultsContainer);
scrollArea->viewport()->installEventFilter(this);
mainLayout->addWidget(primaryToolbar);
mainLayout->addWidget(secondaryToolbar);
mainLayout->addWidget(scrollArea);
setCentralWidget(container);
}
bool TabArchidekt::eventFilter(QObject *obj, QEvent *event)
{
if (obj == scrollArea->viewport() && event->type() == QEvent::Wheel) {
auto *wheelEvent = static_cast<QWheelEvent *>(event);
if (wheelEvent->angleDelta().y() < 0 && !isLoadingMore && isListMode) {
loadNextPage();
wheelEvent->accept();
return false; // allow scrolling
}
}
// Always pass the event to the parent to handle normal scrolling
return QWidget::eventFilter(obj, event);
}
void TabArchidekt::setupFilterWidgets()
{
// Advanced filters (in secondary toolbar)
// EDH Bracket
auto *bracketLabel = new QLabel(tr("Bracket:"), secondaryToolbar);
edhBracketCombo = new QComboBox(secondaryToolbar);
edhBracketCombo->addItem(tr("Any"));
edhBracketCombo->addItems({"1", "2", "3", "4", "5"});
edhBracketCombo->setSizeAdjustPolicy(QComboBox::AdjustToContents);
// Format filter (collapsible)
formatButton = new SettingsButtonWidget(secondaryToolbar);
formatButton->setButtonText(tr("Formats"));
formatButton->setButtonIcon(QPixmap("theme:icons/scale_balanced"));
QWidget *formatContainer = new QWidget(secondaryToolbar);
QGridLayout *formatLayout = new QGridLayout(formatContainer);
formatLayout->setContentsMargins(4, 4, 4, 4);
QStringList formatNames = {"Standard", "Modern", "Commander", "Legacy", "Vintage",
"Pauper", "Custom", "Frontier", "Future Std", "Penny Dreadful",
"1v1 Commander", "Dual Commander", "Brawl"};
for (int i = 0; i < formatNames.size(); ++i) {
QCheckBox *formatCheckBox = new QCheckBox(formatNames[i], navigationContainer);
connect(formatCheckBox, &QCheckBox::clicked, this, &TabArchidekt::doSearch);
int row = 0, col = 0;
for (const QString &formatName : formatNames) {
auto *formatCheckBox = new QCheckBox(formatName, formatContainer);
formatChecks << formatCheckBox;
formatSettingsWidget->addSettingsWidget(formatCheckBox);
formatLayout->addWidget(formatCheckBox, row, col);
connect(formatCheckBox, &QCheckBox::clicked, this, &TabArchidekt::doSearch);
col++;
if (col >= 3) {
col = 0;
row++;
}
}
// EDH Bracket
edhBracketCombo = new QComboBox(navigationContainer);
edhBracketCombo->addItem(tr("Any Bracket"));
edhBracketCombo->addItems({"1", "2", "3", "4", "5"});
formatButton->addSettingsWidget(formatContainer);
connect(edhBracketCombo, &QComboBox::currentTextChanged, this, &TabArchidekt::doSearch);
cardsField = new QLineEdit(secondaryToolbar);
cardsField->setPlaceholderText(tr("Contains card..."));
cardsField->setMinimumWidth(140);
// Search for Card Packages instead of Decks
packagesCheck = new QCheckBox("Packages", navigationContainer);
commandersField = new QLineEdit(secondaryToolbar);
commandersField->setPlaceholderText(tr("Commander..."));
commandersField->setMinimumWidth(140);
connect(packagesCheck, &QCheckBox::clicked, this, [this]() {
bool disable = packagesCheck->isChecked();
for (auto *cb : formatChecks)
cb->setEnabled(!disable);
commandersField->setEnabled(!disable);
deckTagNameField->setEnabled(!disable);
edhBracketCombo->setCurrentIndex(0);
edhBracketCombo->setEnabled(!disable);
doSearch();
});
deckTagNameField = new QLineEdit(secondaryToolbar);
deckTagNameField->setPlaceholderText(tr("Tag..."));
deckTagNameField->setMinimumWidth(100);
// Deck Name
nameField = new QLineEdit(navigationContainer);
nameField->setPlaceholderText(tr("Deck name contains..."));
// Deck size filter (collapsible)
deckSizeButton = new SettingsButtonWidget(secondaryToolbar);
deckSizeButton->setButtonText(tr("Deck Size"));
// Owner Name
ownerField = new QLineEdit(navigationContainer);
ownerField->setPlaceholderText(tr("Owner name contains..."));
QWidget *sizeContainer = new QWidget(secondaryToolbar);
QHBoxLayout *sizeLayout = new QHBoxLayout(sizeContainer);
sizeLayout->setContentsMargins(4, 4, 4, 4);
// Contained cards
cardsField = new QLineEdit(navigationContainer);
cardsField->setPlaceholderText("Deck contains card...");
minDeckSizeSpin = new QSpinBox(sizeContainer);
minDeckSizeSpin->setSpecialValueText(tr("Any"));
minDeckSizeSpin->setRange(0, 200);
minDeckSizeSpin->setValue(0);
// Commanders
commandersField = new QLineEdit(navigationContainer);
commandersField->setPlaceholderText("Deck has commander...");
minDeckSizeLogicCombo = new QComboBox(sizeContainer);
minDeckSizeLogicCombo->addItems({"Exact", "", ""});
minDeckSizeLogicCombo->setCurrentIndex(1);
// DB supplemented card search
sizeLayout->addWidget(new QLabel(tr("Cards:"), sizeContainer));
sizeLayout->addWidget(minDeckSizeSpin);
sizeLayout->addWidget(minDeckSizeLogicCombo);
deckSizeButton->addSettingsWidget(sizeContainer);
// Setup card name autocomplete
auto cardDatabaseModel = new CardDatabaseModel(CardDatabaseManager::getInstance(), false, this);
auto displayModel = new CardDatabaseDisplayModel(this);
displayModel->setSourceModel(cardDatabaseModel);
@ -161,144 +293,119 @@ TabArchidekt::TabArchidekt(TabSupervisor *_tabSupervisor) : Tab(_tabSupervisor)
cardsField->setCompleter(completer);
commandersField->setCompleter(completer);
connect(cardsField, &QLineEdit::textChanged, searchModel, &CardSearchModel::updateSearchResults);
// Keep autocomplete working for both fields
connect(cardsField, &QLineEdit::textChanged, this, [=](const QString &text) {
searchModel->updateSearchResults(text);
QString pattern = ".*" + QRegularExpression::escape(text) + ".*";
proxyModel->setFilterRegularExpression(QRegularExpression(pattern, QRegularExpression::CaseInsensitiveOption));
if (!text.isEmpty())
completer->complete();
});
connect(commandersField, &QLineEdit::textChanged, searchModel, &CardSearchModel::updateSearchResults);
connect(commandersField, &QLineEdit::textChanged, this, [=](const QString &text) {
searchModel->updateSearchResults(text);
QString pattern = ".*" + QRegularExpression::escape(text) + ".*";
proxyModel->setFilterRegularExpression(QRegularExpression(pattern, QRegularExpression::CaseInsensitiveOption));
if (!text.isEmpty())
completer->complete();
});
// Tag Name
deckTagNameField = new QLineEdit(navigationContainer);
deckTagNameField->setPlaceholderText("Deck tag");
// Assemble secondary toolbar
secondaryToolbarLayout->addWidget(bracketLabel);
secondaryToolbarLayout->addWidget(edhBracketCombo);
secondaryToolbarLayout->addWidget(formatButton);
secondaryToolbarLayout->addWidget(cardsField);
secondaryToolbarLayout->addWidget(commandersField);
secondaryToolbarLayout->addWidget(deckTagNameField);
secondaryToolbarLayout->addWidget(deckSizeButton);
secondaryToolbarLayout->addStretch();
}
connect(deckTagNameField, &QLineEdit::textChanged, this, &TabArchidekt::doSearch);
void TabArchidekt::connectSignals()
{
// Advanced filters toggle
connect(advancedFiltersButton, &QPushButton::clicked, this,
[this](bool checked) { secondaryToolbar->setVisible(checked); });
// Search button
searchPushButton = new QPushButton(navigationContainer);
searchPushButton->setText("Search");
// These trigger immediate search (no debounce needed)
connect(orderByCombo, &QComboBox::currentTextChanged, this, &TabArchidekt::doSearch);
connect(orderDirButton, &QPushButton::clicked, [this](bool checked) {
orderDirButton->setText(checked ? tr("Desc.") : tr("Asc."));
doSearch();
});
connect(searchPushButton, &QPushButton::clicked, this, &TabArchidekt::doSearch);
// Card Size settings
settingsButton = new SettingsButtonWidget(this);
cardSizeSlider = new CardSizeWidget(this, nullptr, SettingsCache::instance().getArchidektPreviewSize());
connect(cardSizeSlider, &CardSizeWidget::cardSizeSettingUpdated, &SettingsCache::instance(),
&SettingsCache::setArchidektPreviewCardSize);
settingsButton->addSettingsWidget(cardSizeSlider);
// Min deck size
minDeckSizeLabel = new QLabel(navigationContainer);
// Search button triggers immediate search
connect(searchButton, &QPushButton::clicked, this, &TabArchidekt::doSearchImmediate);
minDeckSizeSpin = new QSpinBox(navigationContainer);
minDeckSizeSpin->setSpecialValueText(tr("Disabled"));
minDeckSizeSpin->setRange(0, 200);
minDeckSizeSpin->setValue(0);
// Size logic
minDeckSizeLogicCombo = new QComboBox(navigationContainer);
minDeckSizeLogicCombo->addItems({"Exact", "", ""}); // Exact = unset, ≥ = GTE, ≤ = LTE
minDeckSizeLogicCombo->setCurrentIndex(1); // default GTE
// These trigger search (but not text fields)
connect(logicalAndCheck, &QCheckBox::clicked, this, &TabArchidekt::doSearch);
connect(packagesCheck, &QCheckBox::clicked, [this]() {
updatePackageModeState(packagesCheck->isChecked());
doSearch();
});
// Format filters trigger search
connect(edhBracketCombo, &QComboBox::currentTextChanged, this, &TabArchidekt::doSearch);
connect(minDeckSizeSpin, qOverload<int>(&QSpinBox::valueChanged), this, &TabArchidekt::doSearch);
connect(minDeckSizeLogicCombo, &QComboBox::currentTextChanged, this, &TabArchidekt::doSearch);
// Page number
pageLabel = new QLabel(navigationContainer);
// Allow Enter key in text fields to trigger search
connect(nameField, &QLineEdit::returnPressed, this, &TabArchidekt::doSearchImmediate);
connect(ownerField, &QLineEdit::returnPressed, this, &TabArchidekt::doSearchImmediate);
connect(cardsField, &QLineEdit::returnPressed, this, &TabArchidekt::doSearchImmediate);
connect(commandersField, &QLineEdit::returnPressed, this, &TabArchidekt::doSearchImmediate);
connect(deckTagNameField, &QLineEdit::returnPressed, this, &TabArchidekt::doSearchImmediate);
pageSpin = new QSpinBox(navigationContainer);
pageSpin->setRange(1, 9999);
pageSpin->setValue(1);
// Format checkboxes trigger search
for (auto *formatCheck : formatChecks) {
connect(formatCheck, &QCheckBox::clicked, this, &TabArchidekt::doSearch);
}
}
connect(pageSpin, qOverload<int>(&QSpinBox::valueChanged), this, &TabArchidekt::doSearch);
void TabArchidekt::updatePackageModeState(bool isPackageMode)
{
// Disable format-specific and commander-specific filters in package mode
for (auto *cb : formatChecks) {
cb->setEnabled(!isPackageMode);
}
// Page display
currentPageDisplay = new QWidget(container);
currentPageLayout = new QVBoxLayout(currentPageDisplay);
currentPageLayout->setContentsMargins(0, 0, 0, 0);
currentPageDisplay->setLayout(currentPageLayout);
edhBracketCombo->setEnabled(!isPackageMode);
if (isPackageMode) {
edhBracketCombo->setCurrentIndex(0);
}
// Layout composition
// Sort section
navigationLayout->addWidget(orderByCombo);
navigationLayout->addWidget(orderDirButton);
// Colors section
navigationLayout->addLayout(colorLayout);
navigationLayout->addWidget(logicalAndCheck);
// Formats section
navigationLayout->addWidget(formatSettingsWidget);
navigationLayout->addWidget(formatLabel);
// EDH Bracket
navigationLayout->addWidget(edhBracketCombo);
// Packages toggle
navigationLayout->addWidget(packagesCheck);
// Deck name
navigationLayout->addWidget(nameField);
// Owner name
navigationLayout->addWidget(ownerField);
// Contained cards
navigationLayout->addWidget(cardsField);
// Commanders
navigationLayout->addWidget(commandersField);
// Deck tag
navigationLayout->addWidget(deckTagNameField);
// Search button
navigationLayout->addWidget(searchPushButton);
// Card size settings
navigationLayout->addWidget(settingsButton);
// Min. # of cards in deck
navigationLayout->addWidget(minDeckSizeLabel);
navigationLayout->addWidget(minDeckSizeSpin);
navigationLayout->addWidget(minDeckSizeLogicCombo);
// Page number
navigationLayout->addWidget(pageLabel);
navigationLayout->addWidget(pageSpin);
mainLayout->addWidget(navigationContainer);
mainLayout->addWidget(currentPageDisplay);
// Ensure navigation stays at the top and currentPageDisplay takes remaining space
mainLayout->setStretch(0, 0); // navigationContainer gets minimum space
mainLayout->setStretch(1, 1); // currentPageDisplay expands as much as possible
setCentralWidget(container);
TabArchidekt::retranslateUi();
getTopDecks();
commandersField->setEnabled(!isPackageMode);
deckTagNameField->setEnabled(!isPackageMode);
}
void TabArchidekt::retranslateUi()
{
searchPushButton->setText(tr("Search"));
formatLabel->setText(tr("Formats"));
minDeckSizeLabel->setText(tr("Min. # of Cards:"));
pageLabel->setText(tr("Page:"));
sortByLabel->setText(tr("Sort by:"));
orderDirButton->setText(orderDirButton->isChecked() ? tr("Desc.") : tr("Asc."));
filterByLabel->setText(tr("Filter by:"));
logicalAndCheck->setText(tr("AND"));
logicalAndCheck->setToolTip(tr("Require ALL selected colors"));
nameField->setPlaceholderText(tr("Deck name..."));
ownerField->setPlaceholderText(tr("Owner..."));
packagesCheck->setText(tr("Packages"));
advancedFiltersButton->setText(tr("Advanced Filters"));
cardsField->setPlaceholderText(tr("Contains card..."));
commandersField->setPlaceholderText(tr("Commander..."));
deckTagNameField->setPlaceholderText(tr("Tag..."));
formatButton->setButtonText(tr("Formats"));
deckSizeButton->setButtonText(tr("Deck Size"));
searchButton->setText(tr("Search"));
settingsButton->setToolTip(tr("Display Settings"));
}
QString TabArchidekt::buildSearchUrl()
@ -306,13 +413,11 @@ QString TabArchidekt::buildSearchUrl()
QUrlQuery query;
// orderBy (field + direction)
{
QString field = orderByCombo->currentText();
if (!field.isEmpty()) {
bool desc = orderDirButton->isChecked();
QString final = desc ? "-" + field : field;
query.addQueryItem("orderBy", final);
}
QString field = orderByCombo->currentText();
if (!field.isEmpty()) {
bool desc = orderDirButton->isChecked();
QString final = desc ? "-" + field : field;
query.addQueryItem("orderBy", final);
}
// Colors
@ -329,29 +434,26 @@ QString TabArchidekt::buildSearchUrl()
query.addQueryItem("logicalAnd", "true");
}
// Formats
// Formats (disabled in package mode)
if (!packagesCheck->isChecked()) {
QStringList formatIds;
for (int i = 0; i < formatChecks.size(); ++i)
for (int i = 0; i < formatChecks.size(); ++i) {
if (formatChecks[i]->isChecked()) {
formatIds << QString::number(i + 1);
}
}
if (!formatIds.isEmpty()) {
query.addQueryItem("deckFormat", formatIds.join(","));
}
}
// edhBracket
if (!packagesCheck->isChecked()) {
if (!edhBracketCombo->currentText().isEmpty()) {
if (edhBracketCombo->currentText() != tr("Any Bracket")) {
query.addQueryItem("edhBracket", edhBracketCombo->currentText());
}
// edhBracket
if (edhBracketCombo->currentIndex() > 0) {
query.addQueryItem("edhBracket", edhBracketCombo->currentText());
}
}
// Search for card packages instead of decks
// Package mode
if (packagesCheck->isChecked()) {
query.addQueryItem("packages", "true");
}
@ -361,54 +463,47 @@ QString TabArchidekt::buildSearchUrl()
query.addQueryItem("name", nameField->text());
}
// owner
// Owner
if (!ownerField->text().isEmpty()) {
query.addQueryItem("ownerUsername", ownerField->text());
}
// cards
// Cards
if (!cardsField->text().isEmpty()) {
query.addQueryItem("cardName", cardsField->text());
query.addQueryItem("cards", cardsField->text());
}
// Commander Name
if (!packagesCheck->isChecked()) {
if (!commandersField->text().isEmpty()) {
query.addQueryItem("commanderName", commandersField->text());
}
// Commander (disabled in package mode)
if (!packagesCheck->isChecked() && !commandersField->text().isEmpty()) {
query.addQueryItem("commanderName", commandersField->text());
}
// deckTagName
if (!packagesCheck->isChecked()) {
if (!deckTagNameField->text().isEmpty()) {
query.addQueryItem("deckTagName", deckTagNameField->text());
}
// Deck tag (disabled in package mode)
if (!packagesCheck->isChecked() && !deckTagNameField->text().isEmpty()) {
query.addQueryItem("deckTagName", deckTagNameField->text());
}
// page number
if (pageSpin->value() <= 1) {
query.addQueryItem("page", QString::number(pageSpin->value()));
}
// Page number (for infinite scroll)
query.addQueryItem("page", QString::number(currentPage));
// Min deck size
if (minDeckSizeSpin->value() != 0) {
query.addQueryItem("size", QString::number(minDeckSizeSpin->value()));
QString logic = "GTE"; // default
QString logic = "GTE";
QString selected = minDeckSizeLogicCombo->currentText();
if (selected == "")
logic = "GTE";
else if (selected == "")
logic = "LTE";
else
logic = ""; // Exact = unset
logic = "";
if (!logic.isEmpty()) {
query.addQueryItem("sizeLogic", logic);
}
}
// build final URL
QUrl url("https://archidekt.com/api/decks/v3/");
url.setQuery(query);
@ -417,7 +512,12 @@ QString TabArchidekt::buildSearchUrl()
void TabArchidekt::doSearch()
{
searchDebounceTimer->start();
// Reset to first page on new search
currentPage = 1;
// We're searching, so we'll be in list mode
isListMode = true;
// Don't debounce - only called by explicit user actions now
doSearchImmediate();
}
void TabArchidekt::doSearchImmediate()
@ -428,6 +528,21 @@ void TabArchidekt::doSearchImmediate()
networkManager->get(req);
}
void TabArchidekt::loadNextPage()
{
if (isLoadingMore) {
return;
}
isLoadingMore = true;
currentPage++;
QString url = buildSearchUrl();
QNetworkRequest req{QUrl(url)};
req.setHeader(QNetworkRequest::UserAgentHeader, QString("Cockatrice %1").arg(VERSION_STRING));
networkManager->get(req);
}
void TabArchidekt::actNavigatePage(QString url)
{
QNetworkRequest request{QUrl(url)};
@ -437,6 +552,7 @@ void TabArchidekt::actNavigatePage(QString url)
void TabArchidekt::getTopDecks()
{
currentPage = 1;
QNetworkRequest request{QUrl(buildSearchUrl())};
request.setHeader(QNetworkRequest::UserAgentHeader, QString("Cockatrice %1").arg(VERSION_STRING));
networkManager->get(request);
@ -445,7 +561,7 @@ void TabArchidekt::getTopDecks()
void TabArchidekt::processApiJson(QNetworkReply *reply)
{
if (reply->error() != QNetworkReply::NoError) {
qDebug() << "Network error occurred:" << reply->errorString();
isLoadingMore = false;
reply->deleteLater();
return;
}
@ -454,17 +570,14 @@ void TabArchidekt::processApiJson(QNetworkReply *reply)
QJsonDocument jsonDoc = QJsonDocument::fromJson(responseData);
if (!jsonDoc.isObject()) {
qDebug() << "Invalid JSON response received.";
isLoadingMore = false;
reply->deleteLater();
return;
}
QJsonObject jsonObj = jsonDoc.object();
// Get the actual URL from the reply
QString responseUrl = reply->url().toString();
// Check if the response URL matches a commander request
if (responseUrl.startsWith("https://archidekt.com/api/decks/v3/")) {
processTopDecksResponse(jsonObj);
} else if (responseUrl.startsWith("https://archidekt.com/api/decks/")) {
@ -473,6 +586,7 @@ void TabArchidekt::processApiJson(QNetworkReply *reply)
prettyPrintJson(jsonObj, 4);
}
isLoadingMore = false;
reply->deleteLater();
}
@ -481,28 +595,27 @@ void TabArchidekt::processTopDecksResponse(QJsonObject reply)
ArchidektDeckListingApiResponse deckData;
deckData.fromJson(reply);
// **Remove previous page display to prevent stacking**
if (currentPageDisplay) {
mainLayout->removeWidget(currentPageDisplay);
delete currentPageDisplay;
currentPageDisplay = nullptr;
// New search → clear everything
if (currentPage == 1) {
QLayoutItem *item;
while ((item = resultsLayout->takeAt(0)) != nullptr) {
delete item->widget();
delete item;
}
listingsWidget = new ArchidektApiResponseDeckListingsDisplayWidget(resultsContainer, deckData, cardSizeSlider);
connect(listingsWidget, &ArchidektApiResponseDeckListingsDisplayWidget::requestNavigation, this,
&TabArchidekt::actNavigatePage);
resultsLayout->addWidget(listingsWidget);
return;
}
// **Create new currentPageDisplay**
currentPageDisplay = new QWidget(container);
currentPageLayout = new QVBoxLayout(currentPageDisplay);
currentPageDisplay->setLayout(currentPageLayout);
auto display = new ArchidektApiResponseDeckListingsDisplayWidget(currentPageDisplay, deckData, cardSizeSlider);
connect(display, &ArchidektApiResponseDeckListingsDisplayWidget::requestNavigation, this,
&TabArchidekt::actNavigatePage);
currentPageLayout->addWidget(display);
mainLayout->addWidget(currentPageDisplay);
// **Ensure layout stays correct**
mainLayout->setStretch(0, 0); // Keep navigationContainer at the top
mainLayout->setStretch(1, 1); // Make sure currentPageDisplay takes remaining space
// Infinite scroll → append
if (listingsWidget) {
listingsWidget->append(deckData);
}
}
void TabArchidekt::processDeckResponse(QJsonObject reply)
@ -510,54 +623,47 @@ void TabArchidekt::processDeckResponse(QJsonObject reply)
ArchidektApiResponseDeck deckData;
deckData.fromJson(reply);
// **Remove previous page display to prevent stacking**
if (currentPageDisplay) {
mainLayout->removeWidget(currentPageDisplay);
delete currentPageDisplay;
currentPageDisplay = nullptr;
// We're in single deck mode - disable infinite scroll
isListMode = false;
// Clear existing results for single deck view
QLayoutItem *item;
while ((item = resultsLayout->takeAt(0)) != nullptr) {
delete item->widget();
delete item;
}
// **Create new currentPageDisplay**
currentPageDisplay = new QWidget(container);
currentPageLayout = new QVBoxLayout(currentPageDisplay);
currentPageDisplay->setLayout(currentPageLayout);
auto display = new ArchidektApiResponseDeckDisplayWidget(currentPageDisplay, deckData, cardSizeSlider);
auto display = new ArchidektApiResponseDeckDisplayWidget(resultsContainer, deckData, cardSizeSlider);
connect(display, &ArchidektApiResponseDeckDisplayWidget::requestNavigation, this, &TabArchidekt::actNavigatePage);
connect(display, &ArchidektApiResponseDeckDisplayWidget::requestSearch, this, &TabArchidekt::doSearchImmediate);
connect(display, &ArchidektApiResponseDeckDisplayWidget::openInDeckEditor, tabSupervisor,
&TabSupervisor::openDeckInNewTab);
currentPageLayout->addWidget(display);
mainLayout->addWidget(currentPageDisplay);
// **Ensure layout stays correct**
mainLayout->setStretch(0, 0); // Keep navigationContainer at the top
mainLayout->setStretch(1, 1); // Make sure currentPageDisplay takes remaining space
resultsLayout->addWidget(display);
}
void TabArchidekt::prettyPrintJson(const QJsonValue &value, int indentLevel)
{
const QString indent(indentLevel * 2, ' '); // Adjust spacing as needed for pretty printing
const QString indent(indentLevel * 2, ' ');
if (value.isObject()) {
QJsonObject obj = value.toObject();
for (auto it = obj.begin(); it != obj.end(); ++it) {
qDebug().noquote() << indent + it.key() + ":";
qInfo().noquote() << indent + it.key() + ":";
prettyPrintJson(it.value(), indentLevel + 1);
}
} else if (value.isArray()) {
QJsonArray array = value.toArray();
for (int i = 0; i < array.size(); ++i) {
qDebug().noquote() << indent + QString("[%1]:").arg(i);
qInfo().noquote() << indent + QString("[%1]:").arg(i);
prettyPrintJson(array[i], indentLevel + 1);
}
} else if (value.isString()) {
qDebug().noquote() << indent + "\"" + value.toString() + "\"";
qInfo().noquote() << indent + "\"" + value.toString() + "\"";
} else if (value.isDouble()) {
qDebug().noquote() << indent + QString::number(value.toDouble());
qInfo().noquote() << indent + QString::number(value.toDouble());
} else if (value.isBool()) {
qDebug().noquote() << indent + (value.toBool() ? "true" : "false");
qInfo().noquote() << indent + (value.toBool() ? "true" : "false");
} else if (value.isNull()) {
qDebug().noquote() << indent + "null";
qInfo().noquote() << indent + "null";
}
}
}