News:

SMF 2.1.4 has been released! Take it for a spin! Read more.

Main Menu

[4843] Previous/next links usability issue?

Started by Sverre, January 16, 2011, 09:22:58 AM

Previous topic - Next topic

Sverre

This might be more of a personal annoyance than a bug, due to the fact that I make heavy use of Opera's smart Fast Forward/Rewind shortcuts (something which is also available as an extension in other popular browsers) to speed up my browsing, but I'd greatly prefer that the previous and next (<link rel="prev"> and <link rel="next">) links in Dispaly.template.php were linked to the previous and next page in multi page topics instead of (the last page in) the next topic. The way it currently works is pretty much useless in my opinion, and this will become even more apparent if/when background preloading becomes a household feature in browsers - it's already available with the Fasterfox extension in Firefox, and Chrome is apparently in the process of implementing it as a standard feature.

Aleksi "Lex" Kilpinen

A similar subject has actually been raised and discussed before within team and beta testers, and I believe it was agreed that having links separately for next and previous page is a good idea. Though thee idea has not been implemented in practise yet to my knowledge, it has been taken into concideration at least.
Slava
Ukraini!
"Before you allow people access to your forum, especially in an administrative position, you must be aware that that person can seriously damage your forum. Therefore, you should only allow people that you trust, implicitly, to have such access." -Douglas

How you can help SMF

Spoogs

Meh... there's a mod for that. I actually like the previous/next topic and use it quite often

Sverre

Quote from: Spoogs on January 16, 2011, 10:30:00 AM
Meh... there's a mod for that.

Maybe I didn't make myself clear in my original post, but it's not really the rendered/dispalyed previous/next links I care about. It's the <link rel="prev"> and <link rel="next"> links found in the document's head element. Ref., http://www.w3.org/TR/html401/struct/links.html#h-12.1.2.

Spoogs

Seems you actually did... I just missed the point ;) but I get ya now

Antechinus

Ah. I actually changed this on a 1.1.x site but it's equally easy to do for 2.0.

http://www.simplemachines.org/community/index.php?topic=210212.msg2548628#msg2548628

It's defo worth thinking about for 2.1. I wanted to do it in 2.0 but somebody started grumbling about "scope creep". ;D

SlammedDime

Quote from: Sverre on January 16, 2011, 09:22:58 AMthis will become even more apparent if/when background preloading becomes a household feature in browsers - it's already available with the Fasterfox extension in Firefox, and Chrome is apparently in the process of implementing it as a standard feature.
Just wanted to comment on this bit... SMF actually detects prefetching in a number of places and throws errors at the browser for doing it...

matt@matt-netbook:/var/www/smf-trunk/public/Sources$ grep -in "PREFETCH" *
Display.php:90: // Not only does a prefetch make things slower for the server, but it makes it impossible to know if they read it.
Display.php:91: if (isset($_SERVER['HTTP_X_MOZ']) && $_SERVER['HTTP_X_MOZ'] == 'prefetch')
Display.php:94: header('HTTP/1.1 403 Prefetch Forbidden');
Load.php:796: // If it's a prefetching agent or we're requesting an attachment.
Load.php:797: if ((isset($_SERVER['HTTP_X_MOZ']) && $_SERVER['HTTP_X_MOZ'] == 'prefetch') || (!empty($_REQUEST['action']) && $_REQUEST['action'] === 'dlattach'))
MessageIndex.php:139: // We can't know they read it if we allow prefetches.
MessageIndex.php:140: if (isset($_SERVER['HTTP_X_MOZ']) && $_SERVER['HTTP_X_MOZ'] == 'prefetch')
MessageIndex.php:143: header('HTTP/1.1 403 Prefetch Forbidden');
Recent.php:442: // Prefetching + lots of MySQL work = bad mojo.
Recent.php:443: if (isset($_SERVER['HTTP_X_MOZ']) && $_SERVER['HTTP_X_MOZ'] == 'prefetch')
Search.php:272: // No, no, no... this is a bit hard on the server, so don't you go prefetching it!
Search.php:273: if (isset($_SERVER['HTTP_X_MOZ']) && $_SERVER['HTTP_X_MOZ'] == 'prefetch')
Security.php:568: if (isset($_SERVER['HTTP_X_MOZ']) && $_SERVER['HTTP_X_MOZ'] == 'prefetch')
Security.php:683: // Make sure a page with session check requirement is not being prefetched.
Security.php:684: if (isset($_SERVER['HTTP_X_MOZ']) && $_SERVER['HTTP_X_MOZ'] == 'prefetch')
SlammedDime
Former Lead Customizer
BitBucket Projects
GeekStorage.com Hosting
                      My Mods
SimpleSEF
Ajax Quick Reply
Sitemap
more...
                     

Joshua Dickerson

This has been requested many times before and I agree with it.
Come work with me at Promenade Group



Need help? See the wiki. Want to help SMF? See the wiki!

Did you know you can help develop SMF? See us on Github.

How have you bettered the world today?

aw06

Yezzir  .. This problem annoys me as well .. don't know why those links are pointed to the last/newest post made in the next topic  >:(

Put 4 links i guess, becasue some topics i see where people already follow and would want to land on the newest reply, but for new users it's major annoying, people complain to me about it all the time..

prev | next <<-->> prev new | next new
:: ShopinJA.com Powered by SMF 1.1.19 | Ig-Oh Theme by Koni | 70 Rock Solid Error Free Mods | Many Custom Edits & Tweaks ::
- Host Unlimited Websites - Free Website Builder & Templates - Unlimited Disk Space & Bandwidth


Antechinus

Yes it makes sense to me (which is why I did it as custom code at the site I run) but if you don't want to wait for it in 2.1 you'll have to convince Norv to put it in a patch. ;)

MrPhil

This appears to be an increasing problem, as some browsers are apparently using the <link rel='prev|next'> tags to prefetch pages, and barfing with a 403 error. Whether or not you want to argue about browser bugs, SMF is definitely using the tag(s) incorrectly anyway, as they are supposed to link to the previous and next pages of this topic, not to the previous and next topics! If a topic fits on a single page, there must be NO <link rel> tag. Only if you're on page 2 or higher should there be a <link rel='prev'>, and only if there is a higher page should there be a <link rel='next'>. This is a bug that is causing failures for users, and the <head> section <link rel> tags should be removed for now (in 2.0.5 and 2.1) if they cannot be easily fixed (i.e., point to pages when needed).

The prev_next Query String value, and the tiny "previous" and "next" links, appear to be correct, although their labels should be changed to "previous topic" and "next topic". They should probably be moved to near the "Pages" link(s) to put all the up/down prev/next navigation together in one place. I don't know if the theme authors want to burn the space, but some forums have a nice and intuitive:

                             back to board
                             previous topic
<< 1   < 2 previous page       Page [  3]       next page 4 >   last page >>
                               next topic

Stüldt Håjt

I'm going to bump this thread. Is this fixed for the new version?

Arantor

It looks as though it has not because it's been tracked on Github under https://github.com/SimpleMachines/SMF2.1/issues/851 and AFAIK is still a problem. There is a concerted effort to move all reporting to Github and any further discussion really should take place there.

Advertisement: