Jonathan Roebuck

Results 32 issues of Jonathan Roebuck

I'm keen to know if it's better to have conditional render logic inside of the component itself or in the parent component. ```jsx const Text = ({ content }) =>...

Resolves #NUMBER **Overall change:** _A very high-level summary of easily-reproducible changes that can be understood by non-devs._ **Code changes:** - _A bullet point list of key code changes that have...

Resolves #NUMBER **Overall change:** _A very high-level summary of easily-reproducible changes that can be understood by non-devs._ **Code changes:** - _A bullet point list of key code changes that have...

Supports https://github.com/bbc/simorgh/pull/10230 **Overall change:** Uses the new JSX Transform as described here https://reactjs.org/blog/2020/09/22/introducing-the-new-jsx-transform.html **Code changes:** - Adds the new babel react runtime in the babel config - React imports removed...

Resolves WSTEAMA-124 **Overall change:** Set up theming with Emotion's ThemeProvider to provide colours, media queries, and spacings. This makes it possible to use theming more easily in styles for our...

Resolves NEWSWORLDSERVICE-1494 **Overall change:** Adds agreed set of component standards to the docs. Replaces old coding standards doc. TODO Still to provide some code examples for some of the standards....

We should refactor the ATI analytics component API to accept the minimal props it needs in order to function. We should remove page type conditional logic from the component and...

Refinement Needed
technical-work

**Is your feature request related to a problem? Please describe.** quicklink uses Intersection Observer API to detect when links come into the viewport and prefetches linked resources during idle time....

Refinement Needed
performance
technical-work
cross-team

**Is your feature request related to a problem? Please describe.** An idea was discussed to change the way we fetch and process data and handle errors at the page route...

Refinement Needed
technical-work
cross-team

**Is your feature request related to a problem? Please describe.** When all services are using Simorgh topics pages, we need to remove the temporary topics (`/new_topics/`) route that was set...

Refinement Needed