Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build System pt.4 - Persistence, Surface alignment, gui improvements #519

Merged
merged 34 commits into from
Nov 5, 2018

Conversation

veteran29
Copy link
Member

@veteran29 veteran29 commented Nov 5, 2018

Q A
Bug fix? no
New feature? yes
Needs wipe? no
Fixed issues #507, #505

Description:

Further development of build system

Content:

  • Align objects to surface
  • Show objects target position bounding box, changed to show icon at target area
  • Allow to place objects on surfaces like roofs
  • Add icons for dragging
  • Allow to build only in FOB area
  • Fix boats spawning at bottom of water containers
  • Object persistence

Tested on:

  • Local MP Vanilla
  • Local MP ACE
  • Dedicated MP Vanilla
  • Dedicated MP ACE

@veteran29 veteran29 requested a review from Wyqer November 5, 2018 02:13
@veteran29 veteran29 added this to the Sprint 7 milestone Nov 5, 2018
Copy link
Member

@Wyqer Wyqer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks basically good to me. Just many small documentation/formatting stuff to remember.
Good work. 👍

Missionframework/modules/04_build/fnc/fn_build_preInit.sqf Outdated Show resolved Hide resolved
Missionframework/modules/04_build/fnc/fn_build_start.sqf Outdated Show resolved Hide resolved
@veteran29 veteran29 self-assigned this Nov 5, 2018
@veteran29 veteran29 merged commit dd10ca7 into v0.97S7 Nov 5, 2018
@veteran29 veteran29 deleted the v0.97S7-507 branch November 5, 2018 11:05
@Wyqer Wyqer added done and removed review/QA labels Nov 5, 2018
@Wyqer Wyqer mentioned this pull request Nov 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants