Closed tyrken closed 5 years ago
No problem, I'm not really sure what I'm doing either - but @Azurency posted a helpful review on my PR, so check that out as well.
OK, for worldviewiconsmanager.lua I've added in the =>256 on line 93 and checked the feedback from #383.
TBH the second parameter to IconManager:FindIconAtlas
is on closer inspection a size, so the new 256 value here & where I add the debatable comment just above might actually be wrong (UI from the mod might not have room compared to the new stock UI). Should be pretty clear once we get the whole thing play-testable.
I'll try to make a review this week 😉
In unitflagmanager.lua do we need something to balance the CQUI instance.FlagBaseDarken on lines 522-536 (War Check), e.g. instance.FlagBaseLighten:SetColor( primaryColor );
?
Got this comment in Screens/governmentscreen.xml: <!--Filters are added dynamically through AddFilterTabButtons-->
... hoping they actually meant RealizeFilterTabs in the matching lua file, added an equivalent there.
In line 1114 of Screens/governmentscreen.lua, I reverted a CQUI change of the variable name from "strSlotType" to "rowSlotType", that looked like a mistake.
screens/civicstree.lua: Line 585 has CQUI removing an else that was there in original & update, I have NOT removed it as it looked wrong, please confirm!
if node["unlockGOV"] == nil then
node["unlockGOV"] = InstanceManager:new( "GovernmentIcon", "GovernmentInstanceGrid", node.UnlockStack );
else <--- This else was deleted by CQUI!
node["unlockGOV"]:DestroyInstances()
end
replacements/* - can you explain what is meant to be merged with what? Confusing...
popups/techciviccompletedpopup.lua - I can't really see where to put any of the changes from CQUI, so reverted to updated 290 version instead.
popups/mappinlistpanel.xml - no changes needed
@Azurency have you managed to start code review yet?
@tyrken not this pull request, I'm on #382 for now
Great, thx for the contribution 👍 🎉
Here is another attempt at helping with the massive changes, starting with just a couple of files in case I'm doing something wrong - please provide feedback as it's very difficult doing it blind - no syntax/lint & little lua knowledge. Thankfully found a old but useful GUI for 5-way merges in http://diffuse.sourceforge.net/index.html I drive with a space-normalisation script.
Sometimes it's been easier to start from a tab-to-spaced form of the updated 290 code and just add back in the CQUI changes that, but sorry this makes a lot of diffs to your CQUI expansion2 branch.
Many apologies @accatyyc I didn't notice until completed that you'd also started at the bottom working on worldviewiconsmanager.lua, please review my changes against what you have!