Fork me on GitHub
Subscribe 4

Ticket #175 (fixed enhancement)

Add <link rel=prev, next, authors, etc.

  • Created: 2010-11-05 00:01:33
  • Reported by: Freeman
  • Assigned to: Franz
  • Milestone: 1.4.4
  • Component: markup
  • Priority: normal

Please generate appropriate HTML header tags for the forum pages, like done on PunBB (seen on PunBB.ru).

History

Freeman 2010-11-05 00:31:05

  • Component changed from code to markup.

Reines 2010-11-08 16:01:02

  • Milestone set to 1.4.4.

Franz 2010-12-08 21:08:42

  • Owner set to Franz.

Franz 2010-12-09 13:35:39

Which relationships should I add?

I would consider

  • top

  • up

  • first

  • prev

  • next

  • last

  • author

  • search

Not sure whether all of them (e.g. author) make sense.

Franz 2010-12-15 20:43:52

I already added the pagination-related tags to viewtopic.php and viewforum.php. Plus "top" for all pages.

"search" and "author" get introduced in HTML5, so wouldn't make sense to add anyway (though they might be interpreted anyway - I've added a ticket to consider in the future).

Anything I forgot?

Franz 2010-12-15 20:48:50

  • Status changed from open to fixed.

I consider this as done. Correct me if I'm wrong.

quy 2010-12-16 01:03:18

"top" is not shown in viewtopic.php.

Should this be done to userlist.php too?

Franz 2010-12-16 08:33:44

Whoops, logic error. Thank you, fixed.

quy 2010-12-16 16:28:53

In viewforum.php, are these redundant?

<link rel="top" href="index.php" title="Forum index" />
<link rel="up" href="index.php" title="Forum index" />

To make "top" be at the top and group with the others, how about changing the code to:

array_insert($page_head, 0, '<link rel="top" href="index.php" title="'.$lang_common['Forum index'].'" />');

Franz 2010-12-16 16:35:14

I don't think they are redundant, as "up" doesn't mean "top" and we do the same thing for "prev" and "first", too.

I don't know whether the order is really that important...

quy 2010-12-16 16:40:52

I don' think order is important either, but it would be nice to group them together instead of "top" being by itself after the feed info.

Franz 2010-12-16 16:49:48

Fix it, if you want. I don't really care either way tongue

Freeman 2010-12-16 16:54:08

I found no "top" or "up" link types here, only "index".

Franz 2010-12-16 16:56:33

They were added back in 1.3, that's why I added them.
I suppose they weren't added because of HTML5, but rather because of the concept of Microdata, which existed (or was in use) before the HTML5 hype began.