There is no way to see per user albums. It would be good if per-user galleries were:
1.) in navigation menu
2.) on profile (for example on new tab)

CommentFileSizeAuthor
#7 add_per_user_albums_page_0.patch2.44 KBvhmauery
#3 add_per_user_albums_page.patch2.15 KBvhmauery

Comments

alippai’s picture

alippai commented
Title: User friendly per-user » User friendly per-user albums
alippai’s picture

alippai commented

I found user/acidfree, but this link can't be added to menu :S
?q=acidfree orginally what display's?

vhmauery’s picture

vhmauery commented
Status: Active » Needs review
StatusFileSize
new add_per_user_albums_page.patch2.15 KB

Try out this patch and tell me if it is what you had in mind.

vhmauery’s picture

vhmauery commented

oh. I forgot to mention how to use it. go to acidfree/per-user-albums and it will show you all the per-user albums.

alippai’s picture

alippai commented

Thanks for the per-user albums (only "problem" is with design - the grid elements have width 100%), the code is working. Now the user/acidfree is not reachable in menu. The other feature is displaying root album contents in profile ;)
Nice work...

alippai’s picture

alippai commented

The acidfree/per-user-albums displays Emtpy albums too.
It would be good if empty albums were not displayed - filter in SQL

vhmauery’s picture

vhmauery commented
StatusFileSize
new add_per_user_albums_page_0.patch2.44 KB

No, empty albums will just show up as empty. IMNSHO, a user interface that changes in that manner is a very poor design.

I realized after I had posted the patch that I had forgotten to add a link to the per-user album on the user profile page.

As for the 100% width thing, they show up fine in Firefox. Are you using IE? It uses the same rendering code as the album views, so this page should look the same as the other album pages.

Here is a new patch that has the user profile stuff (it turns out I had already implemented this, it was just broken).

alippai’s picture

alippai commented

There is a problem with pager/ with albums - there are users without albums :) Have a look at this

vhmauery’s picture

vhmauery commented
Status: Needs review » Fixed

The deleted per-user albums are fixed now. If a user deletes the album, it will be removed from the user profile in the list.

I committed the fix to CVS. Along with a bunch of other stuff.

vhmauery’s picture

vhmauery commented
Status: Fixed » Closed (fixed)