yairm210 / Unciv

Open-source Android/Desktop remake of Civ V
Mozilla Public License 2.0
8.6k stars 1.58k forks source link

Vanilla ruleset : path to "Nanotechnology" not displayed properly #10357

Closed Anon177013 closed 1 year ago

Anon177013 commented 1 year ago

Also the only required tech is "Particle Physics", I don't know if it's the expected behavior.

Platform Android 12

Version 4.8.14 and before (already present in 4.8.8 if I remember correctly) Vanilla ruleset only

Screenshots Screenshot_20231025-202541 Screenshot_20231025-202602

SomeTroglodyte commented 1 year ago

I doubt that's pure vanilla, techs look fine to me - oh wait maybe you mean vanilla vanilla... Yup, you're right :rofl: - haven't looked at that in ages.

SomeTroglodyte commented 1 year ago

We should just move Future Tech:

![Unciv-tech](https://github.com/yairm210/Unciv/assets/63000004/d43a6cf6-c602-4ca0-87f3-3805079f8727)

This is row 2 to 9 animated, I'd say - row 7 or 8?

Anon177013 commented 1 year ago

Wouldn't it be clearer by moving the vertical lines to the right of Nanotechnology?

SeventhM commented 1 year ago

I'd say 6 or 7. Also, what is this cursed tech tree vanilla has?

SomeTroglodyte commented 1 year ago

moving the vertical lines

That would be code change for a rare border case - weighted against changing just one json character...

cursed

You mean because it's Halloween or Samhain or whenever that commerce is established? At least currently Shattered has pumpkin slices instead of pastries, so that must be close... 🎃 🥧

Or do you mean why do dependencies go over more levels than in G&K? Dunno. Didn't doubt those definitions.

SeventhM commented 1 year ago

I mean this ghost town of a final era. Between 2 columns there's only 3 techs, despite the efforts to have 5 techs per column