Skip to content

Commit d2c4fa2

Browse files
committed
fix(ui): avoid duplicate drop failure warnings
* Tracks whether a detailed error dialog was already shown during drag and drop processing * Skips the generic "partially completed" warning when a specific error has already been presented to reduce redundant alerts
1 parent 3b4edfc commit d2c4fa2

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

src/ui/widgets/FileTreeView.cpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1089,6 +1089,7 @@ void FileTreeView::dropEvent(QDropEvent* event) {
10891089

10901090
QStringList result_paths;
10911091
bool all_ok = true;
1092+
bool detailed_error_shown = false;
10921093

10931094
for (const auto& source_path : source_paths) {
10941095
const QFileInfo source_info(source_path);
@@ -1104,6 +1105,7 @@ void FileTreeView::dropEvent(QDropEvent* event) {
11041105

11051106
if (!ok) {
11061107
all_ok = false;
1108+
detailed_error_shown = true;
11071109
continue;
11081110
}
11091111

@@ -1139,7 +1141,7 @@ void FileTreeView::dropEvent(QDropEvent* event) {
11391141
}
11401142
}
11411143

1142-
if (!all_ok) {
1144+
if (!all_ok && !detailed_error_shown) {
11431145
QMessageBox::warning(
11441146
this,
11451147
internal_drag ? tr("Move Partially Completed")

0 commit comments

Comments
 (0)