Opened 7 years ago

Last modified 10 months ago

#4299 assigned defect

[PATCH] Music track should not restart when switching the menu

Reported by: elexis Owned by: Grapjas
Priority: Nice to Have Milestone: Alpha 27
Component: Music & Sound FX Keywords: patch, simple
Cc: Patch: Phab:D4922

Description (last modified by Stan)

The currently played track is restarted when entering the multiplayer lobby, or when returning from any page to the main menu (whenever global.music.setState is called in the JS files).

The number of resets should be minimized - it is basically only needed when first starting the game and when leaving the summary screen after ending a game (the summary screen should still play the win / defeat music of the players civ).

Attachments (1)

4299_music.patch (1.9 KB ) - added by Vladislav Belov 7 years ago.
Removes the music reset

Download all attachments as: .zip

Change History (13)

comment:1 by Vladislav Belov, 7 years ago

Should any music plays in the summary screen, when player exited (not finished a game)?

Last edited 7 years ago by Vladislav Belov (previous) (diff)

by Vladislav Belov, 7 years ago

Attachment: 4299_music.patch added

Removes the music reset

comment:2 by Vladislav Belov, 7 years ago

Keywords: patch rfc added
Milestone: BacklogAlpha 22
Owner: set to Vladislav Belov
Status: newassigned
Summary: Music track should not restart when switching the menu[PATCH] Music track should not restart when switching the menu

in reply to:  1 ; comment:3 by elexis, 7 years ago

Keywords: rfc removed

Replying to vladislavbelov:

Should any music plays in the summary screen, when player exited (not finished a game)?

For my taste, there should always be some music track in the background. So that would indeed be better to play the main menu track at that point, not only when entering the menu.

Also it seems weird to mute the music if there is a message box popping up in singleplayer (i.e. when the game is paused), would you agree?

in reply to:  3 comment:4 by Vladislav Belov, 7 years ago

Replying to elexis:

For my taste, there should always be some music track in the background. So that would indeed be better to play the main menu track at that point, not only when entering the menu.

Ok, I will implement the default music in the summary.

Also it seems weird to mute the music if there is a message box popping up in singleplayer (i.e. when the game is paused), would you agree?

I agree: not mute but set the volume to 0.5x or something like this.

comment:5 by elexis, 7 years ago

Milestone: Alpha 22Work In Progress

Moving to the Work In Progress milestone, since there is a patch asking for feedback, but since it is not strictly bound to a specific release.

comment:6 by Silier, 3 years ago

Keywords: simple removed
severity: simple

comment:7 by Silier, 3 years ago

Keywords: simple added

comment:8 by Stan, 15 months ago

Description: modified (diff)
Milestone: Work In ProgressBacklog
Owner: Vladislav Belov removed
Status: assignednew

comment:9 by Grapjas, 15 months ago

Milestone: BacklogAlpha 27
Owner: set to Grapjas
Status: newassigned

comment:10 by Grapjas, 15 months ago

Patch: Phab:D4922

comment:11 by Stan, 11 months ago

Milestone: Alpha 27Alpha 28

comment:12 by Stan, 10 months ago

Milestone: Alpha 28Alpha 27

Push back

Note: See TracTickets for help on using tickets.