Opened 9 years ago

Closed 9 years ago

#11634 closed defect (fixed)

remove old code from dojo.position()

Reported by: bill Owned by: Douglas Hays
Priority: high Milestone: 1.6
Component: HTML Version: 1.5
Keywords: Cc: James Burke
Blocked By: Blocking:

Description (last modified by bill)

The branch of dojo.position() (formerly dojo._abs()) with the comment:

// FF2 and older WebKit

can be removed, cutting 54 lines out of dojo base.

Actually, since we no longer support FF3 (as stated in http://docs.dojocampus.org/releasenotes/1.5), we can also remove the block starting with:

}else if(d.isFF == 3){

Change History (4)

comment:1 Changed 9 years ago by bill

Description: modified (diff)

comment:2 Changed 9 years ago by Douglas Hays

Cc: James Burke added
Milestone: tbd1.6

jburke, I wanted to get your OK (documented here) on removing FF2, FF3, and Safari 3.x support for dojo.position. Removing this code will be actively desupporting those browsers instead of letting them die thru attrition.

comment:3 Changed 9 years ago by James Burke

I want to leave the FF 3 branch for now. But the FF2/old safari branch can go.

comment:4 Changed 9 years ago by Douglas Hays

Resolution: fixed
Status: newclosed

(In [22771]) Fixes #11634 !strict. Remove dojo.position's support for FF2- and Safari 3-.

Note: See TracTickets for help on using tickets.