Opened 10 years ago
Closed 10 years ago
#8675 closed bug (fixed)
Dialog: Position with pixels in array doesn't work with back compat disabled
Reported by: | Mamen | Owned by: | |
---|---|---|---|
Priority: | blocker | Milestone: | 1.9.1 |
Component: | ui.dialog | Version: | 1.9.0 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
$("#dialog").dialog({position: [200, 200]});
This will render the same as [0, 0]. String values DO work. For example ['center', 'center']
Change History (8)
comment:1 Changed 10 years ago by
Resolution: | → worksforme |
---|---|
Status: | new → closed |
comment:2 Changed 10 years ago by
It doesn't work at all here. However my inital statement was somewhat wrong. In IE, and Chrome it defaults to "center center" if numerics are used. In Opera it defaults to "center top".
comment:3 Changed 10 years ago by
Hmm. I suspect this may be related to frames. The problem is most prominent with Opera. In all browsers I have tested, it works when selecting the live preview (stand alone page). May be problems with changing values live also.
comment:4 Changed 10 years ago by
I'm still not seeing any problems. The behavior you're seeing in Opera with the dialog showing at the top is caused by the fact that the document doesn't have enough height. Add another element to the page that actually makes the document taller and you'll see the positioning is working just fine.
comment:5 Changed 10 years ago by
May be something like that. However it fill fail in all browsers if you add
<script>$.uiBackCompat = false;</script>
The reason is that dialog uses offset, which is depricated. The option [200,200] is however NOT marked as depricated.
comment:6 Changed 10 years ago by
Milestone: | 1.10.0 → 1.9.1 |
---|---|
Priority: | minor → blocker |
Resolution: | worksforme |
Status: | closed → reopened |
Summary: | dialog: position with pixels in array doesn't work → Dialog: Position with pixels in array doesn't work with back compat disabled |
comment:7 Changed 10 years ago by
Status: | reopened → open |
---|
comment:8 Changed 10 years ago by
Resolution: | → fixed |
---|---|
Status: | open → closed |
Dialog: Don't use deprecated offset option for .position(). Fixes #8675 - Dialog: Position with pixels in array doesn't work with back compat disabled.
Changeset: 11effcbd35c7a0b36af2a710b385d741f5849cbf
Works for me: http://jsbin.com/udacal/1/edit