Table groups, metadata, and editor UI memory - #6033
Conversation
|
Excellent update. I do wish there was a representation under the Table Image display, too, though. But to make that complete I suppose we'd need to be able to set a Perhaps in a separate PR? |
Yeah I get that. I did look into this and the Table window image panel uses a thing that is also used for the Library window image panel - so I did not want to change that within this request. b.t.w. I was also wanting to add optional Player / GM Notes for tables to show in the table's tooltip but the current image panel does not handle that nicely so I left those out for now.
I was thinking something akin to how macro groups are shown on macro panels, i.e. a border and a label for each table group. I would see dragging tables between groups being a GM only feature.
Indeed! Other table group enhancements could be rename a table group and to export/import a group of tables. |
resolves #6032
Description of the Change
EditLookupTablePanel
Added group ComboBox

Move table entries into a tab on a new tabbed pane

Added a metadata syntax editor into a tab on the new tabbed pane

Caches the panel from the factory, so retains the size and position when it is reopened rather than creating a new one each time from the DialogFactory.
LookupTableDetailPanel & Model
Added new column for Group visible to GMs and Players

Added new column for Metadata visible to GMs only - which displays the syntax type if there is metadata
Modified MTScript functions:
getTableNames(delim, group)- new group optional parameter to get all/visible table names in a named groupNew MTScript functions:
getTableGroups()- get groups from all (GM) /visible (Player) tablesNew MTScript trusted functions:
getTableGroup(tblName)- get the named table's groupsetTableGroup(tblName, group)- set the named table's groupgetTableMetadata(tblName)- get the named table's metadatasetTableMetadata(tblName, metadata)- set the named table's metadataPossible Drawbacks
Grouping only visible in the table details panel, not table image panel
Otherwise none foreseen
Release Notes
This change is