Bug Tracker

Opened 15 years ago

Closed 12 years ago

#3117 closed bug

$(window).height() problem in opera 9.5

Reported by: michalhtm Owned by: flesler
Priority: major Milestone: 1.3
Component: dimensions Version: 1.2.6
Keywords: Cc: emartin24, michalhtm
Blocked by: Blocking:


There is problem in opera 9.5 if the page is longer (scrollbars showed)




I think repair to it is using


(it shows correct value)

Attachments (1)

dimensions[5759].diff (685 bytes) - added by flesler 15 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 15 years ago by flesler

Component: fxdimensions
Owner: set to flesler
Status: newassigned

I had this problem on jQuery.ScrollTo.. and I think I fixed like this (next patch). Can you apply the past and re-test please ?

Changed 15 years ago by flesler

Attachment: dimensions[5759].diff added

comment:2 Changed 14 years ago by emartin24

Just wanted to check on the status...I just ran into this issue and the suggested patch seems to resolve it.

comment:3 Changed 14 years ago by flesler

That's great. Brandon is busy, but if you provide a test case where I see this patch actually working as expected, I'll commit it myself.

comment:4 Changed 14 years ago by flesler

Cc: emartin24 michalhtm added
Milestone: 1.3
need: ReviewTest Case

comment:5 Changed 14 years ago by emartin24

I put a simple page up that shows the fix in action: http://www.ericmmartin.com/code/jquery/3117/

With Opera 9.5+

  • Click before, click TEST. There is supposed to be a modal dialog, but because if the incorrect height value, it is not in the viewport. Click the overlay to close it, and see the incorrect height value being reported.
  • Click after, click TEST. The dialog is now in the viewport and you will see the correct height being reported.

In any other browser, both pages behave the same way.


comment:6 Changed 14 years ago by dmethvin

See also the discussion in duplicate #3046.

comment:7 Changed 13 years ago by dmethvin

Possible patch in #5987.

comment:8 Changed 12 years ago by snover

Status: assignedpending

This ticket has been marked as missing a test case. In an effort to reduce the number of outstanding tickets in the bug tracker, it will be closed automatically in 30 days. In order to prevent this from happening, please provide a working test case. If a test case has already been provided and our records are wrong, please respond to the ticket so that it can be fixed. Thank you!

comment:9 Changed 12 years ago by trac-o-bot

Status: pendingclosed

Automatically closed due to 14 days of inactivity.

Note: See TracTickets for help on using tickets.