|Anonymous | Login||2017-10-22 22:40 UTC|
|My View | View Issues | Change Log | Roadmap|
|View Issue Details|
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0002730||VCMI||Other||public||2017-07-21 07:52||2017-08-03 06:05|
|Target Version||Fixed in Version||0.next|
|Summary||0002730: The descriptions of some unit abilities are missing or incomplete, so are icons of those abilities.|
|Description||The description for "General Damage Reduction" is misleading because it's a two-part ability with specifically Ranged or Melee damage reduction. There is no mention of that in the game currently.|
"Spell Immunity" right now has no icon for it, and it's impossible to add any other via bonuses.json - it doesn't work.
There is no description for "No Morale" ability and the icon is missing too, and it has a bug connected to it which I will describe in another report.
Immunities to specific schools also have no icons despite being present in the VCMI files.
Fire Shield's icon was renamed but not updated in the bonuses.json. Same for Spell-like Attack but there's no icon in the game files.
Reduce Enemy Defense's name is too big and doesn't fit the tab on 1024*600.
Mind Immunity's description is missing.
|Steps To Reproduce||Check any of the creatures that have the respective abilities, you'll see either an icon or the text missing, or any other behaviour that I've described.|
|Tags||No tags attached.|
|Attached Files|| Bonuses fix.7z [^] (2,992 bytes) 2017-07-21 08:13|
Bonuses desc fix.zip [^] (3,700 bytes) 2017-07-30 12:15
bonuses.json [^] (6,712 bytes) 2017-07-30 13:09
Bonuses.zip [^] (3,742 bytes) 2017-07-30 14:24
|Uploaded an 7z archive which contains 2 files for VCMI which partially fix these issues and improve the descriptions of creature abilities.|
|Updated the fix based on the latest build version.|
|But your fix include new icon and abilities like SOUL_STEAL, SUMMON_GUARDIANS and TRANSMUTATION?|
|The icons for those are already present in VCMI Essential files. Reuploaded a file which referenced those new abilities, for some reason SUMMON_GUARDIANS slipped past my eyes.|
edited on: 2017-07-30 13:38
I propose also add for Cyclops King fixed ability
Yes, it's the same icon as Catapult, but looks nicer than no icon.
edited on: 2017-07-30 14:24
Ah, I've been wondering what this bonus is about. Will do later.
As of 14:24 UTC - updated.
|Fixed in https://github.com/vcmi/vcmi/pull/361 [^]|
|2017-07-21 07:52||Anubis||New Issue|
|2017-07-21 08:13||Anubis||File Added: Bonuses fix.7z|
|2017-07-21 08:14||Anubis||Note Added: 0007151|
|2017-07-30 12:15||Anubis||File Added: Bonuses desc fix.zip|
|2017-07-30 12:16||Anubis||Note Added: 0007186|
|2017-07-30 12:44||misiokles||Note Added: 0007187|
|2017-07-30 13:09||Anubis||File Added: bonuses.json|
|2017-07-30 13:10||Anubis||Note Added: 0007188|
|2017-07-30 13:36||misiokles||Note Added: 0007189|
|2017-07-30 13:38||misiokles||Note Edited: 0007189||View Revisions|
|2017-07-30 14:17||Anubis||Note Added: 0007190|
|2017-07-30 14:24||Anubis||File Added: Bonuses.zip|
|2017-07-30 14:24||Anubis||Note Edited: 0007190||View Revisions|
|2017-08-03 06:05||AVS||Note Added: 0007202|
|2017-08-03 06:05||AVS||Status||new => resolved|
|2017-08-03 06:05||AVS||Fixed in Version||=> 0.next|
|2017-08-03 06:05||AVS||Resolution||open => fixed|
|2017-08-03 06:05||AVS||Assigned To||=> AVS|
|Copyright © 2000 - 2017 MantisBT Team|