from sun's issue here: #849862: Bartik code problems

+++ themes/bartik/scripts/search.js 6 Jul 2010 03:49:00 -0000
@@ -0,0 +1,26 @@
+  Drupal.behaviors.bartik = {
+    attach: function(context) {

This JS behavior effectively is http://drupal.org/project/compact_forms in a core theme. You will quickly find out that the code used here does not work in all browsers/platforms/versions. The issue queue of aforementioned project holds more detailed debugging information.

Comments

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

Just subscribing for now...

ericduran’s picture

ericduran commented

Just a thought. How about replacing the javascript with the new placeholder attribute and then writing some fallback javascript code if the browser doesn't support placeholder.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

The placeholder attribute is invalid (does not validate) for the document type Bartik is using. Second, even if it would validate, the JavaScript issues would remain the same for all other browsers.

Jeff Burnz’s picture

Jeff Burnz commented
Status: Active » Fixed

I think we can marked this as fixed since search.js has been removed (although I cant find the commit where that happened).

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.