Skip to content
This repository was archived by the owner on Dec 15, 2022. It is now read-only.

Can't move selected files to another folder #1033

Closed
Sogl opened this issue Feb 7, 2017 · 10 comments
Closed

Can't move selected files to another folder #1033

Sogl opened this issue Feb 7, 2017 · 10 comments

Comments

@Sogl
Copy link

Sogl commented Feb 7, 2017

Description

I found a bug when using file moving to another folder. When you select some files and move them with mouse, only first moved.

Steps to Reproduce

  1. Create new folder in Atom
  2. Select more than one file with Shift keyboard key
  3. Move selected files to new folder with mouse
  4. Only the first file moved

How it looks:
atom_move

As you can see, selection changed from 3 to 1 file after mouse drag.

Expected behavior: Move all selected files
Actual behavior: Only one file moved
Reproduces how often: 100%

Env:
Windows 7 x64
Atom 1.13.1 ia32
Tree-view 0.211.1

@winstliu
Copy link
Contributor

winstliu commented Feb 7, 2017

Thanks for the issue! I can reproduce this on Atom 1.15.0-dev-4eb3f22, Windows 10 x64.

@sergous
Copy link

sergous commented Feb 12, 2017

In mac 10.12.3, atom 1.14.1:

  1. selecting several files
  2. start drug files group
  3. only one file selected

@anderoonies
Copy link
Contributor

hey!
i had a PR up that did this a good while ago
i'm going to rewrite it since the codebase has changed significantly since then

@winstliu
Copy link
Contributor

Fixed by #1115.

@UziTech
Copy link
Contributor

UziTech commented Aug 31, 2017

@50Wliu pull request #1115 doesn't seem to fix this problem

#793 is supposed to fix it but the author seems to be MIA

@Ben3eeE Ben3eeE reopened this Aug 31, 2017
@Ben3eeE
Copy link
Contributor

Ben3eeE commented Aug 31, 2017

@UziTech Thank you 🙇‍♂️

@anderoonies Are you still interested in rewriting that PR?

@mccallum-sgd
Copy link

Also still having this problem on Windows 10 Home x64 (ver: 1709, build: 16299.309).
Atom: 1.25.0
Tree-view: 0.221.3

@Ben3eeE
Copy link
Contributor

Ben3eeE commented Mar 20, 2018

Closed via #1179

@UziTech Let me know if you think this should still be open for any reason 🙂

@mccallum-sgd This will be available in a future Atom release. We have to fix the tests on master before including these changes in an Atom release /cc: #1203

@Ben3eeE Ben3eeE closed this as completed Mar 20, 2018
@UziTech
Copy link
Contributor

UziTech commented Mar 20, 2018

This was fixed by #1179

@mccallum-sgd
Copy link

Ah okay my mistake, thanks guys!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

7 participants