Skip to content

Commit e8b7db4

Browse files
authored
Selector: Eliminate selector.js depenencies from various modules
There are two main reasons for why some of those dependencies are no longer needed: 1. `jQuery.contains` which is now a part of `core`. 2. `jQuery.find.attr` no longer exists, native `getAttribute` is used instead. Closes gh-5383 Ref gh-5379
1 parent 99151d7 commit e8b7db4

File tree

5 files changed

+0
-7
lines changed

5 files changed

+0
-7
lines changed

src/attributes/attr.js

-2
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,6 @@ import { nodeName } from "../core/nodeName.js";
44
import { rnothtmlwhite } from "../var/rnothtmlwhite.js";
55
import { isIE } from "../var/isIE.js";
66

7-
import "../selector.js";
8-
97
jQuery.fn.extend( {
108
attr: function( name, value ) {
119
return access( this, jQuery.attr, name, value, arguments.length > 1 );

src/attributes/prop.js

-2
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,6 @@ import { jQuery } from "../core.js";
22
import { access } from "../core/access.js";
33
import { isIE } from "../var/isIE.js";
44

5-
import "../selector.js";
6-
75
var rfocusable = /^(?:input|select|textarea|button)$/i,
86
rclickable = /^(?:a|area)$/i;
97

src/css.js

-1
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ import { support } from "./css/support.js";
1717

1818
import "./core/init.js";
1919
import "./core/ready.js";
20-
import "./selector.js"; // contains
2120

2221
var
2322

src/manipulation.js

-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@ import { nodeName } from "./core/nodeName.js";
1515

1616
import "./core/init.js";
1717
import "./traversing.js";
18-
import "./selector.js";
1918
import "./event.js";
2019

2120
var

src/offset.js

-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ import { isWindow } from "./var/isWindow.js";
55

66
import "./core/init.js";
77
import "./css.js";
8-
import "./selector.js"; // contains
98

109
jQuery.offset = {
1110
setOffset: function( elem, options, i ) {

0 commit comments

Comments
 (0)