fheroes2 icon indicating copy to clipboard operation
fheroes2 copied to clipboard

Hero screen, Necromancy skill, improvement of info windows

Open LeHerosInconnu opened this issue 5 years ago • 2 comments

In the information windows of the Necromancy skill add the "Currents Modifiers:" of the skill.

For example:

"Currents Modifiers:

Spade of Necromancy +1 Shrines +2 (2)"

Other example:

"Currents Modifiers:

Spade of Necromancy +1 "Shrine +1"

Necromancy bonus 11

Necromancy bonus 12

LeHerosInconnu avatar Sep 01 '20 09:09 LeHerosInconnu

Hi @LeHerosInconnu , I am inclining not to do it. Necromancy is not the only Secondary Skill which has extra modifiers: Logistics, Leadership, Estates and others can receive modifiers from artifacts. So either do for everything properly or for nothing.

ihhub avatar Aug 25 '23 14:08 ihhub

Hello @ihhub,

Hi @LeHerosInconnu , I am inclining not to do it. Necromancy is not the only Secondary Skill which has extra modifiers: Logistics, Leadership, Estates and others can receive modifiers from artifacts. So either do for everything properly or for nothing.

Necromancy is a particular skill because it can also be affected by one or more buildings when built in one or more cities (shrine(s)). Also, the "+1", "+2" and "+3" next to the skill level (Basic, Advanced, Expert) needed to be explained to the player. That's why I've only made this proposal for necromancy for the time being. But you are right, other skills affected by artifacts should also have these modifiers in their information window. Is it better to open an issue for each skill, or to group all skills together in this same issue, or to open another issue for just the other skills?

LeHerosInconnu avatar Aug 26 '23 21:08 LeHerosInconnu