filebrowser icon indicating copy to clipboard operation
filebrowser copied to clipboard

feat: Add context menu

Open omalk98 opened this issue 1 year ago • 23 comments

New PR for #2871 updated to Vue3 https://github.com/filebrowser/filebrowser/pull/2871

omalk98 avatar Jul 01 '24 07:07 omalk98

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

github-actions[bot] avatar Aug 01 '24 01:08 github-actions[bot]

Bump

omalk98 avatar Aug 01 '24 07:08 omalk98

Please add 2FA with a TOTP.

vihatsoft avatar Aug 05 '24 07:08 vihatsoft

I found a bug in the menu implementation. When you click the right button on a file for the second time, the file becomes unselected and then the menu opens for the entire folder.

https://github.com/user-attachments/assets/5751a903-899d-42bb-b334-dcdcc47b21f7

o1egl avatar Aug 17 '24 17:08 o1egl

I found a bug in the menu implementation. When you click the right button on a file for the second time, the file becomes unselected and then the menu opens for the entire folder.

https://github.com/user-attachments/assets/5751a903-899d-42bb-b334-dcdcc47b21f7

This was an International feature since originally, a right click would still select/deselect an item on the list. If this isn't a feature that is preferred to have, I will address it in a commit soon.

omalk98 avatar Aug 20 '24 11:08 omalk98

I found a bug in the menu implementation. When you click the right button on a file for the second time, the file becomes unselected and then the menu opens for the entire folder.

screen.recording.mov

This was an International feature since originally, a right click would still select/deselect an item on the list. If this isn't a feature that is preferred to have, I will address it in a commit soon.

I'd say it should select but not unselect a file. Otherwise, it's just confusing and looks buggy

o1egl avatar Aug 21 '24 11:08 o1egl

First of all, thank you very much for your contribution. The context menu feature is very helpful to us and is very convenient to use. I immediately patched my code, but I found several problems during use:

  1. When right-clicking in a blank space, if no file or folder is selected, download and info will appear, and it will conflict with the right-click of the browser;
  2. When switching views, there will be only one column;
  3. The mouse position in the pop-up box that appears when right-clicking is too close, you can move it a little to one side;
  4. When checking the Use single clicks to open files and directories option, the user experience is slightly worse;

If you have any questions, please feel free to contact me. Thank you again for your contribution.

Screenshot Screenshot 2024-08-29 at 15-59-58 filebrowser - Files - File Browser

Code diff context_menu.diff.txt

ultwcz avatar Aug 29 '24 08:08 ultwcz

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

github-actions[bot] avatar Sep 29 '24 02:09 github-actions[bot]

Bump, will get to it soon

omalk98 avatar Sep 30 '24 15:09 omalk98

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

github-actions[bot] avatar Nov 01 '24 02:11 github-actions[bot]

Bump

omalk98 avatar Nov 03 '24 17:11 omalk98

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

github-actions[bot] avatar Dec 04 '24 02:12 github-actions[bot]

Bump

omalk98 avatar Dec 06 '24 14:12 omalk98

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

github-actions[bot] avatar Jan 07 '25 01:01 github-actions[bot]

Bump

omalk98 avatar Jan 07 '25 02:01 omalk98

A lot of work seems to have been put into this.

Mysterius avatar Jan 19 '25 09:01 Mysterius

@o1egl Finally completed this issue, please let me know if any updates are still required

omalk98 avatar Feb 27 '25 05:02 omalk98

@ultwcz hope this commit resolves your issues, lmk

omalk98 avatar Feb 27 '25 05:02 omalk98

@ultwcz hope this commit resolves your issues, lmk

Ok, I will test it.

Thank you.

ultwcz avatar Feb 27 '25 07:02 ultwcz

bump

domme1908 avatar Mar 01 '25 19:03 domme1908

bump

Vidariondr avatar Apr 22 '25 19:04 Vidariondr

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

github-actions[bot] avatar May 23 '25 02:05 github-actions[bot]

Bump

omalk98 avatar May 23 '25 02:05 omalk98

@o1egl would be nice to get this merged soon!

omalk98 avatar Aug 02 '25 22:08 omalk98

@omalk98 I'd recommend reading the current project status when it comes to adding new features: https://github.com/filebrowser/filebrowser#project-status

hacdias avatar Aug 03 '25 11:08 hacdias

@omalk98 I'd recommend reading the current project status when it comes to adding new features: https://github.com/filebrowser/filebrowser#project-status

Oh, that's a shame. It would have been great to get this merged.

albinmedoc avatar Aug 12 '25 17:08 albinmedoc