Welcome to Geeklog Saturday, October 21 2017 @ 08:55 pm EDT


Status: offline

::Ben

Forum User
Full Member
Registered: 14/01/2005
Posts: 1569
Location:la rochelle, France
For a better SEO of your forum plugin 2.8.0

public_html/forum/index.php line 109

PHP Formatted Code

//SEO Title
$title = $LANG_GF01['FORUM'];
$forum_id = COM_applyFilter($_REQUEST['forum'],true);
if (isset($_REQUEST['forum'])) $title = stripslashes(DB_getItem($_TABLES['forum_forums'],'forum_name',"forum_id='{$forum_id}'")) ;
// Display Common headers
$display .= gf_siteHeader($title);


public_html/forum/viewtopic.php line 63

PHP Formatted Code
//SEO title
$result = DB_query("SELECT forum, pid, subject, forum_name FROM {$_TABLES['forum_topic']} LEFT JOIN {$_TABLES['forum_forums']} ON forum = forum_id WHERE id = '$showtopic'"); // <- new
list($forum, $topic_pid, $subject, $forum_name) = DB_fetchArray($result); // <- new
$subject = stripslashes($forum_name) . ' | '. $subject;


and for your forum plugin 2.9.0

public_html/forum/index.php line 1026

PHP Formatted Code

//SEO Title
$title = $LANG_GF01['FORUM'];
$forum_id = COM_applyFilter($_REQUEST['forum'],true);
if (isset($_REQUEST['forum'])) $title = stripslashes(DB_getItem($_TABLES['forum_forums'],'forum_name',"forum_id='{$forum_id}'")) ;
$display = gf_createHTMLDocument($display, $title);


public_html/forum/viewtopic.php line 66

PHP Formatted Code
//SEO title
$result = DB_query("SELECT forum, pid, subject, forum_name FROM {$_TABLES['forum_topic']} LEFT JOIN {$_TABLES['forum_forums']} ON forum = forum_id WHERE id = '$showtopic'"); // <- new
list($forum, $topic_pid, $subject, $forum_name) = DB_fetchArray($result); // <- new
$subject = stripslashes($forum_name) . ' | '. $subject;

I'm available to customise your themes or plugins for your Geeklog CMS

Status: offline

Laugh

Site Admin
Admin
Registered: 27/09/2005
Posts: 1232
Good catch Ben. I added your changes to the forum repository so they will be included in the next version of the forum.
One of the Geeklog Core Developers.

Status: offline

::Ben

Forum User
Full Member
Registered: 14/01/2005
Posts: 1569
Location:la rochelle, France
Maybe we could also add a h1 tag to topic_navbar.thtml

PHP Formatted Code
 <td colspan="2" style="padding-bottom:5px;"><h1>{navtopicimg}&nbsp;<b><a href="viewtopic.php?showtopic={topic_id}">{subject}</a></b></h1></td>

I'm available to customise your themes or plugins for your Geeklog CMS

Status: offline

Laugh

Site Admin
Admin
Registered: 27/09/2005
Posts: 1232
Both Modern Curve and Denim use an H1 tag in the header. We should really only have 1 h1 tag per page.

I think we should leave this one and if the person requires it, then they will have to manually add it.

BTW I am hoping at some point soon that the forum templates will receive an update to make them responsive.

Tom

One of the Geeklog Core Developers.

Status: offline

::Ben

Forum User
Full Member
Registered: 14/01/2005
Posts: 1569
Location:la rochelle, France
Maybe the site name or a site logo with an h1 tag on every page is not the most SEO optimisation Wink Anyway, a responsive theme will be a very nice feature.
I'm available to customise your themes or plugins for your Geeklog CMS

Status: offline

Laugh

Site Admin
Admin
Registered: 27/09/2005
Posts: 1232
Like with most SEO it is largely a matter of opinion. I have seen cases brought forth for both (and I am not sure which is best).

With the way the themes are setup that come with Geeklog H1 is the title of the site (or logo) and the Article titles are H2 (along with other things like the Directory). Since the article titles are H2 then any H tags in the actual article should be H3 or higher.
One of the Geeklog Core Developers.

All times are EDT. The time is now 08:55 pm.

  • Normal Topic
  • Sticky Topic
  • Locked Topic
  • New Post
  • Sticky Topic W/ New Post
  • Locked Topic W/ New Post
  •  View Anonymous Posts
  •  Able to post
  •  Filtered HTML Allowed
  •  Censored Content