Gallery link no longer positions gallery on the selected photo

denisegoldberg

The gallery link on the Get a link page no longer takes you to the selected photo within the gallery. Instead, you are presented wtih the first photo in the gallery.

I've been using this link in my blog for quite a while, and the behavior changed recently (sorry, I can't put my finger on the actual date...)

Here's a link that I copied from Get a link... Gallery:

http://denise.smugmug.com/gallery/4804829_fg6MW//392571750_CBsZB

Following this link takes you to the first photo in the gallery, not the photo I selected.

If I change the link by replacing the // with #, then the link takes me to the selected photo within my smugmug-style gallery:

http://denise.smugmug.com/gallery/4804829_fg6MW#392571750_CBsZB

I just wandered through my blog and found this form of a link from a post on September 18th: http://denise.smugmug.com/gallery/6004803_nYkEi/1/375382491_t9u5E. This was copied directly from the Get a link screen, and this one also takes the viewer to the specified photo. Maybe this provides somewhat of a timeframe for this changed behavior.

Am I missing something here? It appears this behavior was changed recently. Can it be restored to the previous behavior, or will I need to (continue to) alter the link so if someone clicks on a photo in my blog they will be taken to the specified photo in my gallery?

--- Denise

denisegoldberg

bump

Andy

Hi Denise, I asked Baldy, I'm not sure if/how/when/ it changed.

jfriend

Andy wrote:

Hi Denise, I asked Baldy, I'm not sure if/how/when/ it changed.

Andy, this looks like a simple coding mistake. When you ask for the link to the gallery, it's giving you a link to:

http://jfriend.smugmug.com/gallery/5608869_vwzCG//344291068_HdnTo

which doesn't make sense. The double slashes aren't a form of URL that Smugmug uses anywhere. It should either be:

http://jfriend.smugmug.com/gallery/5608869_vwzCG

which is a link to just the gallery (no particular starting image), or it should be:

http://jfriend.smugmug.com/gallery/5608869_vwzCG#344291068_HdnTo

which will take you to the current gallery and start out on the current image in that gallery. My guess is that it's supposed to be the latter.

jfriend

jfriend wrote:

Andy, this looks like a simple coding mistake. When you ask for the link to the gallery, it's giving you a link to:

http://jfriend.smugmug.com/gallery/5608869_vwzCG//344291068_HdnTo

which doesn't make sense. The double slashes aren't a form of URL that Smugmug uses anywhere. It should either be:

http://jfriend.smugmug.com/gallery/5608869_vwzCG

which is a link to just the gallery (no particular starting image), or it should be:

http://jfriend.smugmug.com/gallery/5608869_vwzCG#344291068_HdnTo

which will take you to the current gallery and start out on the current image in that gallery. My guess is that it's supposed to be the latter.

Andy, is there any response that this is indeed a bug that should be fixed (and like a pretty trivial one)? It definitely doesn't work as well as it used to and sending out links with double slashes in them just looks completely wrong. It might still sort of work today, but it could very easily break in the future. Since it looks like it's a coding error and simple to fix, I just don't want this to fall through the cracks.

denisegoldberg

What are the chances of getting this fixed?

The gallery link includes the photo link, yet the photo is not shown because the URL is incorrect.

This is a link from the Gallery entry of the Gallery links box:

http://denise.smugmug.com/gallery/6145242_yiNwf//387020598_WawNF

It takes me to the first photo in the gallery.

If I change the URL to replace the // with a #, the link takes me to the designated photo within the gallery. So this link works as desired:

http://denise.smugmug.com/gallery/6145242_yiNwf#387020598_WawNF

What's the point of a gallery link with the photo specified if it doesn't take the viewer to the photo?

I consider this to be a ver annoying bug. Can you please please fix it?

--- Denise

denisegoldberg

This has been corrected, verified just now.

Thanks smug heroes!

--- Denise

Andy

denisegoldberg wrote:

This has been corrected, verified just now.

Thanks smug heroes!

--- Denise

Yes, and release note http://blogs.smugmug.com/release-notes are updated

jfriend

Andy wrote:

Yes, and release note http://blogs.smugmug.com/release-notes are updated <img src="https://us.v-cdn.net/6029383/emoji/thumb.gif&quot; border="0" alt="" >

Andy, it might be useful to add two other things to the most recent release notes that JT put in to help customizers.

1) There's now a class="notLoggedIn" on the body tag when you or your viewer are not logged in. This makes it much easier to hide things when you are not logged in. For example, you can now hide the share button for your visitors (such that it only shows when you are logged in) with this CSS:

.notLoggedIn #shareButton {display:none;}

2) All bread crumb information is now enclosed in <div id="breadCrumbTrail"> so any modifications to the bread crumb (even in categories and sub-categories which wasn't the case before today) can now use breadCrumbTrail instead of breadCrumb. This is important because modifying the HTML in the breadCrumb DIV will disable the other buttons (slideshow, tools, share, etc...) in that div. Those operations can now operate safely on the breadCrumbTrail div without harming the buttons.