Skip to main content

Bug Tracker

Side navigation

#3117 closed bug ()

Opened July 02, 2008 07:58AM UTC

Closed November 11, 2010 11:09PM UTC

$(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:
Description

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

$(window).height()

returns

$(document).height()

I think repair to it is using

document.getElementsByTagName('html')[0].clientHeight

(it shows correct value)

Attachments (1)
Change History (9)

Changed July 02, 2008 02:35PM UTC by flesler comment:1

component: fxdimensions
owner: → 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 August 12, 2008 06:23PM UTC by emartin24 comment:2

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

Changed August 12, 2008 06:26PM UTC by flesler comment:3

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.

Changed August 12, 2008 06:27PM UTC by flesler comment:4

cc: → emartin24, michalhtm
milestone: → 1.3
need: ReviewTest Case

Changed August 12, 2008 07:29PM UTC by emartin24 comment:5

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.

-Eric

Changed May 16, 2009 02:50AM UTC by dmethvin comment:6

See also the discussion in duplicate #3046.

Changed June 22, 2010 01:58AM UTC by dmethvin comment:7

Possible patch in #5987.

Changed October 14, 2010 03:14AM UTC by snover comment:8

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!

Changed November 11, 2010 11:09PM UTC by trac-o-bot comment:9

status: pendingclosed

Automatically closed due to 14 days of inactivity.