Ticket #6763 (closed bug: fixed)
ui.dialog & resizable & button & opera 11
| Reported by: | demimurych | Owned by: | |
|---|---|---|---|
| Priority: | minor | Milestone: | 1.8.18 |
| Component: | ui.dialog | Version: | 1.8.7 |
| Keywords: | Cc: | ||
| Blocking: | Blocked by: |
Description
Run opera 11 Go to http://jqueryui.com/demos/dialog/#modal-message
Try to resize dilog box
Look at button pane
Change History
comment:2 Changed 19 months ago by tj.vantoll
Created a minimal test case - http://jsfiddle.net/aKxgY/2/.
comment:3 Changed 17 months ago by Mamen
Masking Opera as Firefox fixes this. This means that custom code is executed for Opera. I'm guessing this code in ui.resizable is responsible:
//Opera fix for relative positioning
if (/relative/.test(this.element.css('position')) && $.browser.opera)
this.element.css({ position: 'relative', top: 'auto', left: 'auto' });
comment:4 Changed 16 months ago by tj.vantoll
This is the code that specifically creates the issue (in _alsoResize within resize):
// Opera fixing relative position
if ($.browser.opera && /relative/.test(el.css('position'))) {
self._revertToRelativePosition = true;
el.css({ position: 'absolute', top: 'auto', left: 'auto' });
}
In this instance it sets the .ui-dialog-content div to position: absolute which causes the layout issue. If that block is commented out this issue is fixed.
I'm wondering if anyone knows what the original hack for Opera was fixing? The only reference in the bug tracker to this I can find are #5694 & #5695 which are both fixes to the hack, and don't really explain what the hack was originally for. It appears to have been in there since the beginning of Git time and there aren't any unit tests for it.
If I remove the Opera hacks in their entirety all the demos work fine at least. Maybe this is a bug that Opera has fixed...?
comment:5 Changed 16 months ago by scott.gonzalez
Thanks tj. I'm not sure what it was fixing either. Now that we only support Opera 11.5+, it should be safe to remove. Can you send a pull request?
comment:6 Changed 16 months ago by tj.vantoll
Pull request - https://github.com/jquery/jquery-ui/pull/593
comment:7 Changed 16 months ago by scott.gonzalez
- Status changed from new to closed
- Resolution set to fixed
Fixed in f5e5971821dab4caf03dae4a6d207429aef44bae.
comment:8 Changed 16 months ago by scott.gonzalez
- Milestone changed from 1.9 to 1.8.18
Fixed in 1-8-stable in 28fb436b7b75de01cc1caf3132f56381b6c7c190.


I have a screencast for this bug: https://scienceco.de/~levu/dialog_opera_button.ogv