Tiles

[jordankoschei-okta]

No description provided.

[jordankoschei-okta]

Add generic if possible

[KevinGhadyani-Okta]

Callback missing onClose in dependencies.

[KevinGhadyani-Okta]

Missing onClose in dependency arg.

[KevinGhadyani-Okta]

Can we memoize these values with useMemo?

[KevinGhadyani-Okta]

I wonder if we can change these to never?

[KevinGhadyani-Okta]

Can we useMemo these?

[KevinGhadyani-Okta]

Ideally, these styles would be part of the theme.

[KevinGhadyani-Okta]

Can we set this context at the top of the component above the function, so it's a static memory reference? const buttonContextValue

[KevinGhadyani-Okta]

This can be a ternary.

[KevinGhadyani-Okta]

Can we modify this label as a prop? menuButtonLabel

Also, it's not translated.

[KevinGhadyani-Okta]

Can we memoize these?

[KevinGhadyani-Okta]

Remove this file and add to .gitignore.

git rm --cached packages/odyssey-react-mui/src/properties/ts/.ts.DS_Store

[KevinGhadyani-Okta]

Let's add a ticket to fix this.

[KevinGhadyani-Okta]

Can this a direct child of the root?

& > img?

[KevinGhadyani-Okta]

Suggested change

	const storybookMeta: Meta = {
	const storybookMeta: Meta<TileProps> = {

[KevinGhadyani-Okta]

A bunch of small comments.

[github-actions]

🤖 This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

[jordankoschei-okta]

Closing because this is stale and needs some serious rework