Use IE Conditional Tags for excanvas [#1119030]

ExCanvas is only for IE, so we should use Conditional Tags to hide it from less awful browsers.

Comment	File	Size	Author
#2	1119030-excanvas_conditional_tags.patch	973 bytes	xurizaemon
#1	1119030-excanvas_conditional_tags.patch	969 bytes	xurizaemon

Comments

Comment #1

xurizaemon

he/him

Ōtepoti, Aotearoa 🏝

commented 7 April 2011 at 03:29

Status	File	Size
new	1119030-excanvas_conditional_tags.patch	969 bytes

Like this?

    //for IE add this
    $expath = $path . '/other_libs/excanvas_r3/excanvas.compiled.js';
    if (file_exists($expath)) {
      $exhtml = array(
        '#type' => 'markup',
        '#markup' => '<!--[if IE]><script language="javascript" type="text/javascript" src="/' . $expath . '"></script><![endif]-->',
      );      
      drupal_add_html_head($exhtml, 'beautytips');
    }

Using Libraries would be cool too: #1060278: You should use hook_library for libraries in D7