#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: | ||
Blocked by: | Blocking: |
Description
In IE6 incorrect value returned, instead of min value it returns max value. Patch attached.
Attachments (1)
Change History (5)
Changed 15 years ago by
comment:1 Changed 15 years ago by
Milestone: | 1.2 → 1.2.1 |
---|---|
Resolution: | → fixed |
Status: | new → closed |
This should be fixed in SVN, by brandon.
comment:2 Changed 15 years ago by
Resolution: | fixed |
---|---|
Status: | closed → reopened |
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 ?
comment:3 Changed 15 years ago by
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
This is fixed in SVN. We can't use inner[Width/Height] for all browsers except IE because Opera and Mozilla include the width of the scroll bars.
comment:4 Changed 15 years ago by
Milestone: | 1.2.1 → 1.2.2 |
---|
Note: See
TracTickets for help on using
tickets.
patch which fixes that micro bug