Skip to content
This repository has been archived by the owner on Dec 18, 2017. It is now read-only.

Making the yields in the top panel clickable #8

Open
chaorace opened this issue Oct 29, 2016 · 2 comments
Open

Making the yields in the top panel clickable #8

chaorace opened this issue Oct 29, 2016 · 2 comments

Comments

@chaorace
Copy link
Member

Self-explanatory, I'm thinking science icon links to science victory ui, religion to religious victory ui, tourism to culture victory ui, trade to trade ui, diplomats to city state ui, luxury icons to City Status page in reports ui. Unsure of what to do with tourism/gold yields

@jacks0nX
Copy link

Personally I would only make the yields (science, culture, faith, gold) clickable. The components for diplomats and trade routes doesn't really suggest "this is clickable" to the playe, whereas the first four do.

I already implemented this for me, will commit if wanted.

@chaorace
Copy link
Member Author

I'll happily accept a PR if you send one this way! That being said, I can't see a reason why the trade/diplo icons shouldn't go somewhere, there's just no downside.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants