Safari doesn't like bareword 'class'

jpmckinney - September 28, 2009 - 14:45
Project:JavaScript Theming
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:needs review
Description

Line 369 of js_theming.table.js

cell = {data: cell, class:'active'};

Change it to:

cell = {data: cell, 'class':'active'};

Otherwise, Safari will complain about a syntax error.

#1

jpmckinney - September 28, 2009 - 21:42
Status:active» needs review

#2

jpmckinney - October 15, 2009 - 16:29

This patch fixes a Safari bug (bareword class) and an IE bug (trailing comma). It also gets rid of trailing newlines at the end of the file.

AttachmentSize
example.patch 620 bytes
 
 

Drupal is a registered trademark of Dries Buytaert.