firefox-devtools/profiler

ActivityGraph Tooltip should show the sample's page

开放

#3,664 创建于 2021年11月23日

 (1 条评论) (0 个反应) (0 位负责人)TypeScript (486 个派生)auto 404
help wantedtimeline

仓库指标

星标
 (1,515 个星标)
PR 合并指标
 (PR 指标待抓取)

描述

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

贡献者指南