[Tickets #884] browser compatibility

[bugs at bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/?id=884
-----------------------------------------------------------------------
 Ticket     | 884
 Updated By | m.zdila at episoftware.com
 Summary    | browser compatibility
 Queue      | Horde Base
 Version    | HEAD
 State      | Feedback
 Priority   | 2. Medium
 Type       | Bug
 Owners     | Horde Developers
-----------------------------------------------------------------------


m.zdila at episoftware.com (2004-11-26 02:01) wrote:

(((1))) There is another problem with MSIE 5.0. It doesn't support
array.push(object) method, so you should instead use array[array.length] =
object.

For example, now you can't close the branch in the sidebar tree, because of
that problem. Patch:
templates/javascript/tree.js line 429:
- newNodes.push(nodes[i]);
+ newNodes[newNodes.length] = nodes[i];

... but there are other places where the push method is used.

(((2))) About that horizontal space between rows in tree menu: Yes, I use
custom theme, but I only changed a color, and I experience this problem
while using other themes too. This envolves MSIE 5.0, MSIE 5.5, MSIE 6.0.
I'll try to manage to send you the patch asap.

(((3))) Image loading - this problem doesn't show only on page loads, but
everytime I open or close a branch. The MSIE doesn't cache anything for me
(or maybe does, but it doesn't influence that). I have cache turned on, and
Tools/Internet Options/Advanced/Restore Defaults. MSIE 5.0, MSIE 5.5, MSIE
6.0.

As I mentioned before, I've been working on a project that uses Javascript
tree too, supporting dynamic loading and many special features, so I know it
is possible to improve the tree in the Horde so those problems will be
eliminated (problem 2 and 3). I'll try to manage to look at it, but
unfortunately I am under time pressure now.