dotcom-rendering/cdk/lib/renderingStack.ts (4 lines): - line 117: statistic: 'Average', // TODO - should we use p90? - line 195: // TODO – how does this overlap with the DevX debug dashboard? - line 219: // TODO - should we change to 3000? - line 246: // TODO - ideally we should convert these params to use the newer naming style for consistency dotcom-rendering/src/frontend/feArticle.ts (3 lines): - line 53: version: number; // TODO: check who uses? - line 85: publication: string; // TODO: check who uses? - line 100: nav: FENavType; // TODO move this out as most code uses a different internal NAV model. dotcom-rendering/src/components/CalloutEmbedBlockComponent.importable.tsx (3 lines): - line 182: // TODO: check if we need to send this - line 217: // TODO: use `useRef` once supported in Source - line 355: /* TODO: need to find an nicer way of dynamically setting svg dimensions */ dotcom-rendering/src/components/FeatureCard.tsx (3 lines): - line 279: imagePositionOnDesktop?: ImagePositionType /** TODO Remove this prop */; - line 280: imagePositionOnMobile?: ImagePositionType /** TODO Remove this prop */; - line 666: /**TODO: Determine if this is needed */ dotcom-rendering/src/components/BodyLiveblog.amp.tsx (2 lines): - line 20: // TODO check if liveblog background colours are more complex - like regular - line 113: data-max-items-per-page="20" // TODO confirm if this should be dynamic dotcom-rendering/src/server/htmlPageTemplate.ts (2 lines): - line 44: * TODO: Identify critical fonts to preload - line 258: dotcom-rendering/src/components/CommentBlockComponent.tsx (2 lines): - line 106: // TODO: remove avatarLink - line 123: {/* TODO: remove permalink */} dotcom-rendering/src/components/Blocks.amp.tsx (2 lines): - line 77: // TODO ad handling (currently done in elements, which is wrong, so let's lift - line 91: // TODO add last updated for blocks to show here dotcom-rendering/src/components/Discussion/AbuseReportForm.tsx (2 lines): - line 110: // TODO: use ref once forwardRef is implemented @guardian/src-button - line 380: // TODO: use ref once forwardRef is implemented @guardian/src-button dotcom-rendering/src/components/Discussion/Pagination.tsx (2 lines): - line 114: // @TODO: These overrides should be removed once fixed upstream - line 149: // @TODO: These overrides should be removed once fixed upstream dotcom-rendering/src/components/SlideshowCarousel.importable.tsx (2 lines): - line 247: // TODO: data-link-name="slideshow carousel left chevron" - line 264: // TODO: data-link-name="slideshow carousel right chevron" dotcom-rendering/src/lib/ad-targeting.ts (1 line): - line 6: // TODO: this function already exists in commercial-core, consider exporting it to avoid duplication dotcom-rendering/src/components/SubMeta.amp.tsx (1 line): - line 183: {/* TODO link to actual (non-AMP) site here. Also handle comment count behaviour. */} dotcom-rendering/src/components/Doughnut.tsx (1 line): - line 73: // TODO: Support displaying 0% for sections where value is zero dotcom-rendering/src/types/front.ts (1 line): - line 64: // TODO: These may need to be declared differently than the front types in the future dotcom-rendering/src/components/TrendingTopics.tsx (1 line): - line 88: {/* TODO: Add allpath link */} dotcom-rendering/src/components/ArticleContainer.tsx (1 line): - line 61: TODO: find a cleaner solution */ dotcom-rendering/src/components/Masthead/Titlepiece/Pillars.tsx (1 line): - line 136: TODO: export headlineBold15 from source */ dotcom-rendering/src/components/CardPicture.tsx (1 line): - line 36: // @TODO missing image size option dotcom-rendering/src/components/Discussion/CommentForm.tsx (1 line): - line 370: // TODO: Support resending verification email dotcom-rendering/src/components/Discussion/Comment.tsx (1 line): - line 180: // TODO: fix using !important dotcom-rendering/src/layouts/StandardLayout.tsx (1 line): - line 370: // TODO: dotcom-rendering/src/components/Discussion/RecommendationCount.tsx (1 line): - line 81: // TODO: do some error handling dotcom-rendering/src/server/handler.article.web.ts (1 line): - line 31: // TODO: We should rename this to 'article' or 'FEArticle', but first we need to investigate dotcom-rendering/src/components/EmbedBlockComponent.importable.tsx (1 line): - line 51: // TODO: Email embeds are being turned into atoms, so we can remove this hack when that happens dotcom-rendering/src/components/PersonalityQuizAtom.importable.tsx (1 line): - line 360: /* TODO: apply same styles on focus (requires source update) */ dotcom-rendering/src/components/Figure.tsx (1 line): - line 219: // TODO: If we want to support other element types having role position, such dotcom-rendering/src/layouts/ImmersiveLayout.tsx (1 line): - line 246: // TODO: dotcom-rendering/src/layouts/ShowcaseLayout.tsx (1 line): - line 236: // TODO: dotcom-rendering/src/components/FrontSection.tsx (1 line): - line 592: // TODO FIXME: dotcom-rendering/src/components/ArticleHeadline.tsx (1 line): - line 234: // TODO: find a cleaner solution dotcom-rendering/src/components/Onward.amp.tsx (1 line): - line 119: hasSectionMostViewed ? 0 : 1 // TODO not entirely sure why this is needed dotcom-rendering/src/lib/getDataLinkName.ts (1 line): - line 10: * TODO: missing "podcast" dotcom-rendering/src/components/CrosswordLinks.tsx (1 line): - line 26: // TODO: add to palette and replace CSS override if possible dotcom-rendering/src/components/StaticFeatureTwo.tsx (1 line): - line 56: /** TODO check if the pulsing dot should be be supported */ dotcom-rendering/src/components/Subheading.tsx (1 line): - line 138: /** TODO ! dotcom-rendering/src/components/DecideLines.tsx (1 line): - line 18: * **TODO:** move this up to Source Design Kitchen! dotcom-rendering/src/lib/branding.ts (1 line): - line 89: * TODO verify that this is what is necessary for two branding to be equal dotcom-rendering/src/components/ImageBlockComponent.amp.tsx (1 line): - line 67: TODO - Move caption handling to use https://github.com/guardian/dotcom-rendering/blob/master/packages/guui/components/Caption/Caption.tsx dotcom-rendering/src/layouts/CommentLayout.tsx (1 line): - line 291: // TODO: dotcom-rendering/src/lib/escapeData.tsx (1 line): - line 12: * TODO: This could be extended dotcom-rendering/src/components/AudioAtom/AudioAtom.tsx (1 line): - line 330: {/* eslint-disable-next-line jsx-a11y/media-has-caption -- TODO */} dotcom-rendering/src/paletteDeclarations.ts (1 line): - line 5110: // TODO - check if this is correct since it's the same shade as non-hover dotcom-rendering/src/lib/discussionApi.tsx (1 line): - line 61: // TODO: Refactor this for better typesafety. See https://github.com/guardian/dotcom-rendering/pull/8057/commits/da4667399d4a7726589f1944ac60380f1f3f36e1 dotcom-rendering/src/components/marketing/epics/ContributionsEpic.tsx (1 line): - line 335: return <>; // quick exit if something goes wrong. Ideally we'd throw and caller would catch/log but TODO that separately dotcom-rendering/src/components/ContentABTest.amp.tsx (1 line): - line 61: // eslint-disable-next-line react/jsx-no-constructed-context-values -- TODO consider using useMemo? dotcom-rendering/src/components/ArticlePage.amp.tsx (1 line): - line 83: {/* /TODO change to gray bgcolor */} dotcom-rendering/src/layouts/LiveLayout.tsx (1 line): - line 265: // TODO: dotcom-rendering/src/components/Card/Card.tsx (1 line): - line 945: //** TODO: IMPROVE THIS MAPPING */ dotcom-rendering/src/components/Card/Card.stories.tsx (1 line): - line 218: /** TODO : Update stories once initial chromatic check has passed on new font size scale */ dotcom-rendering/src/components/KeyEvents.amp.tsx (1 line): - line 53: // TODO text link style shouldn't be here dotcom-rendering/src/lib/discussionFilters.ts (1 line): - line 22: pageSize: PageSizeType; // TODO: Review these https://trello.com/c/7v4VDNY0/1326-review-page-size-values dotcom-rendering/src/layouts/PictureLayout.tsx (1 line): - line 263: // TODO: dotcom-rendering/src/lib/useIsHorizontalScrollingSupported.ts (1 line): - line 28: // TODO: update version after new bridget patch is published dotcom-rendering/window.guardian.ts (1 line): - line 39: // TODO expose as type from Automat client lib dotcom-rendering/src/components/Answers.tsx (1 line): - line 32: /* TODO: apply same styles on focus (requires source update) */ dotcom-rendering/src/components/MostViewedFooterGrid.tsx (1 line): - line 41: /* TODO: Using a pseudo selector here could be faster? */ dotcom-rendering/src/components/ScrollableFeature.importable.tsx (1 line): - line 51: /** TODO check if the pulsing dot should be be supported */ dotcom-rendering/src/components/marketing/banners/common/types.tsx (1 line): - line 71: articleCounts: ArticleCounts; // TODO - export dotcom-rendering/src/lib/braze/checkBrazeDependencies.ts (1 line): - line 74: // eslint-disable-next-line @typescript-eslint/strict-boolean-expressions -- TODO: Complex types, look into this further apps-rendering/src/client/article.ts (1 line): - line 226: // TODO: show epics on opinion articles dotcom-rendering/src/components/AdSlot.web.tsx (1 line): - line 29: // TODO move to commercial dotcom-rendering/src/lib/decideTrail.ts (1 line): - line 11: altText: '', // TODO: Do we get this from frontend? apps-rendering/src/atoms.ts (1 line): - line 317: /* TODO: imageUrl & imageAlt should be set here,