Add comments

Author: DanielCliftonGuardian

dotcom-rendering/.eslintrc.js

@@ -63,9 +63,11 @@ module.exports = {
 		'eslint:recommended',
 		'@guardian/eslint-config-typescript',
 		'plugin:jsx-a11y/recommended',
-		'prettier',
 		'plugin:ssr-friendly/recommended',
 		'plugin:storybook/recommended',
+		// eslint-config-prettier disables formatting rules that conflict with prettier
+		// needs to go last so it can override other configuration. See https://github.com/prettier/eslint-config-prettier#installation
+		'prettier',
 	],
 	parser: '@typescript-eslint/parser',
 	parserOptions: {

dotcom-rendering/src/components/ContainerTitle.stories.tsx

@@ -5,6 +5,7 @@ export default {
 	title: 'Components/ContainerTitle',
 	globals: {
 		viewport: {
+			// This has the effect of turning off the viewports addon by default
 			value: 'doesNotExist',
 			isRotated: false,
 		},

dotcom-rendering/src/components/FrontSection.stories.tsx

@@ -92,6 +92,7 @@ export default {
 
 	globals: {
 		viewport: {
+			// This has the effect of turning off the viewports addon by default
 			value: 'doesNotExist',
 			isRotated: false,
 		},

dotcom-rendering/src/components/LabsSection.stories.tsx

@@ -23,6 +23,7 @@ export default {
 
 	globals: {
 		viewport: {
+			// This has the effect of turning off the viewports addon by default
 			value: 'doesNotExist',
 			isRotated: false,
 		},

dotcom-rendering/src/components/Section.stories.tsx

@@ -12,6 +12,7 @@ export default {
 	title: 'Components/Section',
 	globals: {
 		viewport: {
+			// This has the effect of turning off the viewports addon by default
 			value: 'doesNotExist',
 			isRotated: false,
 		},