augustus icon indicating copy to clipboard operation
augustus copied to clipboard

Feature/god advisor

Open dvincent56 opened this issue 1 year ago • 18 comments

Adding an information panel in religion advisor to see all epithets description for each gods. The jupiter asset is a placeholder, should be replace in future

dvincent56 avatar Jan 28 '24 20:01 dvincent56

Some screenshots: image image

Ouaz avatar Jan 28 '24 21:01 Ouaz

Can you please solve the merge conflicts first? Thanks!

crudelios avatar Jan 31 '24 09:01 crudelios

epithets4 Update color for readability

dvincent56 avatar Feb 12 '24 20:02 dvincent56

Perhaps change the white font in the last image for the one used in the titles in the second image. The more pixelated white font in the last image is a bit visually jarring compared to the smoother font - which is used more commonly across the game's UI.

Areldir avatar Mar 10 '24 11:03 Areldir

Perhaps change the white font in the last image for the one used in the titles in the second image. The more pixelated white font in the last image is a bit visually jarring

The normal white font is too big and the 3 epithet descriptions do not fit in the box. :(

This tiny font trick is used in C3, when there's not enough space, e.g.:

image

Ouaz avatar Mar 11 '24 18:03 Ouaz

I have just added an asset to replace the AI generated portrait/button of Zeus (Commit: d5cdab0 - asset ID "Pantheon_Epithet_Button_01"), as it is not fitting for the Pantheon and is also the source of some controversy due to AI being used in its creation.

Please change over to the new image and remove the AI generated one. The button can be created from the image using the same method as the Colosseum games buttons, with the borders and vignette drawn from the code.

Areldir avatar Apr 01 '24 06:04 Areldir

I think it's good right now

dvincent56 avatar Apr 01 '24 11:04 dvincent56

Well, it has merge conflicts now. Can you please fix them?

crudelios avatar Apr 04 '24 12:04 crudelios

Done

dvincent56 avatar Apr 04 '24 19:04 dvincent56

It has merge conflicts again 😭

crudelios avatar Apr 11 '24 09:04 crudelios

Conflicts resolved again !

dvincent56 avatar Apr 11 '24 18:04 dvincent56

but this window is just static game information.

Yes, it allows to check all god's epithets without the need of opening the manual each time (I think nobody knows them by heart ^^). It's more convenient to have these infos available directly in-game without the need of alt-tabbing, imo.

We put it in the Religion advisor, like the advisor informs the governor about what will be granted by gods if a Great Temple and its epithet is dedicated to them.

You also can't change advisors while you don't click the arrow.

That's the common behavior on advisor panels having a sub-panel ("show Rome prices" on Trade Advisor, "Hold Games" on Entertainement Advisor, "Hold new festival" on Religion Advisor, etc.)

To exit, you must click the little arrow at the bottom, I think it would be better to keep a button on the same spot where you pressed to get to the Epiteph info window anyway.

Hmmm... Exit button on the top right corner would look awkward (as all exit buttons are on the lower right corner). To make the exit more noticeable, what about using the "Right-click to Exit" string (<group id="13"><string id="1">), used for example in the "Show Rome Prices" subpanel?

image

Ouaz avatar Apr 12 '24 20:04 Ouaz

We put it in the Religion advisor, like the advisor informs the governor about what will be granted by gods if a Great Temple and its epithet is dedicated to them.

Fine by me. A bit unrelated but the smaller font may be avoided by using rich text and a scrollbar instead of regular text. However there's no support for centered titles using rich text (yet).

That's the common behavior on advisor panels having a sub-panel ("show Rome prices" on Trade Advisor, "Hold Games" on Entertainement Advisor, "Hold new festival" on Religion Advisor, etc.)

It's true, but those windows are smaller or darken the background, ie it's clear they are sub windows.

In this case that doesn't happen, as the window is exactly the same size, so it seems to be an alternative display to the religion advisor.

It's like the original housing advisor, it was also a secondary panel but it allowed moving between advisors. Only later did we add a proper tab for it.

Anyway, maybe darkening the entire background would do the trick!

Oh and also I'd rather have the button than the "right click to exit" text.

crudelios avatar Apr 12 '24 20:04 crudelios

A bit unrelated but the smaller font may be avoided by using rich text and a scrollbar instead of regular text.

Problem with the scroll bar is epithets are not a continuous text, and it displayed epithets only one by one, while scrolling the bar (which was very awkward). Hence the no-scroll-bar, the giant panel to fit all epithets in one box (needing the smaller font size though). :massage:

Ouaz avatar Apr 12 '24 21:04 Ouaz

A bit unrelated but the smaller font may be avoided by using rich text and a scrollbar instead of regular text.

Problem with the scroll bar is epithets are not a continuous text, and it displayed epithets only one by one, while scrolling the bar (which was very awkward). Hence the no-scroll-bar, the giant panel to fit all epithets in one box (needing the smaller font size though). 💆

Yeah we'd be able to make it continuous if I changed the way rich text works.

Maybe something to consider for the future...

crudelios avatar Apr 12 '24 22:04 crudelios

I'm going to have a go at adding a @H option to rich text so it has headings, that way we can easily add the scrollbars.

I don't that will take too long.

This PR can stay on hold for that.

crudelios avatar Apr 18 '24 11:04 crudelios

Yes I need to revamp this one @Ouaz sent me the update I need to do epithet_UI_bar

dvincent56 avatar Apr 18 '24 17:04 dvincent56

I've now added headings to the rich text.

By using @H you'll add a heading, then @L or @P closes the heading.

They look basically like the heading in the images here, with a bit more spacing between the heading and the next line.

crudelios avatar Apr 19 '24 06:04 crudelios