Cool theme (from the screenshot anyway), I know you just uploaded it for drupal 5, but wondering if there are any plans to also make it available for drupal 6?

CommentFileSizeAuthor
#4 ablock_d6_port.patch1.96 KBnaheemsays
#2 ablock.info258 bytesnaheemsays

Comments

naheemsays’s picture

naheemsays commented

Giving the theme an info file (attached) mostly works, but gives the following errors:


    * recoverable fatal error: Object of class stdClass could not be converted to string in C:\wamp\www\drupal-6.3\sites\all\themes\ablock\page.tpl.php on line 4.
    * recoverable fatal error: Object of class stdClass could not be converted to string in C:\wamp\www\drupal-6.3\sites\all\themes\ablock\page.tpl.php on line 4.
naheemsays’s picture

naheemsays commented
Status: Active » Needs work
StatusFileSize
new ablock.info258 bytes

Updated ablog.info attached.

To get rid of the above mentioned error (comment 1), simple replace all (well, two) instances of $language with $language->language.

Also remove ablock_region from the template.php as that is now useless code.

Not working properly: the breadcrumb line/primary links.

If there is no primary link, that line does not show, and the breadcrumb (if there is one) starts off too far to the right.

If there are primary links, the breadcrumb line works as expected, but the primary links are not visible.

naheemsays’s picture

naheemsays commented
Status: Needs work » Needs review

and to fix the viewing of the primary and secondary links, on lines 58 and 64 replace theme('menu_links' with theme('links'.

This fixes the module for when there are primary/seconday links, but the theme is still broken if neither exist.

naheemsays’s picture

naheemsays commented
StatusFileSize
new ablock_d6_port.patch1.96 KB

And being a semi-good citizen, a patch for what I described above (it includes the .info file, so no need to take the one posted earlier.).

Anonymous’s picture

Anonymous (not verified) commented

Woooo thanks !
Apparently I didn't activated the email notifications for issues, I just noticed your messages !

Thanks very much for all the help, I'll include your modification into the new drupal 6 branch. There is a lot of code modification to come, I reviewed the entire page and node template, and work on more stable css.
It is kinda tricky not to have a standard page area in drupal :)

naheemsays’s picture

naheemsays commented

yup - this is a unique theme. Just a tad bright for me, but if I cannot figure out colour module integration, I will just change the colours to something a little darker before I use it for my personal site (and stop the hovered title having the same colour as the background).

Anonymous’s picture

Anonymous (not verified) commented

yeah this is bug is one of the fixed one, and there's many more , it should be available in 6-7 hours from now.

Happy to see that you're considering this theme for your own site :)
Remember to send me a link when it's live !

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Closed (fixed)

Ported to drupal 6, in dev.