Originally Posted by SD
those won't be in html.inc.php, but you can easily add those two lines to the common header insert.

Adding stuff to includes/header-insert.php is a start, but since "Facebook Open Graph" inserts are currently partialy being coded to one place, for the sake of being "proper," they should be completely coded in that one place.

There really isn't any user-configurable data that cannot be derived from current config $strings -- so dropping the two lines I posted above, in to header-insert.php seems silly. Especially since without them, your "Facebook Open Graph" code wont validate without warnings.

EXAMPLE: https://developers.facebook.com/too...2Ftopics%2F254231%2FSEO_Meta_Description

And, having incomplete Facebook Open Graph on many other pages that are not generated by html.inc.php (such as online, login, register...etc.) only makes your pages that people wont usually link to, unnecessarily larger.

Originally Posted by SD
yes, but i confirmed that it WON'T be in there in the official release. ( responding to id242 )

Cool - So you're taking out the "Facebook Open Graph" code?

If not, what I was trying to state in my previous code, was that if you're going to add Open Graph code to the html.inc.php (which is included in UBBT v7.5.8) then I suggest following their guidelines and having the code validate without warnings, through their debugger. smile

Anyways - keep up the good work and take my suggestions for correction as only a hope for code improvement 2c - nothing is personal thumbsup

Last edited by id242; 12/07/2013 12:59 AM.

current developer of UBB.threads php forum software
current release: UBB.threads 8.0.1 // wip: UBB.threads 8.1.0
isaac @ id242.com // my forum @ CelicaHobby.com