Hi,
I've made a patch for themeable output. I think users must decide how to show content. Code takken from previous version of module.

CommentFileSizeAuthor
#4 filebrowser_4.patch1.71 KBbeholder
#2 filebrowser_3.patch1.42 KBbeholder
filebrowser_2.patch1.42 KBbeholder

Comments

Susurrus’s picture

Susurrus commented
Status: Needs review » Needs work

I think this is a good patch and RTBC, though I think you should specify the ID attribute on the table, as there's no point to wrap the table in a div just to give it an ID.

beholder’s picture

beholder commented
Status: Needs work » Patch (to be ported)
StatusFileSize
new filebrowser_3.patch1.42 KB

ok. here is a new patch without wrapping div and with ID for table

Susurrus’s picture

Susurrus commented

the last thing would be a nice message on the page if no files are displayed. If nothing is shown, people will think there's an error instead of just an empty directory...

beholder’s picture

beholder commented
Version: 6.x-1.1-beta1 » 6.x-1.x-dev
StatusFileSize
new filebrowser_4.patch1.71 KB

OK, here is patch for 6.x-1.x-dev version with support for themes and message when directory is empty.

Susurrus’s picture

Susurrus commented
Status: Patch (to be ported) » Fixed

Fixed, though I changed the patch slightly.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.