#UDN: Feature suggestion; Content browser can sort by a second column
#branch UE4
#change
Added two new images for secondary sort ascending and descending.
(Updated styles where needed).
Added new enum EColumnSortPriority: currently only lists Primary and Secondary, but can easily accommodate more*.
(FOnSortModeChanged was modified to also have the priority as a param, fixedup existing usage).
SHeaderRow now also has a SortPriority attribute to specify the SortMode order
(Defaults to Primary if unused).
Modified TUniquePtr so that assigning one from another worked correctly.
(Reviewed by Steve Robb).
SAssetView is the only table that has been modified, so far, to take advantage of the secondary sort. SetMajorityAssetType has been updated to correctly filter out all those sorts which are no longer relevant and bump the priority of the remaining sorts to fill in the ægapsÆ made by non-longer-relevant ones.
FAssetViewSortManager has been overhauled to take SortPriority into consideration when sortingà
Firstly, duplicate comparison structs were removed in favour of single structs which have æascendingÆ as a paramà any remaining duplicate code was removed in favour of an inherited usage. The base struct has an array of æfurther methodsÆ to employ in the result of a tie. Should a tie occur the ænextÆ comparison is done, and so on until itÆs not a tie or we run out of comparisons to perform.*
The manager defaults to having no secondary sort, so it relies on the interface thatÆs using it to provide it.
Whenever a column is assign the code makes sure that itÆs not already assigned to another column and corrects the order to take this into account.
Fixed a bug in FCompareFAssetItemByTagNumericalAscending comparing A with A (instead of B).
Various optimizations to the sort to descrease times
*The only places æSecondaryÆ is referred to in code is in GetSortingBrush (so it can display the correct icon for the sort), and OnTitleClicked (so it can set to correct sort based on input). The sorting code itself has no concept as to a secondary sort, it can support any number of sorting levels so if in future a tertiary (or more) is added, it is only these two function which should need updating as the sort will automatically accommodate it.
reviewed by Thomas.Sarkanen, Bob.Tellez
[CL 2119201 by Andrew Brown in Main branch]
Includes following engine CLs:
CL 2080462:
Added verbosity selection to category filters in Log Visualizer window
CL 2081939:
Fixed crash in Log Visualizer and missing entry on timeline with log containing only 2 entries.
CL 2082118:
Fixes and improovements for VisLogs
- fixed logs visualization for logs with only one entry (Log Visualizer window)
- fixed logs spam about missing json field (Log Visualizer window)
- fixed crash with categories in Log Visualization window
- added category and verbosity serialization for vislog shape elements (vlog files)
CL 2080253:
Added categories to VisLog shapes
CL 2081762:
Fixes to Log Visualizer. Logs without any log lines but with geometry elements are set on timeline properly.
CL 2081859:
Added double click to Log Visualizer action, to move editor camera to view current log location
CL 2080159:
Gameply Tag Pin now only allows single select
#TTP 335383 - Editor: GameplayTag Pin should only allow a single selection
#proj UE4
#change Removes all other tags from the correct container when in single select mode, was a missed variable change from a previous TTP
CL 2081306:
Made changes to navigation filters that allow for usage of virtual verstion dtQueryFilter #UE4
The scary part here is that having a dtQueryFilter contan a vftable was messing up FRecastQueryFilter's vftable it was getting from INavigationQueryFilterInterface. Reversing order of parent clas declaration of FRecastQueryFilter fixed the issue, but it's scary and looks like a compiler issue.
[CL 2088758 by Billy Bramer in Main branch]
