As mention in http://drupal.org/node/11929 i have removed the JavaScript from goofy. In this version all the Java is gone but a also moved the stule sheet to a externall file and changes a bit of the comment function.

e.g. i removed the if statement in the commend script, i don't see the point of it.

please test if i did break something.

Mad Maks

CommentFileSizeAuthor
#4 goofy_xthml1112.27 KBMordio
#3 goofy.patch17.76 KBMad Maks
#1 style.css2.05 KBMad Maks
goofy_modifed_011.8 KBMad Maks

Comments

Mad Maks’s picture

Mad Maks commented
StatusFileSize
new style.css2.05 KB

the style sheet

LimitMan’s picture

LimitMan commented

Hey man ... Great .. with this two files You solowed all problems.

1. Header information error dose not apear any more (and it was apearing at moment of submiting ANY form)
2. I think that crazy guest number is now ok (but I also think that was google crawling bots, since I was tracing logs. They are coming from adsence)
3. Java Script error is not apearing on first opening of page (like it did with original goofy)
4. And finaly I can put adsence on my page :-)

I will see if I notice some more bugs, but for now on it looks ok :-)

(just to mention I am admin of zawireless.afz.biz that posted that bug report)

Thanks for doing this. Goofy communtiy owns You ;-)

Mad Maks’s picture

Mad Maks commented
StatusFileSize
new goofy.patch17.76 KB

patch agains HEAD

Mordio’s picture

Mordio commented
StatusFileSize
new goofy_xthml1112.27 KB

Hello!

I made your modification XHTML 1.1 compliant. I post it here in case anybody is interested in it. To make it look properly in Firefox (or any other Gecko based-browser) you have to add the follwowing lines to your style.css:

td>img {
  display: block;
}

-- Mordio

TDobes’s picture

TDobes commented

FYI: I'll be committing this soon (after some minimal testing of my own). If anyone knows of any bugs, speak now, please. :-)

Mad Maks’s picture

Mad Maks commented

TDobes, what is the status of your testing?

TDobes’s picture

TDobes commented

Thanks for your work, Mad Maks.
Also thanks to Mordio for the "display: block" tip.

I have committed a somewhat modified version of this patch to CVS HEAD. (A few tweaks to make the theme XHTML rather than HTML and whitespace cleanup)

I did my best to maintain the exact same appearance as the javascript version. Please open a new bug report if anyone finds any problems.

Anonymous’s picture

(not verified) commented