Dario Russo

Results 14 comments of Dario Russo

> https://github.com/bbc/simorgh/blob/e37762a5f4a068255ffd9978a716305b482d8f36/src/app/legacy/containers/PageHandlers/withOptimizelyProvider/getOptimizelyUserId/index.js#L17 > > Might as well add it here while you're at it Added 👌

> Nice one 👍 > > Main things I can see: > > * There's a new convention around locating components close to where they're used, and only moving them...

> The fonts will be coming along soon I believe, I believe we can slow down the release for now. We are also still waiting for more translations. I will...

From following [this guide](https://github.com/stylelint/stylelint/blob/14.0.0/docs/migration-guide/to-14.md#syntax-option-and-automatic-inferral-of-syntax): Due to deprecated 1. I installed [@stylelint/postcss-css-in-js](https://www.npmjs.com/package/@stylelint/postcss-css-in-js) 2. I installed @stylelint/postcss-css-in-js dependency [postcss-syntax](https://www.npmjs.com/package/postcss-syntax) 3. I added `customSyntax: '@stylelint/postcss-css-in-js'` In the stylelint.config.js

> Great work @DarioR01 > > **Related content** > > * H2 missing mapping id for the aria-labelledby from the section element > > * The wrapping div for the...

@joshcoventry No updates 🤕. Google is still working on it probably.

Block reason: eslint v8 is only supported by node version 12.22.0, 14.17.0, or >=16.0.0 . We run version 12.18.4 https://github.com/eslint/eslint/tree/v8.0.1#installation-and-usage

In previous versions, if the max value was null/undefined/0 it would default to an infinity value. This has changed therefore I had to use a fallback value. I am using...

As Jro pointed out, the maxAge was deprecated in favour of ttl therefore that is the only change I have made to this PR. The new version of the cache...

****Investigation report**** When we started investigating this bug we immediately noticed that if we open the accessibility tree in Firefox we can notice the keyboard's warnings attached to all graphic...