search

We-the-People-civ4col-mod / Mod

This is the repository where the mod resides.
90 stars 37 forks source link

Quests in notes: what to do? #393

Open Sorxus opened 3 years ago

Sorxus commented 3 years ago

Currently you get some quests and in notes (sheet Quests) you can see about those quests.

Quests-what to do

But there isn't specific info what to do.

My suggestion is that if you hoover over quest there should be written "build 4 harbours" or "have 5 units hunters" etc.

devolution79 commented 3 years ago

I agree that this would be rather useful. It should be a simple matter of parsing the quest requirements (from their respective properties) and then displaying it somewhere.

raystuttgart commented 3 years ago

Somebody simply needs to take the time to rewrite all the text entries ... It is an incredibly tedious todo but it is not difficult.

So if sombody volunteers to do it, I am of course absolutely fine with it. I am not interested to be that volunteer though - because I do not feel it is a problem.

Careful though if you "hardcode" numbers into atmospheric text that are balanced in the Events XML. Those 2 files have absolutely no connection and once you change balancing the texts will not match anymore.

Summary: Do not write any numbers in the XML texts that are stored in Quest log. You will regret it if you do, because you need to keep them consitent with balancing XML. This is extremley tedious maintenance effort that nobody will ever take care of.

MrZorG33 commented 2 years ago

And one more thing: it would be nice to show how far you have already progressed in completing the quest.

pedro-walter commented 2 years ago

Since this issue was started more than two years ago, not sure if it's still relevant. Came here looking to suggest that we get quest progression, like the event that tells you that you have to trade X amount of goods with Europe, the current progress could also be displayed there.

Should it be an issue on it's own or is it fine to be kept here? I could contribute to this with some pointers (no C pun intended).

devolution79 commented 2 years ago

I suppose that should take back the claim that this is a "good first issue" but I still think that there's some low-hanging fruit to be picked here. Alas, we (the coders) are currently rather short-staffed so it would be very helpful if any new contributes were to stand up. Any takers ?

raystuttgart commented 1 year ago

@raubwuerger Maybe this can be combined with what you want to do for "Trade Quests Screen".?