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

'X has conquered Y' omitted from gossip filter menu #517

Open
legdrop opened this issue May 3, 2017 · 11 comments
Open

'X has conquered Y' omitted from gossip filter menu #517

legdrop opened this issue May 3, 2017 · 11 comments

Comments

@legdrop
Copy link

legdrop commented May 3, 2017

Good afternoon, firstly, thank you very much for such a wonderful modification, which I use with the acquisition of the game.
But there is a problem - gossip on the screen about AI does not appear at all, just notifications about my studied technologies of culture and science.
How can I solve this problem? Or tell me, which file or string I should cut to get rid of the gossip removal function.
I'm playing the Russian version of the game.
Thank you very much!

@chaorace
Copy link
Member

This may be an issue with the heuristic used for intercepting gossips. I did my best to render it language neutral, but there's only so much I can achieve as an English-only speaker.

Hey @E1ectron, does this issue affect you as well?

@solo-s
Copy link

solo-s commented May 13, 2017

Sorry, I can't check it now and in the near future (don't have access to actual game version).

@solo-s
Copy link

solo-s commented May 19, 2017

Hey, @chaorace. Test it on a 1.0.4 (1.0.0.129) Mac version on Russian and English.
For a hour, I haven't seen any gossips. So problem is not the language.

May be platform? @legdrop, you play on Win or Mac?

@legdrop
Copy link
Author

legdrop commented May 19, 2017

Windows 10, 64 bit.

@bollbas
Copy link

bollbas commented May 19, 2017

I think @legdrop wants to read popup gossips that appear on vanilla on center of the screen. If so you need to remove line 35 in UI/Panels/StatusMessagePanel.lua that was added on this commit f04d0b5

@legdrop
Copy link
Author

legdrop commented May 20, 2017

@bolbass
Yes, you understood everything correctly, thank you all very much!
I do not have the option now to run the game, but please check, I selected the correct code to delete?
1

@bollbas
Copy link

bollbas commented May 20, 2017

Don't delete it :)
The commit I mentioned above was already fixed in CQUI so my last comment was wrong.

The last commits that affect gossips are c896313 and f9d919e. And this commit 6c30940 affects the size of gossip messages.
The result of these commits is that you can choose gossips you want to see or disable on CQUI settings that are above the minimap. Have you tried changing it to turn on all the gossips?
I've tested CQUI gossip settings a bit and couldn't find a gossip when other civs enter new era for example.
To see all the gossips you can also try to manually delete lines 284-293 in StatusMessagePanel.lua
On a picture above you've already selected them but first 4 lines should stay.
You should start deleting from if (type == ReportingStatusTypes.GOSSIP) then

@legdrop
Copy link
Author

legdrop commented May 21, 2017

Thanks a lot, it works!

@chaorace
Copy link
Member

Okay... so did we ever figure out if this issue was caused by CQUI not correctly showing gossips?

@bollbas
Copy link

bollbas commented May 27, 2017

Yes, there is at least one type of gossips that is missing on gossips settings screen (a gossip when other civs enter new era for example).

@bollbas
Copy link

bollbas commented Jun 2, 2017

This type of gossips is now under "City>>X has counquered Y" settings. We should fix it and can close this issue as soon as it's enough to unchek "Trim gossip messages" to see all the gossips. And maybe we need to edit a tooltip for this setting because it's not obvious that all gossips will be shown if we uncheck it.

@chaorace chaorace changed the title Missing gossip on screen 'X has conquered Y' omitted from gossip filter menu Jul 18, 2017
@chaorace chaorace added the easy label Jul 18, 2017
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

4 participants