Skip to content

Conversation

HansMuller
Copy link
Contributor

Heroes are now identified exclusively by their object-valued tag.

Fixes #5403

@HansMuller
Copy link
Contributor Author

cc @Hixie

key: new ObjectKey(stock),
tag: stock,
turns: 2,
child: new StockArrow(percentChange: stock.percentChange)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

since you're passing in the stock anyway, why not do this in the StockSymbolView builder?

@Hixie
Copy link
Contributor

Hixie commented Aug 19, 2016

This patch is beautiful. LGTM.

@HansMuller HansMuller merged commit 03b117a into flutter:master Aug 19, 2016
@HansMuller HansMuller deleted the remove_hero_mvp_keys branch August 19, 2016 16:59
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove the mostValuableKeys concept and have heroes exclusively look at tags, not keys
2 participants