Opened 15 years ago

Last modified 11 years ago

#208 closed defect (FIXED)

If there are no next and previous media, the navigation buttons should not appear

Reported by: Jef van Schendel Owned by: Caleb Davis
Priority: minor Milestone: 0.0.5
Component: programming Keywords:
Cc: Parent Tickets:

Description

Speaks for itself. If you only have one piece of media for whatever reason, the navigation still shows up. But both of the buttons are disabled, so they may just as well be gone completely.

I have no idea how hard this is to do though.

Attached is a screenshot.

Attachments (1)

Screenshot-16.png (98.2 KB ) - added by Jef van Schendel 15 years ago.
Screenshot-16.png

Download all attachments as: .zip

Change History (8)

by Jef van Schendel, 15 years ago

Attachment: Screenshot-16.png added

Screenshot-16.png

comment:1 by Jef van Schendel, 15 years ago

Typo in the ticket title: "not disappear" should of course be "not appear".

comment:2 by Caleb Davis, 15 years ago

Owner: set to Caleb Davis
Status: NewFeedback

Jef, like this?

?
.. raw:: html

   <del>

[https://gitorious.org/cfdv/mediagoblin/cfdvs-mediagoblin/commit/e3df834a8a22a45ba77940efbd083c7d5a23764e](https://gitorious.org/cfdv/mediagoblin/cfdvs-mediagoblin/commit/e3df834a8a22a45ba77940efbd083c7d5a23764e)

?
.. raw:: html

   </del>

EDIT: Chris pointed out you didn't want the Xs to go away completely, just in the case where the user had one media item only. Oops! How about this one?

[https://gitorious.org/cfdv/mediagoblin/cfdvs-mediagoblin/commit/0a100476b24d81355342cae5320e1a5a6c83014d](https://gitorious.org/cfdv/mediagoblin/cfdvs-mediagoblin/commit/0a100476b24d81355342cae5320e1a5a6c83014d)

comment:3 by Jef van Schendel, 15 years ago

Yes, that seems like it should work to me.

Thanks Caleb!

comment:4 by Caleb Davis, 15 years ago

Great, please review for merge. This was a bit of additional template logic.

comment:5 by Christopher Allan Webber, 15 years ago

Milestone: 0.0.5

comment:5 by Christopher Allan Webber, 15 years ago

Status: FeedbackClosed

Merged. Thanks!

comment:6 by Will Kahn-Greene, 14 years ago

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

Note: See TracTickets for help on using tickets.