Skip to content

Commit bf87fb7

Browse files
elusivecodeselusivecodes
committed
- Fixed delegate event handling
1 parent f9fefa5 commit bf87fb7

File tree

6 files changed

+80
-45
lines changed

6 files changed

+80
-45
lines changed

‎dist/fquery.js‎

Lines changed: 46 additions & 40 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎dist/fquery.js.map‎

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎dist/fquery.min.js‎

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎dist/fquery.min.js.map‎

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎dist/test.html‎

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
<!DOCTYPE html>
2+
<html lang="en">
3+
4+
<head>
5+
<meta charset="UTF-8">
6+
<meta http-equiv="X-UA-Compatible" content="IE=edge">
7+
<meta name="viewport" content="width=device-width, initial-scale=1.0">
8+
<title>Document</title>
9+
<script src="fquery.js"></script>
10+
</head>
11+
12+
<body>
13+
<button>A</button>
14+
<button>B</button>
15+
<button>C</button>
16+
<script>
17+
$.addEventDelegateOnce(document.body, 'click', 'button', e => {
18+
console.log(e);
19+
});
20+
</script>
21+
</body>
22+
23+
</html>

‎src/events/event-factory.js‎

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,8 +73,14 @@ export function delegateFactory(node, selector, callback) {
7373
return;
7474
}
7575

76-
Object.defineProperty(event, 'currentTarget', delegate);
77-
Object.defineProperty(event, 'delegateTarget', node);
76+
Object.defineProperty(event, 'currentTarget', {
77+
value: delegate,
78+
configurable: true
79+
});
80+
Object.defineProperty(event, 'delegateTarget', {
81+
value: node,
82+
configurable: true
83+
});
7884

7985
return callback(event);
8086
};

0 commit comments

Comments
 (0)