Skip to content

Feat/compt 31 dom event hooks #2

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
a-elkhiraooui-ciscode wants to merge 4 commits into from

feat(COMPT-31): add useMediaQuery, useWindowSize, useClickOutside, us…

c4172f3
Select commit
Loading
Failed to load commit list.
Closed

Feat/compt 31 dom event hooks #2

feat(COMPT-31): add useMediaQuery, useWindowSize, useClickOutside, us…
c4172f3
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Mar 30, 2026 in 23s

Quality Gate failed

Failed conditions
15.2% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

Annotations

Check warning on line 31 in src/hooks/useWindowSize.test.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=CISCODE-MA_HooksKit&issues=AZ0-ie3kGOacgaJboQ25&open=AZ0-ie3kGOacgaJboQ25&pullRequest=2

Check warning on line 48 in src/hooks/useClickOutside.test.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `childNode.remove()` over `parentNode.removeChild(childNode)`. 

See more on https://sonarcloud.io/project/issues?id=CISCODE-MA_HooksKit&issues=AZ0-ie24GOacgaJboQ2q&open=AZ0-ie24GOacgaJboQ2q&pullRequest=2

Check warning on line 36 in src/hooks/useWindowSize.test.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=CISCODE-MA_HooksKit&issues=AZ0-ie3kGOacgaJboQ26&open=AZ0-ie3kGOacgaJboQ26&pullRequest=2

Check warning on line 73 in src/hooks/useWindowSize.test.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=CISCODE-MA_HooksKit&issues=AZ0-ie3kGOacgaJboQ2_&open=AZ0-ie3kGOacgaJboQ2_&pullRequest=2

Check failure on line 100 in src/hooks/useIntersectionObserver.test.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to not always return the same value. 

See more on https://sonarcloud.io/project/issues?id=CISCODE-MA_HooksKit&issues=AZ0-ie3sGOacgaJboQ3I&open=AZ0-ie3sGOacgaJboQ3I&pullRequest=2

Check warning on line 11 in src/hooks/useMediaQuery.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Unexpected negated condition. 

See more on https://sonarcloud.io/project/issues?id=CISCODE-MA_HooksKit&issues=AZ0-ie3BGOacgaJboQ2u&open=AZ0-ie3BGOacgaJboQ2u&pullRequest=2

Check warning on line 56 in src/hooks/useWindowSize.test.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=CISCODE-MA_HooksKit&issues=AZ0-ie3kGOacgaJboQ29&open=AZ0-ie3kGOacgaJboQ29&pullRequest=2

Check warning on line 47 in src/hooks/useClickOutside.test.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `childNode.remove()` over `parentNode.removeChild(childNode)`. 

See more on https://sonarcloud.io/project/issues?id=CISCODE-MA_HooksKit&issues=AZ0-ie24GOacgaJboQ2p&open=AZ0-ie24GOacgaJboQ2p&pullRequest=2

Check warning on line 49 in src/hooks/useWindowSize.test.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=CISCODE-MA_HooksKit&issues=AZ0-ie3kGOacgaJboQ28&open=AZ0-ie3kGOacgaJboQ28&pullRequest=2

Check warning on line 84 in src/hooks/useWindowSize.test.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=CISCODE-MA_HooksKit&issues=AZ0-ie3kGOacgaJboQ3B&open=AZ0-ie3kGOacgaJboQ3B&pullRequest=2

Check warning on line 78 in src/hooks/useIntersectionObserver.test.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `childNode.remove()` over `parentNode.removeChild(childNode)`. 

See more on https://sonarcloud.io/project/issues?id=CISCODE-MA_HooksKit&issues=AZ0-ie3sGOacgaJboQ3G&open=AZ0-ie3sGOacgaJboQ3G&pullRequest=2

Check warning on line 69 in src/hooks/useClickOutside.test.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `childNode.remove()` over `parentNode.removeChild(childNode)`. 

