Skip to content

Conversation

@gave92
Copy link
Member

@gave92 gave92 commented Aug 26, 2020

Fix #1767

This PR allows to eject a connected USB drive without closing the app. It's sufficient to close the tab or navigate away.

Please check if you agree on how to detect when a tab was closed, I don't know much about how tabs are handled.

@ghost ghost added the ready for review Pull requests that are ready for review label Aug 26, 2020
@yaira2 yaira2 requested a review from tsvietOK August 26, 2020 20:10
@tsvietOK
Copy link
Contributor

Getting an exception when trying to close tab
image
Steps:
Open app
Open one more tab(don't select it)
Close created tab

@gave92 gave92 marked this pull request as draft August 26, 2020 23:42
@gave92 gave92 marked this pull request as ready for review August 26, 2020 23:50
@yaira2 yaira2 added ready to merge Pull requests that are approved and ready to merge and removed ready for review Pull requests that are ready for review labels Aug 27, 2020
@yaira2 yaira2 changed the title Close filesystem watcher on tab closing Fixed an issue where external devices cannot be removed while app is opened Aug 27, 2020
@yaira2 yaira2 merged commit d20ecff into files-community:master Aug 27, 2020
@gave92 gave92 deleted the cleanup_tabs branch August 27, 2020 04:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready to merge Pull requests that are approved and ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

External device cannot be removed while app is opened

3 participants