Selectively disable annotations

[JustinHoldstock]

Annotations can now be selectively disabled through preview options.
Viewer options override global showAnnotations value, for that viewer.
Tests to follow.

// Turn on/off annotations for all viewers
preview.show(..., {
    showAnnotations: Boolean
});

Combined with the following:

// [LEGACY] BOOLEAN - turn on/off annotations for a viewer
preview.show(..., {
    viewers: {
        VIEWER_NAME: {
            annotations: Boolean
        }
    }
});

// [NEW] OBJECT - turn on/off annotations for a viewer.
preview.show(..., {
    viewers: {
        VIEWER_NAME: {
            annotations: {
                enabled: Boolean, // Enables/disables if set. Respects "showAnnotations" if empty
                disabledTypes: String[] // List of annotation types to disable
            }
        }
    }
});

And an example. Enable all annotations, turn off for Image Viewers, and
disable point annotations on Document viewer:

preview.show(fileId, token, {
    showAnnotations: true,
    viewers: {
        Image: {
            annotations: {
                enabled: false
            }
        },
        Document: {
            annotations: {
                disabledTypes: ['point']
            }
        }
    }
});

[pramodsum]

test?

[pramodsum]

const annotator = this.getAnnotatorsForViewer(viewerName, disabledAnnotators);
const modifiedAnnotator = null;
if (!annotator) {
    return modifiedAnnotator;
}

modifiedAnnotator = ....
if (viewerConfig.enabled) {
    if (Array.isArray(....)) {
        ...
    }
}
return modifiedAnnotator;

[JustinHoldstock]

Much better, ty. Done

[pramodsum]

Can we do this instead
determineAnnotator(viewerName, viewerConfig = {}, disabledAnnotators = []) {

[JustinHoldstock]

+1, and done

[pramodsum]

See comment above but if we modify the function declaration, then we can call it like this
boxAnnotations.determineAnnotator(viewerName, annotationsConfig);

[JustinHoldstock]

DONE! :D

[pramodsum]

reorder these

[JustinHoldstock]

This already has been reorder to what we've already discussed

[pramodsum]

👍 I think i misread it because both were arrays here

[pramodsum]

Can this be moved below and then you can just do return modifiedAnnotator; below?

[JustinHoldstock]

Done

[pramodsum]

const config = (option !== null && option !== undefined) ? option : {};

but also does the previous version not work?
const config = this.getViewerOption('annotations') || {};

[JustinHoldstock]

Previous doesn't work because the option can return false

[JustinHoldstock]

done