ExternalVendorCode/openmct
1
0
Fork 0
mirror of https://github.com/nasa/openmct.git synced 2025-06-01 23:20:50 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

[Edit] Default to inclusive mode

Browse Source 
Default to inclusive mode in Edit mode toolbar;
simplifies adding additional toolbar elements for
Fixed Position view, WTD-881.
This commit is contained in:
Victor Woeltjen 2015-02-23 15:13:46 -08:00
parent 46c8a456b7
commit f894233182
3 changed files with 7 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
platform/commonUI/edit/README.md
View File

@ -17,10 +17,10 @@ view's scope.) These additional properties are:
an argument to set.) an argument to set.)
* `method`: Name of a method to invoke upon a selected object when * `method`: Name of a method to invoke upon a selected object when
a control is activated, e.g. on a button click. a control is activated, e.g. on a button click.
* `inclusive`: Optional; true if this control should be considered * `exclusive`: Optional; true if this control should be considered
applicable whenever at least one element in the selection has applicable only when all elements in the selection has
the associated property. Otherwise, all members of the current the associated property. Otherwise, only at least one member of the
selection must have this property for the control to be shown. current selection must have this property for the control to be shown.
Controls in the toolbar are shown based on applicability to the Controls in the toolbar are shown based on applicability to the
current selection. Applicability for a given member of the selection current selection. Applicability for a given member of the selection

2
platform/commonUI/edit/src/representers/EditToolbar.js
View File

@ -108,7 +108,7 @@ define(
function isApplicable(item) { function isApplicable(item) {
var property = (item || {}).property, var property = (item || {}).property,
method = (item || {}).method, method = (item || {}).method,
exclusive = !(item || {}).inclusive; exclusive = (item || {}).exclusive;
// Check if a selected item defines this property // Check if a selected item defines this property
function hasProperty(selected) { function hasProperty(selected) {

3
platform/commonUI/edit/src/representers/EditToolbarRepresenter.js
View File

@ -57,6 +57,7 @@ define(
commit("Changes from toolbar."); commit("Changes from toolbar.");
} }
// Initialize toolbar (expose object to parent scope)
function initialize() { function initialize() {
// If we have been asked to expose toolbar state... // If we have been asked to expose toolbar state...
if (attrs.toolbar) { if (attrs.toolbar) {
@ -67,7 +68,7 @@ define(
// Represent a domain object using this definition // Represent a domain object using this definition
function represent(representation) { function represent(representation) {
// // Expose the toolbar object to the parent scope
initialize(); initialize();
// Clear any existing selection // Clear any existing selection
scope.selection = []; scope.selection = [];