search

indexiatech / ember-idx-modal

Ember Indexia Modal
http://indexiatech.github.io/ember-idx-modal/#/gettingstarted
Apache License 2.0
17 stars 13 forks source link

Modal issue with Safari (compatibility problems with liquid-fire) #29

Open tommyblue opened 9 years ago

tommyblue commented 9 years ago

Using Safari (v.8.0.7) the position of the{{#em-modal-form}} is completely wrong.

I'm using twitter bootstrap css, the form is inside a complex .row > .col-md-* skeleton. With chrome, firefox and IE it works correctly, with Safari the modal doesn't correctly expand to the full window.

Ember: 1.12.0
Ember Data: 1.0.0-beta.18
jQuery: 2.1.4
ember-idx-modal: 0.1.4
liquid-fire: 0.19.4

I'm using the workaround described here to avoid compatibility problems with liquid-fire: https://github.com/ef4/liquid-fire/issues/197#issuecomment-100152175

Here the working example with Chrome and the broken modal with Safari. The button triggering the modal is the button with the plus sign "+"

schermata 2015-08-06 alle 15 21 29 schermata 2015-08-06 alle 15 21 16

tommyblue commented 9 years ago

It definitely seems a problem with liquid-fire. Replacing all the liquid-fire tags with standard handlebars tags, the modals work correctly