firefox-devtools/profiler

ActivityGraph Tooltip should show the sample's page

Aperta

#3664 aperta il 23 nov 2021

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (0 assegnatari)TypeScript (486 fork)auto 404
help wantedtimeline

Metriche repository

Star
 (1515 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

I noticed that the tooltip in the activity graph (that's the tooltip with SampleTooltipContents) doesn't show the sample's page url. As a preliminary task, it would be good to extract and merge the code for pages in the existing tooltips: https://github.com/firefox-devtools/profiler/blob/8da8e85846efc90971908ad6fd765f08234018a8/src/components/tooltip/Marker.js#L100-L146 and https://github.com/firefox-devtools/profiler/blob/8da8e85846efc90971908ad6fd765f08234018a8/src/components/tooltip/CallNode.js#L243-L284 They both do interesting but different things.

Afterwards we can use that in https://github.com/firefox-devtools/profiler/blob/8da8e85846efc90971908ad6fd765f08234018a8/src/components/shared/SampleTooltipContents.js#L62, probably right before the Stack.

We could also move SampleTooltipContents from shared to tooltip in the process.

┆Issue is synchronized with this Jira Task

Guida contributor

Direzione di ricerca
Estrai e unisci il codice dell'URL della pagina dai componenti tooltip Marker.js e CallNode.js, quindi integralo in SampleTooltipContents.js. Considera di spostare SampleTooltipContents nella cartella tooltip.
Tech stack
typescriptreact
Dominio
frontend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitTypeScriptReact
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65