filestash icon indicating copy to clipboard operation
filestash copied to clipboard

[bug] Download option disappears when viewing search results

Open Ewokosaur opened this issue 1 year ago • 2 comments

Description of the bug

The download option which normally appears when selecting checkboxes of files and folders disappears when using the search function.

I am self-hosting FileStash using docker behind an nginx reverse proxy.

Filestash is connected to an S3 bucket and displays all files and folders normally.

 

Step by step instructions to reproduce the bug:

Input any search matching a file or folder name and view the results.

Can you replicate that error from the demo?

Yes, this behavior is present in the demo as well.

Observed behavior

Download option does not appear, but I am still able to select multiple files/folders using the checkboxes.

Expected behavior

Due to the level of functionality observed when not searching, I believe that a download option should be present for the results of a search, allowing users to locate items and download them using multiselect.

If this is not intended functionality, no worries :)

While viewing search results: with_search

While viewing files normally: without_search

Ewokosaur avatar Feb 09 '24 23:02 Ewokosaur

Looking at the site through dev tools, it seems to be something related to the div class "menubar no-select search_focus" which does not allow the download button to appear, despite showing up as a span once I have selected something while searching.

If I change the class back to "menubar no-select" after having selected the folders/files I searched up I am then able to view the download button.

search_focus no_search_focus

Ewokosaur avatar Feb 12 '24 20:02 Ewokosaur

I was able to temporarily work around this using the following CSS:

.button-download { display: inline-block !important; visibility: visible !important; opacity: 1 !important; }

Using this temporary fix, I am now able to select and download files from search results, but I want to leave this up so it can be addressed permanently in a future patch.

Thanks! :)

Ewokosaur avatar Feb 12 '24 22:02 Ewokosaur