paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-11-05 22:43:42 +00:00
Code Issues Packages Projects Releases Wiki Activity

zjquery: Fix chaining semantics.

Browse Source 
We now make it so that $('foo').addClass(whatever) and similar
functions properly return the wrapper object for chaining
purposes.  We may eventually want to change the wrapper object
to automatically dispatch to the first child object, but this
should work for now.
This commit is contained in:
Steve Howell
2017-06-12 07:24:58 -04:00
parent cabcdabfbc
commit 8da391b51e
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
frontend_tests/zjsunit/zjquery.js
View File

@@ -58,11 +58,11 @@ exports.make_zjquery = function () {
}, },
addClass: function (class_name) { addClass: function (class_name) {
classes.set(class_name, true); classes.set(class_name, true);
return self; return self.wrapper;
}, },
removeClass: function (class_name) { removeClass: function (class_name) {
classes.del(class_name); classes.del(class_name);
return self; return self.wrapper;
}, },
hasClass: function (class_name) { hasClass: function (class_name) {
return classes.has(class_name); return classes.has(class_name);
@@ -108,7 +108,7 @@ exports.make_zjquery = function () {
return self; return self;
}, },
on: function () { on: function () {
return self; return self.wrapper;
}, },
remove: function () { remove: function () {
if (my_parent) { if (my_parent) {
@@ -127,6 +127,7 @@ exports.make_zjquery = function () {
result[attr] = elem[attr]; result[attr] = elem[attr];
} }
} }
elem.wrapper = result;
return result; return result;
} }