Skip to content

Connect finished signal of download thread only once.#670

Merged
dirkhh merged 1 commit intosubsurface:masterfrom
bstoeger:devel
Oct 16, 2017
Merged

Connect finished signal of download thread only once.#670
dirkhh merged 1 commit intosubsurface:masterfrom
bstoeger:devel

Conversation

@bstoeger
Copy link
Copy Markdown
Collaborator

@bstoeger bstoeger commented Oct 16, 2017

Connect finished signal of download thread only once in constructor of DownloadFromDCWidget instead of every time the Download / Cancel / Retry button is clicked.

Fixes minor nuisance: On repeated download attempts multiple message boxes were shown.

Connect finished signal of download thread only once in constructor of
DownloadFromDCWidget instead of every time the Download / Cancel / Retry
button is clicked.

Fixes minor nuisance: On repeated download attempts multiple massage
boxes were shown.

Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
@dirkhh
Copy link
Copy Markdown
Collaborator

dirkhh commented Oct 16, 2017

Thanks @bstoeger - this looks good to me. I don't expect any problems with the Travis check, so I'm planning to merge this PR once the checks have completed.

@dirkhh dirkhh merged commit 1fe76b0 into subsurface:master Oct 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants