Skip to content

Commit 36e50f1

Browse files
utarwynutarwyn
committed
Apply some PR feedbacks
1 parent 7f91907 commit 36e50f1

File tree

4 files changed

+39
-54
lines changed

4 files changed

+39
-54
lines changed

‎eslint-plugin/README.md‎

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -74,21 +74,21 @@ Add `@creedengo` to the `plugins` section of your `.eslintrc`, followed by rules
7474
⚠️ Configurations set to warn in.\
7575
✅ Set in the `recommended` configuration.
7676

77-
| Name                               | Description | ⚠️ |
78-
| :------------------------------------------------------------------------------------- | :------------------------------------------------------------------------ | :- |
79-
| [avoid-autoplay](docs/rules/avoid-autoplay.md) | Disallow autoplay and enforce preload='none' for video and audio elements | ✅ |
80-
| [avoid-brightness-override](docs/rules/avoid-brightness-override.md) | Should avoid to override brightness | ✅ |
81-
| [avoid-css-animations](docs/rules/avoid-css-animations.md) | Avoid usage of CSS animations | ✅ |
82-
| [avoid-high-accuracy-geolocation](docs/rules/avoid-high-accuracy-geolocation.md) | Avoid using high accuracy geolocation in web applications | ✅ |
83-
| [limit-db-query-results](docs/rules/limit-db-query-results.md) | Should limit the number of returns for a SQL query | ✅ |
84-
| [no-empty-image-src-attribute](docs/rules/no-empty-image-src-attribute.md) | Disallow usage of image with empty source attribute | ✅ |
85-
| [no-import-all-from-library](docs/rules/no-import-all-from-library.md) | Should not import all from library | ✅ |
86-
| [no-multiple-access-dom-element](docs/rules/no-multiple-access-dom-element.md) | Disallow multiple access of same DOM element | ✅ |
87-
| [no-multiple-style-changes](docs/rules/no-multiple-style-changes.md) | Disallow multiple style changes at once | ✅ |
88-
| [no-torch](docs/rules/no-torch.md) | Should not programmatically enable torch mode | ✅ |
89-
| [prefer-collections-with-pagination](docs/rules/prefer-collections-with-pagination.md) | Prefer API collections with pagination | ✅ |
90-
| [prefer-shorthand-css-notations](docs/rules/prefer-shorthand-css-notations.md) | Encourage usage of shorthand CSS notations | ✅ |
91-
| [provide-print-css](docs/rules/provide-print-css.md) | Enforce providing a print stylesheet | ✅ |
77+
| Name | Description | ⚠️ |
78+
| :------------------------------------------------------------------------------------- | :-------------------------------------------------------- | :- |
79+
| [avoid-autoplay](docs/rules/avoid-autoplay.md) | Avoid autoplay for videos and audio content | ✅ |
80+
| [avoid-brightness-override](docs/rules/avoid-brightness-override.md) | Should avoid to override brightness | ✅ |
81+
| [avoid-css-animations](docs/rules/avoid-css-animations.md) | Avoid usage of CSS animations | ✅ |
82+
| [avoid-high-accuracy-geolocation](docs/rules/avoid-high-accuracy-geolocation.md) | Avoid using high accuracy geolocation in web applications | ✅ |
83+
| [limit-db-query-results](docs/rules/limit-db-query-results.md) | Should limit the number of returns for a SQL query | ✅ |
84+
| [no-empty-image-src-attribute](docs/rules/no-empty-image-src-attribute.md) | Disallow usage of image with empty source attribute | ✅ |
85+
| [no-import-all-from-library](docs/rules/no-import-all-from-library.md) | Should not import all from library | ✅ |
86+
| [no-multiple-access-dom-element](docs/rules/no-multiple-access-dom-element.md) | Disallow multiple access of same DOM element | ✅ |
87+
| [no-multiple-style-changes](docs/rules/no-multiple-style-changes.md) | Disallow multiple style changes at once | ✅ |
88+
| [no-torch](docs/rules/no-torch.md) | Should not programmatically enable torch mode | ✅ |
89+
| [prefer-collections-with-pagination](docs/rules/prefer-collections-with-pagination.md) | Prefer API collections with pagination | ✅ |
90+
| [prefer-shorthand-css-notations](docs/rules/prefer-shorthand-css-notations.md) | Encourage usage of shorthand CSS notations | ✅ |
91+
| [provide-print-css](docs/rules/provide-print-css.md) | Enforce providing a print stylesheet | ✅ |
9292

9393
<!-- end auto-generated rules list -->
9494

‎eslint-plugin/docs/rules/avoid-autoplay.md‎

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,4 @@
1-
# Disallow autoplay and enforce preload='none' for video and audio elements (`@creedengo/avoid-autoplay`)
2-
3-
⚠️ This rule _warns_ in the ✅ `recommended` config.
4-
5-
<!-- end auto-generated rule header -->
1+
# Avoid autoplay for videos and audio content (`@creedengo/avoid-autoplay`)
62

73
⚠️ This rule _warns_ in the ✅ `recommended` config.
84

@@ -43,5 +39,6 @@ This rule is build for [React](https://react.dev/) and JSX.
4339
- [Mozilla Web Technology for Developers](https://developer.mozilla.org/en-US/docs/Web/HTML/Element/video#attr-preload) -
4440
Preload in HTML
4541
- [W3C](https://w3c.github.io/sustyweb/star.html#UX16-1) - Autoplay attribute
42+
- [RGESN](https://www.arcep.fr/mes-demarches-et-services/entreprises/fiches-pratiques/referentiel-general-ecoconception-services-numeriques.html) - Reference Rule 4.1
4643

4744

‎eslint-plugin/lib/rules/avoid-autoplay.js‎

Lines changed: 8 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
/*
2-
* ecoCode JavaScript plugin - Provides rules to reduce the environmental footprint of your JavaScript programs
3-
* Copyright © 2023 Green Code Initiative (https://www.ecocode.io)
2+
* creedengo JavaScript plugin - Provides rules to reduce the environmental footprint of your JavaScript programs
3+
* Copyright © 2023 Green Code Initiative (https://green-code-initiative.org)
44
*
55
* This program is free software: you can redistribute it and/or modify
66
* it under the terms of the GNU General Public License as published by
@@ -23,18 +23,15 @@ module.exports = {
2323
meta: {
2424
type: "suggestion",
2525
docs: {
26-
description:
27-
"Disallow autoplay and enforce preload='none' for video and audio elements",
26+
description: "Avoid autoplay for videos and audio content",
2827
category: "eco-design",
2928
recommended: "warn",
3029
},
3130
messages: {
32-
NoAutoplay:
33-
"Avoid using autoplay attribute for <video> and <audio> elements. Reference to Rule RGESN 4.1 : https://www.arcep.fr/mes-demarches-et-services/entreprises/fiches-pratiques/referentiel-general-ecoconception-services-numeriques.html",
34-
EnforcePreloadNone:
35-
"Set preload='none' for <video> and <audio> elements. Reference to Rule RGESN 4.1 : https://www.arcep.fr/mes-demarches-et-services/entreprises/fiches-pratiques/referentiel-general-ecoconception-services-numeriques.html",
36-
NoAutoplay_EnforcePreloadNone:
37-
"Avoid using autoplay attribute and set preload='none' for <video> and <audio> elements. Reference to Rule RGESN 4.1 : https://www.arcep.fr/mes-demarches-et-services/entreprises/fiches-pratiques/referentiel-general-ecoconception-services-numeriques.html ",
31+
NoAutoplay: "Avoid autoplay for video and audio elements.",
32+
EnforcePreloadNone: "Set preload='none' for video and audio elements.",
33+
NoAutoplayAndEnforcePreloadNone:
34+
"Avoid autoplay and set preload='none' for video and audio elements.",
3835
},
3936
schema: [],
4037
},
@@ -54,7 +51,7 @@ module.exports = {
5451
) {
5552
context.report({
5653
node: autoplayAttr || preloadAttr,
57-
messageId: "NoAutoplay_EnforcePreloadNone",
54+
messageId: "NoAutoplayAndEnforcePreloadNone",
5855
});
5956
} else {
6057
if (autoplayAttr) {

‎eslint-plugin/tests/lib/rules/avoid-autoplay.js‎

Lines changed: 14 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
/*
2-
* ecoCode JavaScript plugin - Provides rules to reduce the environmental footprint of your JavaScript programs
3-
* Copyright © 2023 Green Code Initiative (https://www.ecocode.io)
2+
* creedengo JavaScript plugin - Provides rules to reduce the environmental footprint of your JavaScript programs
3+
* Copyright © 2023 Green Code Initiative (https://green-code-initiative.org)
44
*
55
* This program is free software: you can redistribute it and/or modify
66
* it under the terms of the GNU General Public License as published by
@@ -38,47 +38,38 @@ const ruleTester = new RuleTester({
3838
},
3939
},
4040
});
41-
const expectedError1 = {
41+
42+
const noAutoplayError = {
4243
messageId: "NoAutoplay",
4344
type: "JSXAttribute",
4445
};
45-
const expectedError2 = {
46+
const enforcePreloadNoneError = {
4647
messageId: "EnforcePreloadNone",
4748
type: "JSXAttribute",
4849
};
49-
const expectedError3 = {
50-
messageId: "NoAutoplay_EnforcePreloadNone",
50+
const BothError = {
51+
messageId: "NoAutoplayAndEnforcePreloadNone",
5152
type: "JSXAttribute",
5253
};
5354

5455
ruleTester.run("autoplay-audio-video-attribute-not-present", rule, {
55-
valid: [
56-
`
57-
<audio preload="none"></audio>
58-
`,
59-
'<video preload="none"></video>',
60-
],
61-
56+
valid: ['<audio preload="none"></audio>', '<video preload="none"></video>'],
6257
invalid: [
6358
{
64-
code: `
65-
<audio autoplay></audio>
66-
`,
67-
errors: [expectedError3],
59+
code: "<audio autoplay></audio>",
60+
errors: [BothError],
6861
},
6962
{
70-
code: `
71-
<video autoplay preload="auto"></video>
72-
`,
73-
errors: [expectedError3],
63+
code: '<video autoplay preload="auto"></video>',
64+
errors: [BothError],
7465
},
7566
{
7667
code: '<video autoplay preload="none"></video>',
77-
errors: [expectedError1],
68+
errors: [noAutoplayError],
7869
},
7970
{
8071
code: '<audio preload="auto"></audio>',
81-
errors: [expectedError2],
72+
errors: [enforcePreloadNoneError],
8273
},
8374
],
8475
});

0 commit comments

Comments
 (0)