0003178: Redundant building requirement text

View Issue Details [ Jump to Notes ]

[ Issue History ] [ Print ]

Project

Category

View Status

Date Submitted

Last Update

0003178

VCMI

GUI - Town screen

public

2021-05-01 10:53

2024-01-30 19:18

Reporter

jlieb

Assigned To

Priority

normal

Severity

minor

Reproducibility

always

Status

closed

Resolution

open

Platform

OS Version

Product Version

Target Version

Fixed in Version

Summary

0003178: Redundant building requirement text

Description

In the original HoMM3, when a building has multiple requirements the text is similar to the following:

Requires:
Mage Guild Level 1, Demon Gate

In VCMI, the text is similar to the following:

Requires:
All of the following:
Mage Guild Level 1
Demon Gate

I think that including the qualifier ("All of the following") is unnecessary, unless there are cases where there is a logical OR for the requirements of some building (can't think of such a case at the moment).

In addition, even if the qualifier is actually necessary, I would format the text as follows:

Requires all of the following:
Mage Guild Level 1, Demon Gate

Lastly, I would separate the requirements using commas rather than newlines as in the original game.

Tags

No tags attached.

Attached Files

vcmi.png [^] (252,919 bytes) 2021-05-01 10:53

original.png [^] (31,288 bytes) 2021-05-01 10:53

Relationships

Notes
(0008605) krs (reporter) 2023-04-14 18:00	Confirmed! Moved to Git: https://github.com/vcmi/vcmi/issues/1988 [^] Can be closed.

(0008894) Ivan (developer) 2024-01-30 19:18	Closing issues that were migrated to Github

Issue History
Date Modified	Username	Field	Change
2021-05-01 10:53	jlieb	New Issue
2021-05-01 10:53	jlieb	File Added: vcmi.png
2021-05-01 10:53	jlieb	File Added: original.png
2023-04-14 18:00	krs	Note Added: 0008605
2023-04-15 12:30	Ivan	Status	new => confirmed
2024-01-30 19:18	Ivan	Note Added: 0008894
2024-01-30 19:18	Ivan	Status	confirmed => closed

Site | Forums | Wiki | Slack | GitHub