See more on https://sonarcloud.io/project/issues?id=CISCODE-MA_HooksKit&issues=AZ0-ie24GOacgaJboQ2r&open=AZ0-ie24GOacgaJboQ2r&pullRequest=2

Check warning on line 19 in src/hooks/useWindowSize.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=CISCODE-MA_HooksKit&issues=AZ0-iezmGOacgaJboQ2k&open=AZ0-iezmGOacgaJboQ2k&pullRequest=2

Check warning on line 72 in src/hooks/useWindowSize.test.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=CISCODE-MA_HooksKit&issues=AZ0-ie3kGOacgaJboQ2-&open=AZ0-ie3kGOacgaJboQ2-&pullRequest=2

Check warning on line 94 in src/hooks/useIntersectionObserver.test.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `childNode.remove()` over `parentNode.removeChild(childNode)`. 

See more on https://sonarcloud.io/project/issues?id=CISCODE-MA_HooksKit&issues=AZ0-ie3sGOacgaJboQ3H&open=AZ0-ie3sGOacgaJboQ3H&pullRequest=2

Check warning on line 7 in src/hooks/useMediaQuery.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=CISCODE-MA_HooksKit&issues=AZ0-ie3BGOacgaJboQ2t&open=AZ0-ie3BGOacgaJboQ2t&pullRequest=2

Check warning on line 14 in src/hooks/useClickOutside.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis.window` over `window`. 

See more on https://sonarcloud.io/project/issues?id=CISCODE-MA_HooksKit&issues=AZ0-ie3SGOacgaJboQ2y&open=AZ0-ie3SGOacgaJboQ2y&pullRequest=2

Check warning on line 44 in src/hooks/useWindowSize.test.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=CISCODE-MA_HooksKit&issues=AZ0-ie3kGOacgaJboQ27&open=AZ0-ie3kGOacgaJboQ27&pullRequest=2

Check warning on line 31 in src/hooks/useWindowSize.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=CISCODE-MA_HooksKit&issues=AZ0-iezmGOacgaJboQ2n&open=AZ0-iezmGOacgaJboQ2n&pullRequest=2

Check warning on line 11 in src/hooks/useIntersectionObserver.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis.window` over `window`. 

See more on https://sonarcloud.io/project/issues?id=CISCODE-MA_HooksKit&issues=AZ0-ie3JGOacgaJboQ2x&open=AZ0-ie3JGOacgaJboQ2x&pullRequest=2

Check warning on line 135 in src/hooks/useMediaQuery.test.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=CISCODE-MA_HooksKit&issues=AZ0-ie3bGOacgaJboQ22&open=AZ0-ie3bGOacgaJboQ22&pullRequest=2

Check warning on line 11 in src/hooks/useMediaQuery.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis.window` over `window`. 

See more on https://sonarcloud.io/project/issues?id=CISCODE-MA_HooksKit&issues=AZ0-ie3BGOacgaJboQ2v&open=AZ0-ie3BGOacgaJboQ2v&pullRequest=2

Check warning on line 103 in src/hooks/useWindowSize.test.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`. 

See more on https://sonarcloud.io/project/issues?id=CISCODE-MA_HooksKit&issues=AZ0-ie3kGOacgaJboQ3D&open=AZ0-ie3kGOacgaJboQ3D&pullRequest=2

Check warning on line 9 in src/hooks/useWindowSize.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis.window` over `window`. 

See more on https://sonarcloud.io/project/issues?id=CISCODE-MA_HooksKit&issues=AZ0-iezmGOacgaJboQ2i&open=AZ0-iezmGOacgaJboQ2i&pullRequest=2

Check warning on line 25 in src/hooks/useClickOutside.test.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `childNode.remove()` over `parentNode.removeChild(childNode)`. 

See more on https://sonarcloud.io/project/issues?id=CISCODE-MA_HooksKit&issues=AZ0-ie24GOacgaJboQ2o&open=AZ0-ie24GOacgaJboQ2o&pullRequest=2
View more details on SonarQubeCloud