Modify ↓
Ticket #1604 (closed bug: fixed)
IE6 incorrect value of jQuery(window).height()
| Reported by: | Q-Zma | Owned by: | |
|---|---|---|---|
| Priority: | major | Milestone: | 1.2.2 |
| Component: | core | Version: | 1.2 |
| Keywords: | Cc: | ||
| Blocking: | Blocked by: |
Description
In IE6 incorrect value returned, instead of min value it returns max value. Patch attached.
Attachments
Change History
comment:1 Changed 6 years ago by john
- Status changed from new to closed
- Resolution set to fixed
- Milestone changed from 1.2 to 1.2.1
This should be fixed in SVN, by brandon.
comment:2 Changed 6 years ago by arrix
- Status changed from closed to reopened
- Resolution fixed deleted
The latest svn version still has some problems.
See new $(window).height() functionality in the dev list.
Here is another patch.
Index: E:/zm/jquery/jquery/src/core.js
===================================================================
--- E:/zm/jquery/jquery/src/core.js (revision 3329)
+++ E:/zm/jquery/jquery/src/core.js (working copy)
@@ -1115,8 +1115,8 @@
jQuery.fn[ n ] = function(h) {
return this[0] == window ?
- jQuery.browser.safari && self["inner" + name] ||
- jQuery.boxModel && Math.max(document.documentElement["client" + name], document.body["client" + name]) ||
+ self["inner" + name] ||
+ jQuery.boxModel && document.documentElement["client" + name] ||
document.body["client" + name] :
this[0] == document ?
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.
Note: See
TracTickets for help on using
tickets.


patch which fixes that micro bug