Bug Tracker

Modify

Ticket #11724 (closed bug: fixed)

Opened 2 years ago

Last modified 23 months ago

$(document).height() changed in Firefox 12

Reported by: David Owned by: mikesherov
Priority: high Milestone: 1.8
Component: dimensions Version: 1.7.2
Keywords: Cc: mikesherov
Blocking: Blocked by:

Description

I believe the fix for Ticket #3838 (http://bugs.jquery.com/ticket/3838) changed the value of $(document).height() in Firefox (using 12). While in Chrome and IE9 the value is consistent.

I can't reproduce this in jsFiddle as I think the result iframe's css works around this issue but this pastebin (  http://pastebin.com/MaUuLjU2) will demonstrate the problem.

In 1.7.1, the scrollHeight of the body element will be returned, while in 1.7.2 the clientHeight of the html element will be returned.

(I reported this in the comments of ticket #3838 - http://bugs.jquery.com/ticket/3838#comment:14)

Change History

comment:1 Changed 2 years ago by sindresorhus

  • Cc mikesherov added
  • Priority changed from undecided to low
  • Status changed from new to open
  • Component changed from unfiled to dimensions

Confirmed in Firefox 11 and 12.

comment:2 Changed 23 months ago by mikesherov

  • Owner set to mikesherov
  • Priority changed from low to high
  • Status changed from open to assigned
  • Milestone changed from None to 1.8

comment:3 Changed 23 months ago by Mike Sherov

  • Status changed from assigned to closed
  • Resolution set to fixed

Fix #11724, $(document).height() in Firefox 12. Closes gh-802.

This reopens #3838 for IE6 which is a regression on a fix in 1.7.2, but we'd rather break a really old IE than a really recent Firefox.

Changeset: ba70f8aeb60038ae73b188e432c471f3b115a790

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.