Opened 7 years ago

Last modified 4 years ago

#4839 assigned defect

Broken camera placement for observers — at Version 2

Reported by: elexis Owned by: elexis
Priority: Nice to Have Milestone: Backlog
Component: UI – In-game Keywords:
Cc: Patch:

Description (last modified by elexis)

The camera position should always be centered in observermode. r20370 removed some code that pretends to center the camera. But that only works for observers, it should be on a per-player basis as long as the other maps 'support' that too. My impression that ParseCamera in MapReader.cpp is unusably broken. The Player component also has camera positioning code which appears unused.

The GameView code always focuses on the first CC if there exists one. Thus overwriting the StartingCamera from the (skirmish/scenario/random) map if one exists, reverting its only use case.

Change History (2)

comment:1 by elexis, 6 years ago

When starting Sicila Nomad in observermode we have the same issue. Since that is not a random map, the problem is unrelated from the rmgen/ camera centring removal and it certainly can be reproduced in alpha 22.

Still don't see the use in this StartingCamera feature at all, so it should be removed and the camera should always center in observermode (independent of maptype).

comment:2 by elexis, 6 years ago

Description: modified (diff)
Milestone: Alpha 23Backlog
Priority: Must HaveNice to Have

Perhaps the StartingCamera feature could be used in a campaign where we do have a civic center + a hero somewhere else and it should focus the hero instead.

The feature is still broken in in the map loader, in the simulation and in the gameview afaics. Most maps likely have a starting camera value that should be removed in case the code will sometime prefer the starting camera of players specified in the map.

Note: See TracTickets for help on using tickets.