User gallery page

[Christopher Allan Webber]

Added to our growing list of user pages should be the user gallery, hosted at /u/{username}/gallery/

This should have a paginated list of all works done by this author.

See also on pagination: http://bugs.foocorp.net/issues/329

[Elrond]

Dones't that exist in some way as /u/{user}/ already?

Should this be moved on its own /u/{user}/gallery/ thing then?

[Christopher Allan Webber]

Yup, it does, and I think that the paginated stuff should actually be on the gallery page. We'll need to use the explicit paginator url generation route so you can pass in the */gallery/ url explicitly for that but it should work.

[Jef van Schendel]

What will these pages look like? As I understand it now, we'll have two pages, a user page and a user's gallery page. The latter will be simply all the images that belong to that account, and the former will have only a few images as examples, together with other user information like a bio, albums and such.

Is that about right?

[Christopher Allan Webber]

That's exactly right.

[Caleb Davis]

0001-copied-user_home-for-a-user_gallery.patch

[Caleb Davis]

Hi all. Here's my $0.02.

[Christopher Allan Webber]

Hiya Caleb / gullydwarf,

It looks like all that's attached here currently is a template? We should also have a modification to the user_pages/views.py and user_pages/routing.py. Maybe you missed it while generating the patch? Ping me on IRC if you need help doing this.

[Caleb Davis]

yeah, can you tell I've never used git before? The missing commit -- [https://gitorious.org/_{cfdv/mediagoblin/cfdvs-mediagoblin/commit/64511d83e4e287ba969e43400c9041ebc5d9ec35](https://gitorious.org/}cfdv/mediagoblin/cfdvs-mediagoblin/commit/64511d83e4e287ba969e43400c9041ebc5d9ec35)

[Christopher Allan Webber]

So a good start. Some thoughts, here:

• The gallery view doesn't work after merging in the mainline master because Response is no longer imported. See moving things over to the new render_to_response() function
• It would be good if there was a link to the gallery from the user homepage over the "first page" of that gallery.
• The pagination on the user page doesn't make sense anymore, but I'm not sure what the right solution is?
  • Keep the pagination links, but link to pages on the gallery (this is possible if we use paginator.get_page_url_explicit() and pass in the particular url for the gallery, and also requires that we make smart adjustments to all of pagination.html, and maybe even object_gallery.html and gallery.html... and though I can think of how to do it it's awfully hard to describe in a ticket)
  • Just link to the gallery in case you want to see more pictures (this doesn't seem intuitive at this present stage)
  • Screw it, just keep the present pagination (this doesn't seem sensible either)

I mean, eventually there won't be pagination on the user homepage because the images that appear there will be "ajaxy" with a link to the gallery for actual pagination, but we aren't there.

So here's what I think. Caleb, can you do #1 and #2 and pass back to me? If people have commentary on #3 I'm happy to hear it. Otherwise I'll implement option 3.1.

[Caleb Davis]

Okay, thanks. I switched over to render_to_response() with my latest commit. I'm still trying to figure out pagination for #2

[Caleb Davis]

okay, and I added the link to the gallery from the user homepage. I placed it next to the '(logout)' link. Hopefully that does it for #1 and #2. [https://gitorious.org/_{cfdv/mediagoblin/cfdvs-mediagoblin/commit/d0b8aafdaa1b5bc6eea54bd355b9a2adeef99982](https://gitorious.org/}cfdv/mediagoblin/cfdvs-mediagoblin/commit/d0b8aafdaa1b5bc6eea54bd355b9a2adeef99982)

[Caleb Davis]

now there is no link back to the user's home page from the gallery. I'm thinking the header_top_right would go '{user}'s account home gallery (logout)', with appropriate links and bold white type for navigation and 'you are here', respectively.

[Caleb Davis]

I added a link back to the user page. I don't know if I can do anything more on this until we make some navigation decisions.

Here's my #330 branch - [https://gitorious.org/_{cfdv/mediagoblin/cfdvs-mediagoblin/commits/is330](https://gitorious.org/}cfdv/mediagoblin/cfdvs-mediagoblin/commits/is330)

[Christopher Allan Webber]

Sounds great. I'll look soon, hopefully tonight!

[Christopher Allan Webber]

Merged. I should be able to handle the rest of this soonish.

[Caleb Davis]

I fixed some whitespace issues that were annoying me. No functional changes - [https://gitorious.org/_{cfdv/mediagoblin/cfdvs-mediagoblin/commit/e8fda91bc2326911ea9a0197e595bc333540c282](https://gitorious.org/}cfdv/mediagoblin/cfdvs-mediagoblin/commit/e8fda91bc2326911ea9a0197e595bc333540c282)

[Christopher Allan Webber]

Thanks Caleb, I merged that whitespace change.

[Elrond]

As discussed on irc, setting to 0.0.3. Chris wants to handle the remaining stuff.

[Christopher Allan Webber]

I handled the remaining stuff. ;)

The pagination is there still but it points to the user's gallery instead.

[Will Kahn-Greene]

The original url for this bug was ​http://bugs.foocorp.net/issues/330 .