Comments

johnbarclay’s picture

Version: 7.x-1.x-dev » 7.x-2.x-dev
Category: task » feature
mgifford’s picture

liam morland’s picture

Status: Active » Needs review
StatusFileSize
new6.77 KB

This patch handles asterisks for both required field and table row reordering. The JS patches don't work properly I think because the custom JS is not loading after the core JS. I could use some help with this. Perhaps there is a better way to override the core JS.

erynn’s picture

I had a similar problem with the patch for the user.js. You need to add the js file and change its weight. What I did for the login fix:
so drupal_add_js('/sites/all/modules/accessible/accessible_fix/module_fixes/user.js', array('type' => 'external', 'weight' => 20,));
Note that the module structure needs to change because you won't be able to use hook_fix_js_alter again as it is called in the user.inc file.

liam morland’s picture

StatusFileSize
new7.01 KB

OK, here is a version of the patch that uses drupal_add_js(). Please have a look; it's not working on my system. With drupal_add_js() there, jQuery doesn't load. If I comment it out, jQuery returns. What I am doing wrong?

liam morland’s picture

StatusFileSize
new7.72 KB

OK, I've learned something about using JS with Drupal. Now it works!

mgifford’s picture

Issue summary: View changes
Status: Needs review » Needs work

No longer applies.