* Add player initilization safety to setSpectator
Public function which should wait for player unit to exist before running.
With this change an additional variable is now set on the player unit which
can be used to check whether players are spectators or not (part of
grander plans).
* Add UI initalization safety to setSpectator
Should make it so that calling the public function before the main display
has initalized will correctly wait until it has before starting spectator
* Overhaul spectator
Initial WIP overhaul of spectator component. Based on BI's End Game
Spectator and AACO's work on POTATO spectate. The essential functionality
is in place, but some things are broken and some are still to be
transferred.
* Remove redundant canInteract condition
Spectator UI will be open anyway so this is pointless.
* Fix follow camera controls
Also ensure unit camera modes have a target more consistently
* Fix widget weapon icon for unarmed
* Fix follow camera zooming
* Fix entity list not updating correctly
* Remove redundant conditions
* Fix list click behaviour
* Remove a bunch of old debug
* Alter behaviour of camera speed
* Restructure the spectator addon
* Fix camera tracking behaviour
* Fix very fast camera toggle
* Fix initial camera mode
* Fix double click behaviour
* Make AI name display consistent
* Add UI toggle
* Fix current camera mode button highlighting
* Fix focus info widget when clicked on in 3D world
* Improve key handling
Use of `inputAction` could result in weirdness when multiple keys pressed
at the same time. Hardcoded for now.
* Implement dynamic help UI element
* Remove some redundant code
* Fix follow camera jitter
* Add projectile drawing
Technically the capability to draw projectiles has been in since the
start. However it took me a bit of time to figure out how best to
add/remove the required event handlers.
I decided to add them during the process of finding things to draw, since
I'm iterating over the list of spectatable entities here anyway. With an
additional GVAR containing up to date entities, the fired EHs can remove
themselves when applicable. This avoids any granularity issues with adding
EHs en masse.
* Improve naming convention related to drawing
* Remove outdated unit and side settings
Also re-add unit whitelisting/blacklisting support
* Rename public function for consistency
Adds a deprecated warning to the old function name.
* Update stringtable
Some entries are replaced with BI translations, others are no longer
needed. I pieced together one of the camera mode settings by combining the
BI translations of the individual versions.
* Add location public functions and list population
Just need to take care of drawing and then camera interaction now
* Add location 2D and 3D drawing
* Respect whitelist and blacklist correctly
* Add location camera focus capability
* Fix local helicopters follow mode bug
* Add compatibility with `counter` template
Simply repositions the counter UI to the top of the screen out of the way
* Update respawn template
This template is only compatible with respawn types 2 & 3
* Rename compatibility function
* Add compatibility for respawn types 1,4 and 5
* Update setCameraAttributes function
* Fix draw3D for respawn types 1,4 and 5
* Fix screeching seagulls #4265
* Fix script command typo
* SQF Linter Pass
* Add obsolete warning for public func params
* Deprecate interrupt function
* Fix incorrect function call
* Make playable virtual spectator work
* Improve focus handling in public function
- User might want to reset focus for free cam, in which case objNull
cannot be reserved for random
- Focus is still required for unit camera
- Update some function headers
* Note explicitly the use of `player`
* Remove unused base classes
* Improve seagull culling
* Add projectile drawing to vehicles
* Fix same tab selection clearing list
* Make projectile lines last 1 frame longer
* Add true zeus compatibility
- Re-opens spectator when zeus closes
- Transfers the curator module over in the respawn template
- Only handle uiForced in setSpectator to avoid issues with esc key
- Add safety to setFocus so that it only updates the UI when applicable
* Fix player moving in FPP
* Fix zeus transfer and re-opening
* Consolidate icon definitions
* Fix AI setting for player groups
* Improve handling of alive spectators
Instead of disabling simulation, just stage them automatically if alive.
* Fix respawn template for types 2 & 3
* Fix pre-set camera mode with no focus
- Improve handling of UI updates in camera related functions
- More explicit UI initalisation
- Find a focus if one wasn't pre-set for a camera mode which requires a
focus
* Localise key names
* Fix camera mode and vision settings on init
When spectator starts the available camera modes and vision modes should
be consulted to ensure they are respected upon opening.
* Fix key localisation
* Add next/prev entity keys
* Fix vision modes update while camera active
- Updates the camera vision mode in case the current was removed
* Add public getCameraAttributes function
* Fix spectator zeus compatibility
- Camera control wasn't returned on exit due to change in fnc_ui
* Improve vision mode handling
- Curator now resets vision mode on entry
- First person also reset vision mode so that it is consistent between
camera changes
* Improve location handling
- It makes more sense to specifiy the position ATL since the text is going
to be rendered there and a z of 0 is then ground height.
- Updates the ui immediately if the list is already on locations
* Disable list keyboard search
* Fix handling of forced camera mode with no focus
* Fix handling of free mode removal while no target
* Fix locations respecting available modes
* Fix list entry removal code
- Replace getTargetGroups with getTargetEntities for consolidated
filtering.
- Fix the step of the removal loops and make them go to index 0 instead of
1
* Improve side handling of unit list
- Accounts for possibly of whitelisted random objects that belong to other
sides
- Move the index decrement to loop, not within every loop
* Add linter ignore directive
* Fix list code
* Improve entity fetching
- Change whitelist/blacklist application to optimize the code
* Include dead units in the list
- Allow getTargetEntities to return dead units too
- Iterate over allGroups in list updates to reduce overhead
* Take care of TODOs
* Mark stageSpectator as non-public
* Fix updating list focus
* Adjust default camera speed
* Include the function file
* Generalise camera button UI
- Makes it easier to extend with future camera mode buttons
- Removes potential error when display doesn't exist
* Remove obsolete staged event
- Add vision toggle to help
- Restructure some function PREP
* Add public function to return all spectators
* Change default briefing name used on map display
* Fix the order of 4 main sides in list
* Restore non-toggle behaviour of camera speed
* Fix UI gap between list and tabs
* Force update UI list when player is staged
Hopefully this fixes any issues related to the player appearing in the
list
* Update controls strings
* Fix entity tooltips showing short name
- Also revert changes to common function and just handle in this component
* Include dead in unit switching
* Show rank and vehicle icon in list
- Rank is shown in tooltip
- Vehicle icon is used to see which units in the list are mounted
* Change default vehicle icon
* Show whether medic/engineer in tooltip
* Improve list double click behaviour
* Improve group icon handling
* Use dynamic group icons for drawing
* Fix EFUNC call
* Update all function headers
* All the changes I had staged before arma broke git
Arma managed to crash my whole system again, breaking my git repo and I
resorted to nuking it and pasting the changes I had comitted locally back
in. From memory:
- Fix icon paths
- use distanceSqr for minor speed boost
- Add setting to show map locations in the UI
- Limit location rendering based on distance
- Probably some other crap
* Fix squared comparisons
* Remove unnecessary variable publicity
* Deprecate fnc_stageSpectator
The function was a source of confusion alongside fnc_setSpectator and so
it is replaced by an additional parameter to hide the unit as part of
fnc_setSpectator. No longer is the unit moved either, to reduce complexity
and give more control to mission maker.
* Fix undefined variable
* Update spectator framework doc
* Update spectator feature doc
* Fix typos and grammar
* Revent version introduced number
* Update with new changes
* Everything
* Fixed missing ;
* Fix missing ; and double private
* Fixed cannot isNull on number
* Turn _temparture back to isNil
* Fix error from merge
* advanced_fatigue stringtable german
* cargo stringtable german
* cookoff stringtable german
* fastroping stringtable german
* hellfire stringtable german
* nlaw stringtable german
* overpressure stringtable german
* quickmount stringtable german
* refuel stringtable german
* repair stringtable german
* scopes stringtable german
* slideshow stringtable german
* spectator stringtable german
* zeus stringtable german
* cookoff stringtable german #2
* hellfire stringtable german #2
* nlaw stringtable german #2
* overpressure stringtable german #2
* quickmount stringtable german#2
* refuel stringtable german #2
* scopes stringtable german #2
* slideshow stringtable german #2
* zeus stringtable german #2
* tiny fix
* tiny fix
* quickmount stringtable german #3
* fastroping stringtable german #2
* captives stringtable german
* cargo stringtable german
* cookoff stringtable german
* tiny change
* cargo stringtable german
* fcs stringtable german
* Fix german strings for scopes
* update the Jpn translation
Add translation for new words and small changes.
* Update Jpn translation
add translation for new words and some tweaks.
* Update Jpn translation
fix miss translation and minor tweak.
* Optimise cursor object updates
* Restrict distance units can be selected from
* Fix potential for no icons to render in FPP
* Limit projectile drawing distance
* Optimise icon updates
* Fix group names not being drawn in vehicles
* Fix selection nameplate not drawing for vehicles
* Fix selection of non-spectatable entities
* Fix incorrect function input for spectator hiding
* Remove locations tab from spectator
- Doesn't add much value, adds complexity and the implementation is half
baked. Would rather add back in at a later date (if at all) with a
better implementation.
- I have an idea to replace the locations tab with a meta tab so users
can toggle things like projectile drawing via the UI and are not forced
to use a hotkey. Might also be a good place to display the extended controls.
- Overhauls the spectator module entirely to share a similar UX to BI's "End Game Spectator" while maintaining some of the extended flexibility of ACE Spectator.
- Simplifies spectator setup by reducing the number of settings. More advanced setup is still possible via the API functions provided.
* French translation for the two last entries
* French translation for the last entrie
* French Translation for the first entrie
* correct the spelling error
Added Italian translation for:
-advanced fatigue
-advanced throwing
-cook off
-dogtags
-gunbag
-ui-scopes and much more...
Fixed a lot of grammatical errors
* Add the Japanese translation for cargo
Add the Japanese translation for cargo, fix to html tag and few improvement
* fix the Japanese transltion
Fix the Japanese translation for attach. it was used wrong html tag.
* changed The Japanese translation
Changed the Japanese translation of azimuth angle
* fix The Japanese translation
Fix the Japanese translation for chemlights. it was used wrong html tag.
* fix The Japanese translation
Fix the Japanese translation for laserpointer. it was used wrong html tag.
* Fix the Japanese translation
Fix the Japanese translation for medical_blood. it was used wrong translation.
* Fix the Japanese translation
Fix the Japanese translation for rearm. it was used wrong translation.
* add The Japanese translation
add The Japanese translation to new words
* Improve The Japanese translation
Improve, fix and change The Japanese translation
* minor fix to Japanese translation
minor fix to Japanese translation. see b1b07c5
The default values of the arrays manipulated by these functions overwrite
any changes applied before settings have finished initalising. This won't
be an issue after #4456 is merged in future.
- A `displayName` shown in the 3DEN multiplayer attributes display
- A `respawnTypes` array to define which respawn types the template is compatible with (currently only includes `2,3` - need to investigate compatibility with other types further).
Finally found out how to do this. Having search enabled on the unit tree might seem useful, but it doesn't interact well with keyboard shortcuts used for other purposes like camera control as it causes the tree to jump around unexpectedly.
* Add files via upload
* Add files via upload
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
* Added Korean translation
japanese translation caused error while translating process, had to remove some duplicated keys in stringtable.xml.
* Fix encoding
* Optimizations with private, params, and isEqualType
* Fixed tab being used instead of space
* Fixed tabs inserted by notepad++
* More usage of new private syntax and params
- changed a few checks for an array being empty to `_arr isEqualTo []`
rather than `count _arr == 0`
- added more uses of `private` on the same line as the variable is
declared
- added more uses of params to assign variables passed as parameters
- removed unnecessary parentheses
- removed several unnecessary variable declarations with private array
syntax
* clean up and formatting
Reduce the time between automated refreshing of the unit tree. Also makes an initial call to the tree populating code upon first opening to remove the initial delay.
- Account for decrement in node index when culling the tree of sides/groups/units
- Fix non-zero-based index `to` value when navigating the units in the tree
Minor oversight introduced in pabst's fix for #2989, the icon handler is a mission event handler that isn't self terminating so the index needs to be preserved until after it is removed via `removeMissionEventHandler`. It is safe to let the `onUnload` event take care of that (and preferable so that icons disappear while temporarily closed).
This allows me to explicitly close the right display and hopefully fix the interface staying open upon mission end.
@Commy2's theory is that perhaps the main display `46` is killed upon mission end, meaning that `findDisplay` can't find the spectator display to close it because the game thinks it doesn't exist.
Should fix#2989
There is a race condition between these 4 PFEH and the "onUnload" event
in handleInterface.
If the PFEHs run first they will use nil variables and throw a script
error.
This sets them to nil immediately when exiting spectator
Turns out custom radio channels are kind of a pain to work with. However this should work fine assuming the radioChannelAdd commands will work on client machines.
Fixes#2514 by only rendering individual unit icons when zoomed in and only rendering the units currently within the screen bounds.
Behaves as expected, but whether or not it has a significant performance increase under scaled up testing conditions remains to be seen.
Previously they were only hidden, but this didn't account for the annoying audio they play so now they're entirely staged like a spectator unit would be.
