bug(stories): Clarify graph icon names

[rbro112]

Was working on some new UI and using stories, noticed the naming for the IconGraph was duplicated for all other graph icon types (IconGraphCircle, IconGraphBar, etc).

This correctly names each now.

[rbro112]

• bug(stories): Clarify graph icon names #96495 👈 (View in Graphite)
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

[cursor]

Bug: Graph Icon Naming Causes Component Lookup Errors

The name property for graph icons (circle, bar, area, scatter) was changed from 'Graph' to specific variants (e.g., 'GraphCircle'). This causes component lookup failures because these icons share a single IconGraph component, differentiated by type props, but the new names cause lookups for non-existent IconGraphCircle components. The line graph icon correctly retains the 'Graph' name.

Locations (1)

• static/app/icons/icons.stories.tsx#L1128-L1164

Fix in Cursor • Fix in Web

[ryan953]

I believe <IconGraph> refers to the file : https://github.com/getsentry/sentry/blob/master/static/app/icons/iconGraph.tsx#L12-L25
People can use <IconGraph type='bar' /> to make their bar chart.

Having the stories recommend <IconGraphBar /> directly is a fine change, but then we should remove the defaultProps: {type: "bar"} business in the stories too.

We could also go all the way along and look at why is <IconGraph /> even there at all, maybe just purge the whole site of <IconGraph/>

[JonasBa]

Thank you @rbro112

[getsantry]

This pull request has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you add the label WIP, I will leave it alone unless WIP is removed ... forever!

---

"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