From ce5807a022c3caed7678e2480334a12aacc2ad37 Mon Sep 17 00:00:00 2001 From: Andrew Date: Wed, 20 Aug 2014 14:17:54 -0600 Subject: [PATCH] refactor(domUtil): remove centerElementByMargin. Use flex containers. --- js/utils/dom.js | 21 --------------------- 1 file changed, 21 deletions(-) diff --git a/js/utils/dom.js b/js/utils/dom.js index 103f765639..22c4e24e4e 100644 --- a/js/utils/dom.js +++ b/js/utils/dom.js @@ -189,27 +189,6 @@ dest.parentNode.insertBefore(src, dest); }, - /** - * @private - */ - centerElementByMargin: function(el) { - el.style.marginLeft = (-el.offsetWidth) / 2 + 'px'; - el.style.marginTop = (-el.offsetHeight) / 2 + 'px'; - }, - //Center twice, after raf, to fix a bug with ios and showing elements - //that have just been attached to the DOM. - centerElementByMarginTwice: function(el) { - ionic.requestAnimationFrame(function() { - ionic.DomUtil.centerElementByMargin(el); - setTimeout(function() { - ionic.DomUtil.centerElementByMargin(el); - setTimeout(function() { - ionic.DomUtil.centerElementByMargin(el); - }); - }); - }); - }, - elementIsDescendant: function(el, parent, stopAt) { var current = el; do {