search

IowaComputerGurus / dnn.SimpleFileList

The Simple File List module is a Free DotNetNuke module that will list all files within a specific folder under the specific portal folder within DNN for users to be able to download. The initial release of this module has been targeted at simple functionality. Future releases of the module will include more functionality and security.
MIT License
3 stars 1 forks source link

Make the filename the link; remove "Download" #7

Open dgingold opened 10 months ago

dgingold commented 10 months ago

I like the way you provide the file path as the download link. The core links module has a folder mode, but the link it generates is ?FileId= instead of "/portals//folders.../filename" like yours. with yours, if the file is identified by the browser, it will display it. perfect for showing a folder full of PDFs. Each will display instead of download. but row upon row of " Download" seems redundant; "Filename" looks better. While you're under the hood, making it work for folder providers, (which is the case with my current client), maybe you'd consider this change as well? (You could remove the header too. In fact, making it a ul would be the best. I can style it as needed)

mitchelsellers commented 10 months ago

@dgingold Right now this is designed for only NON-Secure folders thus not supporting folder providers. I could easily update it to support folder providers as desired.

We have also thought about implementing "Templates" in a loose manner for a "table", "List" or similar display.

The balance point is that there is not much functionality here. Thoughts?

dgingold commented 9 months ago

Re: "Right now..." yes, saw the request to make it support folder providers, so i thought "while you're under the hood..."

Re: "Templates" sure, but not if it causes rethinking and puts off updating. I'd keep it as-is if it means a quicker update. just changing the output from the table with the "File" header to a simple