Skip to content

Conversation

yaira2
Copy link
Member

@yaira2 yaira2 commented Apr 17, 2020

Fixed an issue where opening folder in the GridView layout would sometimes crash the app.

@tsvietOK
Copy link
Contributor

@yaichenbaum this PR not fixes first folder opening, it fixes crash on other folder opening(except first)

@yaira2 yaira2 changed the title Fixed first folder in gridview layout not opening Fixes crash when opening folders in GridView Apr 17, 2020
@yaira2 yaira2 changed the title Fixes crash when opening folders in GridView Fixed crash when opening folders in GridView Apr 17, 2020
@ghost ghost added the needs - code review label Apr 17, 2020
@tsvietOK
Copy link
Contributor

By the way, dragndrop is not working(drop part).

@tsvietOK
Copy link
Contributor

@yaichenbaum dragndrop fixes: this line replace "pastedItems" with "itemsToPaste". I think we can add this change to this PR.

@yaira2
Copy link
Member Author

yaira2 commented Apr 19, 2020

@tsvietOK Have you tested that change and confirmed that it works?

@tsvietOK
Copy link
Contributor

@yaichenbaum Yes. Before this change if you try to move one folder onto another folder it copying it and after removing just copied folder. Now it works

@tsvietOK
Copy link
Contributor

Logically, i don't think it is a good idea to delete just moved files:)

@yaira2 yaira2 requested a review from lukeblevins April 19, 2020 01:57
@yaira2 yaira2 merged commit 9d5a593 into develop Apr 19, 2020
@yaira2 yaira2 deleted the FixedOpeningFolder branch April 24, 2020 19:43
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.

3 participants