no AJAX compatibility

would you like to make a patch?

in the latest dev release, I added Drupal.behaviors.syntaxhighlighter() for AJAX support, please test. I don't have any AJAX setup to test this so I need your help.

If you report back this works, then I'll make a release.

Thanks!

I misspelled behviors instead of behaviors

I've checked in the fix, please try again.

Hope I didn't waste too much of your time. This is what I get when I don't have the right setup to test things myself. I'll buy you a bear if you go to Drupalcon next year for this mix up.

Let me know how it goes and make sure you get the latest 'dev' version after the release bot get to it.

You can be sure it's the latest if the version string in syntaxhighlighter.module is:

// $Id: syntaxhighlighter.module,v 1.31 2009/12/04 22:46:01 mattyoung Exp $

~

>Fancy giving me co-maintainership?

How do I do this? I've given you CVS access. What else do I do to make you co-maintainer?

On page load, highlight doesn't happen because there is no page load ready handler in the new syntaxhighlighter.js file, I add one and took out the 'defer' flag but it's not working. Can you see why it's not working? I had to go to a meeting now and cannot debug it at the moment.

/**
 * Page ready handler
 */
jQuery(function($) {
	alert("I'm ready");
  SyntaxHighlighter.highlight();
  if (Drupal.settings.syntaxhighlighter.legacy_mode) {
    dp.SyntaxHighlighter.HighlightAll('code');
  }
  
});

also, when everything is working, minify the js file and use that instead.

Thx!

It doesn't work for me.

see http://hddigitalworks.com/syntax-highlighter

no highlighting...

tried on chrome, firefox, opera and safari.

still not working :(

see http://hddigitalworks.com/syntax-highlighter

Until everything is working, don't use the minify js file yet for better debugging (maybe the minify version is different?). Put 'alert("....")' in there to see if it's actually running?

Also, don't use the '$' variable in syntaxhighlighter.js, use 'jQuery' instead. This is the recommended way in D7 actually for better compatibility with other js lib because they might be defining the '$' at the global level.

Tom: it works!

See: http://hddigitalworks.com/syntax-highlighter

You are a js wizard :). My head spins just looking at your beautiful code.