Allow Observers to show building spaces

Bug #1419537 reported by GunChleoc
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
widelands
Fix Released
Wishlist
Martin Schmidt

Bug Description

As it says on the tin ;)

Observers should see all player's building spaces when hitting the space bar.

https://wl.widelands.org/forum/topic/1646/

Tags: ui

Related branches

SirVer (sirver)
Changed in widelands:
status: New → Confirmed
Changed in widelands:
assignee: nobody → Martin Schmidt (martinschmidt)
Revision history for this message
Martin Schmidt (martinschmidt) wrote :

Ok, observers can now see building spaces as well (from all players)
It is probably good to test/review the branch before merging

Revision history for this message
TiborB (tiborb95) wrote :

I compiled it, and it works and it is great.

Would it make sense to show bulding spaces also on unowned territory? As in editor. What do you think?

Revision history for this message
SirVer (sirver) wrote : Re: [Bug 1419537] Re: Allow Observers to show building spaces

> Am 12.02.2015 um 13:41 schrieb TiborB <email address hidden>:
>
> I compiled it, and it works and it is great.
>
> Would it make sense to show bulding spaces also on unowned territory?
> As in editor. What do you think?

Interesting question. It boils down to: should the observer have information that none of the players have or should the information of the observer be at most (and ideally) be the Union of the information of all players.

I'd say that requires a bit more discussion in a new bug). Right now we build more towards the second, commercial game clearly go towards the first.

I think this should be merged first and the rest discussed in a bug. I did not review this (on the phone right now) so no comment about readiness for merging right now from me :). Could you send a merge proposal?

>
> --
> You received this bug notification because you are subscribed to
> widelands.
> https://bugs.launchpad.net/bugs/1419537
>
> Title:
> Allow Observers to show building spaces
>
> Status in Widelands:
> Confirmed
>
> Bug description:
> As it says on the tin ;)
>
> Observers should see all player's building spaces when hitting the
> space bar.
>
> https://wl.widelands.org/forum/topic/1646/
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/widelands/+bug/1419537/+subscriptions

Revision history for this message
SirVer (sirver) wrote :

Merged in r7404.

Changed in widelands:
status: Confirmed → Fix Committed
milestone: none → build19-rc1
GunChleoc (gunchleoc)
Changed in widelands:
status: Fix Committed → Fix Released
Revision history for this message
GunChleoc (gunchleoc) wrote :

Fixed in build19-rc1.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.