Extend Ctrl+C functionality to requiredByItem and customPathItem
Refactor appendResource to remove redundant insertChildren calls Signed-off-by: Dylan Schooner <dschooner05@gmail.com>
This commit is contained in:
@@ -461,13 +461,12 @@ void ResourceUpdateDialog::appendResource(CheckUpdateTask::Update const& info, Q
|
|||||||
auto requiredByItem = new QTreeWidgetItem(item_top);
|
auto requiredByItem = new QTreeWidgetItem(item_top);
|
||||||
if (requiredBy.length() == 1) {
|
if (requiredBy.length() == 1) {
|
||||||
requiredByItem->setText(0, tr("Required by: %1").arg(requiredBy.back()));
|
requiredByItem->setText(0, tr("Required by: %1").arg(requiredBy.back()));
|
||||||
|
requiredByItem->setData(0, Qt::UserRole, requiredBy.back());
|
||||||
} else {
|
} else {
|
||||||
requiredByItem->setText(0, tr("Required by:"));
|
requiredByItem->setText(0, tr("Required by:"));
|
||||||
auto i = 0;
|
|
||||||
for (auto req : requiredBy) {
|
for (auto req : requiredBy) {
|
||||||
auto reqItem = new QTreeWidgetItem(requiredByItem);
|
auto reqItem = new QTreeWidgetItem(requiredByItem);
|
||||||
reqItem->setText(0, req);
|
reqItem->setText(0, req);
|
||||||
reqItem->insertChildren(i++, { reqItem });
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -68,18 +68,14 @@ void ReviewMessageBox::appendResource(ResourceInformation&& info)
|
|||||||
filenameItem->setText(0, tr("Filename: %1").arg(info.filename));
|
filenameItem->setText(0, tr("Filename: %1").arg(info.filename));
|
||||||
filenameItem->setData(0, Qt::UserRole, info.filename);
|
filenameItem->setData(0, Qt::UserRole, info.filename);
|
||||||
|
|
||||||
auto childIndx = 0;
|
|
||||||
itemTop->insertChildren(childIndx++, { filenameItem });
|
|
||||||
|
|
||||||
if (!info.custom_file_path.isEmpty()) {
|
if (!info.custom_file_path.isEmpty()) {
|
||||||
auto customPathItem = new QTreeWidgetItem(itemTop);
|
auto customPathItem = new QTreeWidgetItem(itemTop);
|
||||||
customPathItem->setText(0, tr("This download will be placed in: %1").arg(info.custom_file_path));
|
customPathItem->setText(0, tr("This download will be placed in: %1").arg(info.custom_file_path));
|
||||||
|
customPathItem->setData(0, Qt::UserRole, info.custom_file_path);
|
||||||
itemTop->insertChildren(1, { customPathItem });
|
|
||||||
|
|
||||||
itemTop->setIcon(1, QIcon(QIcon::fromTheme("status-yellow")));
|
itemTop->setIcon(1, QIcon(QIcon::fromTheme("status-yellow")));
|
||||||
itemTop->setToolTip(
|
itemTop->setToolTip(
|
||||||
childIndx++,
|
1,
|
||||||
tr("This file will be downloaded to a folder location different from the default, possibly due to its loader requiring it."));
|
tr("This file will be downloaded to a folder location different from the default, possibly due to its loader requiring it."));
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -87,23 +83,19 @@ void ReviewMessageBox::appendResource(ResourceInformation&& info)
|
|||||||
providerItem->setText(0, tr("Provider: %1").arg(info.provider));
|
providerItem->setText(0, tr("Provider: %1").arg(info.provider));
|
||||||
providerItem->setData(0, Qt::UserRole, info.provider);
|
providerItem->setData(0, Qt::UserRole, info.provider);
|
||||||
|
|
||||||
itemTop->insertChildren(childIndx++, { providerItem });
|
|
||||||
|
|
||||||
if (!info.required_by.isEmpty()) {
|
if (!info.required_by.isEmpty()) {
|
||||||
auto requiredByItem = new QTreeWidgetItem(itemTop);
|
auto requiredByItem = new QTreeWidgetItem(itemTop);
|
||||||
if (info.required_by.length() == 1) {
|
if (info.required_by.length() == 1) {
|
||||||
requiredByItem->setText(0, tr("Required by: %1").arg(info.required_by.back()));
|
requiredByItem->setText(0, tr("Required by: %1").arg(info.required_by.back()));
|
||||||
|
requiredByItem->setData(0, Qt::UserRole, info.required_by.back());
|
||||||
} else {
|
} else {
|
||||||
requiredByItem->setText(0, tr("Required by:"));
|
requiredByItem->setText(0, tr("Required by:"));
|
||||||
auto i = 0;
|
|
||||||
for (auto req : info.required_by) {
|
for (auto req : info.required_by) {
|
||||||
auto reqItem = new QTreeWidgetItem(requiredByItem);
|
auto reqItem = new QTreeWidgetItem(requiredByItem);
|
||||||
reqItem->setText(0, req);
|
reqItem->setText(0, req);
|
||||||
reqItem->insertChildren(i++, { reqItem });
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
itemTop->insertChildren(childIndx++, { requiredByItem });
|
|
||||||
ui->toggleDepsButton->show();
|
ui->toggleDepsButton->show();
|
||||||
m_deps << itemTop;
|
m_deps << itemTop;
|
||||||
}
|
}
|
||||||
@@ -112,8 +104,6 @@ void ReviewMessageBox::appendResource(ResourceInformation&& info)
|
|||||||
versionTypeItem->setText(0, tr("Version Type: %1").arg(info.version_type));
|
versionTypeItem->setText(0, tr("Version Type: %1").arg(info.version_type));
|
||||||
versionTypeItem->setData(0, Qt::UserRole, info.version_type);
|
versionTypeItem->setData(0, Qt::UserRole, info.version_type);
|
||||||
|
|
||||||
itemTop->insertChildren(childIndx++, { versionTypeItem });
|
|
||||||
|
|
||||||
ui->modTreeWidget->addTopLevelItem(itemTop);
|
ui->modTreeWidget->addTopLevelItem(itemTop);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user