Hi:

I love Litejazz. We were exchanging messages in some thread when you were not the owner of the project. Good to see this is yours now.

I'm attaching the fixed version of page.tpl.php, as an initial approach to improve the theme quality. The main changes done are:

* Correct indentation (using 2 characters and spaces)
* Remove trailing spaces and several extra spaces
* Include $title_attributes in the H1 tag
* Included all the missing ";"
* All control structures switched to the form: if (): else: endif;
* Standarization of the print() instead of some echo()
* Other minor details
- At least the "Coder" module is able to validate this file now

I'm using this theme and I will keep fixing it as time permits. I you think this could be of any help, please let me know, and I will continue to upload the modifications. Notice this is against the 7.x-2.2 version. Feel free to ask for anything you see fits.

HTH

MyXelf

PS: There are more deep changes I would like to do, but I will go step-by-step

CommentFileSizeAuthor
page-tpl-php.txt8.78 KBMyXelf
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Kisugi Ai’s picture

Kisugi Ai’s picture

Status: Needs review » Closed (fixed)

Is in the actual Dev in.