MantisBT - VCMI
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0001143VCMIGUI - PreGamepublic2012-10-21 20:282014-05-30 17:41
Reporterimfz99 
Assigned ToTow dragon 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version0.9 
Target VersionFixed in Version0.92 
Summary0001143: Bug with campaign bonus selection
DescriptionWhen selecting bonus for campaign, start button doesn`t become active when selecting the first bonus with the first selection. But, it becomes active, if this bonus is selected after the button has already been active
Steps To Reproduce1. Start new campaign
2. Select _first_ bonus from the left.
3. The start button is not active
4. Select any other bonus, and the start button becomes active
Additional InformationLooked down in the code, and the quick fix looks like this -

file : CPreGame.cpp,

line : if (id != sInfo.campState->currentBonusID())

this line`s intent looks like trying to avoid the redraw of all the buttons in case the selection has not changed. Sadly enough, it cannot differ absence of selection (initial state 0) from selecting the first cell - state 0 also. Therefore, imho, it is possible to make absence of selection look like some other number, or, if it`s intentional to have state 0 selected by default - make start button enabled by default,
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2012-10-21 20:28imfz99New Issue
2012-10-21 20:28imfz99Statusnew => assigned
2012-10-21 20:28imfz99Assigned To => Tow
2013-02-05 18:49Tow dragonNote Added: 0003356
2013-02-05 18:49Tow dragonStatusassigned => resolved
2013-02-05 18:49Tow dragonFixed in Version => 0.92
2013-02-05 18:49Tow dragonResolutionopen => fixed
2013-02-05 18:54Tow dragonAssigned ToTow => Tow dragon
2013-02-05 18:54Tow dragonStatusresolved => assigned
2013-02-05 18:54Tow dragonStatusassigned => resolved
2014-05-30 17:41beegeeStatusresolved => closed

Notes
(0003356)
Tow dragon   
2013-02-05 18:49   
Fixed in rev 3158