#UE-13299 - Comments and comment balloons don't obey colors set in editor preferences
#Branch UE4
#Proj BehaviourTreeEditor, EnvironmentQueryEditor, GraphEditor
#codereview Dan.Oconnor
[CL 2505562 by Ben Cosh in Main branch]
CL# 2453024: added navigation filter to EQS pathfinding tests
CL# 2466155: added batch pathfinding test for EQS
CL# 2466237: EQS generators will now cache NavLocation of items if navmesh projection is enabled
CL# 2466356: added support for deprecated nodes in EQS editor
CL# 2466358: pathing grid EQS generator is now deprecated
CL# 2466439: added failsafe in EQS editor for recreating missing nodes
CL# 2467466: fixed recreating test nodes in EQS graph
CL# 2467476: disabled HPA* in pathfinding EQS test
CL# 2467508: removed hierarchical pathfinding option from EQS tests
CL# 2468574: EQS generators will now cache NavLocation of items if navmesh projection is enabled (crash fix)
[CL 2472208 by Lukasz Furman in Main branch]
And not crash when one gets removed!
4.7 critical fix.
This is to be fixed properly in time for 4.8, to reuse the solution BT editor has.
#lockdown Zachary.EdgertonJones
[CL 2443252 by Mieszko Zielinski in Main branch]
Fixed up the details panel name area to be aware of in world component editing.
Fixed up usage of the details panel name area to use new enum based usage flag for name areas
[CL 2419254 by Ben Marsh in Main branch]
I fixed the crash found in the material editor, but discovered some rendering issues. Backing this out for now.
[CL 2412583 by Dan Oconnor in Main branch]
Remove duplicated implementations of DrawConnection in subclasses that draw non-curved splines, overriding ComputeSplineTangent instead
Upgrade note: Any editor plugins that contain a FConnectionDrawingPolicy subclass will need to update calls to DrawSplineWithArrow, DrawConnection, and DetermineWiringStyle
[CL 2402661 by Michael Noland in Main branch]
UETOOL-213 - Minimize Slate FString -> FText conversion (remove SLATE_TEXT_ATTRIBUTE)
This fixes any editor/engine specific code that was passing text to Slate as FString rather than FText.
[CL 2399803 by Jamie Dale in Main branch]
Introduces the concept of "Active Ticking" to allow Slate to go to sleep when there is no need to update the UI.
While asleep, Slate will skip the Tick & Paint pass for that frame entirely.
- There are TWO ways to "wake" Slate and cause a Tick/Paint pass:
1. Provide some sort of input (mouse movement, clicks, and key presses). Slate will always tick when the user is active.
- Therefore, if the logic in a given widget's Tick is only relevant in response to user action, there is no need to register an active tick.
2. Register an Active Tick. Currently this is an all-or-nothing situation, so if a single active tick needs to execute, all of Slate will be ticked.
- The purpose of an Active Tick is to allow a widget to "drive" Slate and guarantee a Tick/Paint pass in the absence of any user action.
- Examples include animation, async operations that update periodically, progress updates, loading bars, etc.
- An empty active tick is registered for viewports when they are real-time, so game project widgets are unaffected by this change and should continue to work as before.
- An Active Tick is registered by creating an FWidgetActiveTickDelegate and passing it to SWidget::RegisterActiveTick()
- There are THREE ways to unregister an active tick:
1. Return EActiveTickReturnType::StopTicking from the active tick function
2. Pass the FActiveTickHandle returned by RegisterActiveTick() to SWidget::UnregisterActiveTick()
3. Destroy the widget responsible for the active tick
- Sleeping is currently disabled, can be enabled with Slate.AllowSlateToSleep cvar
- There is currently a little buffer time during which Slate continues to tick following any input. Long-term, this is planned to be removed.
- The duration of the buffer can be adjusted using Slate.SleepBufferPostInput cvar (defaults to 1.0f)
- The FCurveSequence API has been updated to work with the active tick system
- Playing a curve sequence now requires that you pass the widget being animated by the sequence
- The active tick will automatically be registered on behalf of the widget and unregister when the sequence is complete
- GetLerpLooping() has been removed. Instead, pass true as the second param to Play() to indicate that the animation will loop. This causes the active tick to be registered indefinitely until paused or jumped to the start/end.
[CL 2391669 by Dan Hertzka in Main branch]