search

xpipe-io / xpipe

Your entire server infrastructure at your fingertips
https://xpipe.io
Apache License 2.0
2.3k stars 66 forks source link

FR: Option to move files in the integrated file manager #247

Closed MaggiWuerze closed 1 month ago

MaggiWuerze commented 1 month ago

Please add the option to move file, instead of copying them, in the integrated file manager /browser.

Either though a context menu item or through a modifier key + the drag option. Or both.

crschnick commented 1 month ago

So which modifier key would be best here?

MaggiWuerze commented 1 month ago

I mean usually you move by default and copy by pressing ctrl or something

crschnick commented 1 month ago

This is now possible in 9.2 with ALT, as that was the only key left without a function in that context: https://github.com/xpipe-io/xpipe/releases/tag/9.2

MaggiWuerze commented 1 month ago

Wow, that was fast. Thanks a lot

MaggiWuerze commented 1 month ago

Would you mind telling me what button that refers to on MacOS?

crschnick commented 1 month ago

That might be a little bit awkward, but it seems like I did not consider macOS not having the ALT button but instead the option button. I'm not sure whether that will work with the current implementation, I might need to change that to include option as well.

crschnick commented 1 month ago

If it does not work with the option key, you might have to wait until 9.3 so I can fix this properly. I use my Windows keyboard for my mac mini, so I wasn't aware of that.

crschnick commented 1 month ago

I'm a little bit confused. I tried it on my mac with the option key and it worked as expected. Not explicitly mentioning the option key into changelog it was an oversight, but functionally it should work.

Does it not work for you?