The file 'uc_disable_zones.js' has a line with :text() selector instead :text, and that breaks site JavaScript.

The patch I attach fixes this.

CommentFileSizeAuthor
#2 uc_disable_zones-js_break.jpg213.85 KBsjovanig
uc_disable_zones-js_text_type.patch1.02 KBsjovanig

Comments

rocketeerbkw’s picture

rocketeerbkw commented

Thanks, that's clearly wrong. I've tested and used this in all the most recent browsers without issue though. Can you tell me what environment it failed in?

I'd like to reproduce your error for future testing before making an official release, but I'll get this patch into the -dev release ASAP.

sjovanig’s picture

sjovanig commented
StatusFileSize
new uc_disable_zones-js_break.jpg213.85 KB

Hi,

I attach screenshot that demonstrates the bug I reported.

BTW, the jQuery's selector ":text()" does not exist. The right way is ":text". See http://api.jquery.com/text-selector/

Regards.

rocketeerbkw’s picture

rocketeerbkw commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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