Skip to content

Commit 3df1ad5

Browse files
committed
chore(): move nested functions out of if blocks
1 parent 55fb60e commit 3df1ad5

File tree

1 file changed

+24
-23
lines changed

1 file changed

+24
-23
lines changed

ionic/util/dom.ts

Lines changed: 24 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -91,25 +91,25 @@ export let CSS: {
9191

9292
export function transitionEnd(el: HTMLElement, callback: Function) {
9393
if (el) {
94-
function unregister() {
95-
CSS.transitionEnd.split(' ').forEach(eventName => {
96-
el.removeEventListener(eventName, onEvent);
97-
});
98-
}
99-
100-
function onEvent(ev) {
101-
if (el === ev.target) {
102-
unregister();
103-
callback(ev);
104-
}
105-
}
106-
10794
CSS.transitionEnd.split(' ').forEach(eventName => {
10895
el.addEventListener(eventName, onEvent);
10996
});
11097

11198
return unregister;
11299
}
100+
101+
function unregister() {
102+
CSS.transitionEnd.split(' ').forEach(eventName => {
103+
el.removeEventListener(eventName, onEvent);
104+
});
105+
}
106+
107+
function onEvent(ev) {
108+
if (el === ev.target) {
109+
unregister();
110+
callback(ev);
111+
}
112+
}
113113
}
114114

115115
export function ready(callback?: Function) {
@@ -124,17 +124,17 @@ export function ready(callback?: Function) {
124124
callback();
125125

126126
} else {
127-
function completed() {
128-
document.removeEventListener('DOMContentLoaded', completed, false);
129-
window.removeEventListener('load', completed, false);
130-
callback();
131-
}
132-
133127
document.addEventListener('DOMContentLoaded', completed, false);
134128
window.addEventListener('load', completed, false);
135129
}
136130

137131
return promise;
132+
133+
function completed() {
134+
document.removeEventListener('DOMContentLoaded', completed, false);
135+
window.removeEventListener('load', completed, false);
136+
callback();
137+
}
138138
}
139139

140140
export function windowLoad(callback?: Function) {
@@ -149,15 +149,16 @@ export function windowLoad(callback?: Function) {
149149
callback();
150150

151151
} else {
152-
function completed() {
153-
window.removeEventListener('load', completed, false);
154-
callback();
155-
}
156152

157153
window.addEventListener('load', completed, false);
158154
}
159155

160156
return promise;
157+
158+
function completed() {
159+
window.removeEventListener('load', completed, false);
160+
callback();
161+
}
161162
}
162163

163164
export function pointerCoord(ev: any): {x: number, y: number} {

0 commit comments

Comments
 (0)