Page MenuHomeWildfire Games
Differential D5130

Move the game loading out of the SavegameLoader
Needs ReviewPublic
Actions

Authored by phosit on Sat, Sep 16, 2:45 PM.

Details

Reviewers
Freagarach
Summary

The SavegameLoader should not know that it has to start a game afterwards.
Note: Now if there's an Error in Engine.StartSavedGame(gameId); the Dialog is still closed.

This simplifies D4770.

Test Plan

Load a game from the from the main menu and the campain page.
Also test other ways the dialog can be closed (cancel, saveing of a game).

Diff Detail

Event Timeline

phosit created this revision.Sat, Sep 16, 2:45 PM
Vulcan added a comment.Sat, Sep 16, 2:46 PM

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/macos-differential/7290/display/redirect

phosit added a comment.Sat, Sep 16, 2:47 PM

There is duplication but i didn't know where to put the function. Propably "gui/common/functions_global_object.js"

Vulcan added a comment.Sat, Sep 16, 2:57 PM

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/8379/display/redirect

Harbormaster completed remote builds in B22339: Diff 22262.Sat, Sep 16, 3:03 PM
phosit requested review of this revision.Sat, Sep 16, 3:03 PM
phosit updated this revision to Diff 22276.Sun, Sep 17, 5:56 PM
phosit edited the summary of this revision. (Show Details)

Make it a member function in CampaignMenu.

Vulcan added a comment.Sun, Sep 17, 5:59 PM

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/macos-differential/7302/display/redirect

Vulcan added a comment.Sun, Sep 17, 6:04 PM

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/8391/display/redirect

Harbormaster completed remote builds in B22351: Diff 22276.Sun, Sep 17, 6:32 PM
phosit updated this revision to Diff 22280.Mon, Sep 18, 6:47 PM
  • Add forgoten .bind(this)
  • Inline pData

Based on a review by elexis.

Vulcan added a comment.Mon, Sep 18, 6:47 PM

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/macos-differential/7304/display/redirect

Vulcan added a comment.Mon, Sep 18, 6:50 PM

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/8393/display/redirect

Harbormaster completed remote builds in B22356: Diff 22280.Mon, Sep 18, 7:02 PM
phosit updated this revision to Diff 22315.Sun, Sep 24, 3:51 PM

Use .bind instead of arrow functions more often.
Correct a indent mistake.

Vulcan added a comment.Sun, Sep 24, 3:53 PM

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/macos-differential/7329/display/redirect

Vulcan added a comment.Sun, Sep 24, 3:58 PM

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/8418/display/redirect

Harbormaster completed remote builds in B22389: Diff 22315.Sun, Sep 24, 4:23 PM

Revision Contents
Changeset List

PathSize
binaries/
data/
mods/
public/
gui/
campaigns/
default_menu/
31 lines
loadgame/
27 lines
pregame/
28 lines

Diff 22315

View Options

binaries/data/mods/public/gui/campaigns/default_menu/CampaignMenu.js

Loading...
View Options

binaries/data/mods/public/gui/loadgame/SavegameLoader.js

Loading...
View Options

binaries/data/mods/public/gui/pregame/MainMenuItems.js

Loading...
Wildfire Games · Phabricator