add a named anchor to pre/next links that load top of page at the image node content [#550900]

Hope I said that right. When navigating through photos in default mode. The photo is many time below visible screen level, scroll page up to see whole photo. Then click next(next photo) and the page loads (normally) which hides lower portion of photo again.

Many people may have huge resolution monitors which might make this not necessary.
This is not a big deal, just a possible refinement.

Comments

Comment #1

Tally CreditAttribution: Tally commented 17 August 2009 at 05:42

What you need is an Ajax type control that loads only the content and not the whole page. You could do this with the pager in Views.

I have several blocks created in views where the contents are paged independently of the whole page. One block shows all galleries for a user and another shows all images (thumbnails) for a user. If the user has more images than can be seen on his profile page, the views pager uses Ajax to change only the images without having to render the complete page.

Comment #2

dbeall CreditAttribution: dbeall commented 17 August 2009 at 06:39

would you like to share your views code? It would be great to have a few more views blocks for people to import.
It would neat to post it in the Style, Views and Themeing handbook page. I do ok with views, but the arguments UI gets the best of me.
We have code posted for latest photo, most recent photo blocks and gallery list table view.

Comment #3

dbeall CreditAttribution: dbeall commented 17 August 2009 at 21:47

I have been thinking about this(my opinion).
The target attribute would be an out of the box feature that everyone can enjoy.

Many people that use Drupal are New to Drupal and looking for a photo gallery. They may not have views or don't know it exists yet, let alone be able to figure it out. It is all too clear what my first ~~day~~ ~~month~~ 3 months was with Drupal. It is easy for a php or views wizard to stick features in the way they want, but for the rest of the people, this can be a huge undertaking.

In my testing of photo galleries over the last year, I noticed that project/photo had this feature. I thought it was a nice touch.

All I am saying, is please consider making some of these little things part of the base install.
Maybe consider it part of usability, or just one more reason to install this one and keep it.

Comment #4

dbeall CreditAttribution: dbeall commented 9 December 2009 at 06:59

Component:

Miscellaneous

» Code

I like it !!
Justintime came up with the code to make this happen..

When you get to the image node and click previous or next, the new page loads at the top of the image node, rather than the top of the web page..
I like it because you do not have to scroll each page to see the photo and it also exposes the fivestar rating widget better. I have the 5star widget at the bottom of the images.

Feedback is good..

See it work in these galleries
http://buckeyelake.org/photo-galleries

Comment #5

justintime CreditAttribution: justintime commented 10 December 2009 at 02:49

Title:	add a target atribute to pre/next that loads top of page at breadcrumb	» add a named anchor to pre/next links that load top of page at the image node content
Assigned:	Unassigned	» justintime
Status:	Active	» Fixed

Committed this feature. There is a checkbox at /admin/settings/node_gallery/settings with the name "Use Named Anchors on Image "Previous" and "Next" links?". When selected, the "Previous" and "Next" links will behave as above. The default behavior is to keep backwards compatibility by not selecting the checkbox.