Is it possible to add an anchor link to every picture? This way, when you click next or previous, you don't have to scroll down a bit to see the whole picture.

The way I see it, it could be a setting in the admin panel of the module (Link to anchor on previous/next: yes/no)

CommentFileSizeAuthor
#5 photos.links_whith_anchor.diff3.24 KBberdir

Comments

eastcn’s picture

eastcn commented

I do not understand.
What kind of anchor, in what location?

SnowFredje’s picture

SnowFredje commented

In my case now, I have a rather large header on my site. When I click next or previous, the next or previous page comes up but is displayed at the top. That way, I have to scroll down every time again to see the picture.

If you would implement the possibility of an anchor located at the image, one could click next and have the page immediately centered on the picture.

Like this:
/photos/image/1693#photo

eastcn’s picture

eastcn commented

ajax picture view, this is a good idea, but it was not planned, at least for the time being.
Try "dfgallery", I think it is better than ajax browsing

SnowFredje’s picture

SnowFredje commented

It has nothing to do with AJAX, nor with Flash. Just, when you go to the next or previous page, you get centered on the picture again. You can do that with anchors: http://www.december.com/html/x1/element/a.html

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
StatusFileSize
new photos.links_whith_anchor.diff3.24 KB

I've created a simple patch which adds an anchor and updates the previous and next links (and also corrects the spelling of previous) against DRUPAL-6--2-5-BETA1.

There is no option to choose the behaviour at this time, if needed, I will add this later.

Please test the patch and tell me if it does work for you.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Active » Needs review
Babalu’s picture

Babalu commented

works the patch on version 6.x-1.3 too ?

berdir’s picture

berdir
Primary language German
Location Switzerland
commented

You can't patch it automaticaly because of changes in the text. It should however easily be possible to apply the patch manually, the line numbers haven't changed.

It is actually just one new line and 4 extended links.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs review » Fixed
Anonymous’s picture

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

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