Skip to content

Commit ce5807a

Browse files
committed
refactor(domUtil): remove centerElementByMargin. Use flex containers.
1 parent 6d859f4 commit ce5807a

File tree

1 file changed

+0
-21
lines changed

1 file changed

+0
-21
lines changed

js/utils/dom.js

-21
Original file line numberDiff line numberDiff line change
@@ -189,27 +189,6 @@
189189
dest.parentNode.insertBefore(src, dest);
190190
},
191191

192-
/**
193-
* @private
194-
*/
195-
centerElementByMargin: function(el) {
196-
el.style.marginLeft = (-el.offsetWidth) / 2 + 'px';
197-
el.style.marginTop = (-el.offsetHeight) / 2 + 'px';
198-
},
199-
//Center twice, after raf, to fix a bug with ios and showing elements
200-
//that have just been attached to the DOM.
201-
centerElementByMarginTwice: function(el) {
202-
ionic.requestAnimationFrame(function() {
203-
ionic.DomUtil.centerElementByMargin(el);
204-
setTimeout(function() {
205-
ionic.DomUtil.centerElementByMargin(el);
206-
setTimeout(function() {
207-
ionic.DomUtil.centerElementByMargin(el);
208-
});
209-
});
210-
});
211-
},
212-
213192
elementIsDescendant: function(el, parent, stopAt) {
214193
var current = el;
215194
do {

0 commit comments

Comments
 (0)