dijit._editor._Plugin: custom plugins UI does not show up if they are meant to handle custom commands

[lipik]

I am creating a pluging to handle the save command - a button is displayed that would carry out some custom action (save editor content upon being clicked). However, the UI comes in hidden due to the following code in _Plugin.js:

if (!this.editor.queryCommandAvailable(this.command)) {
 // console.debug("hiding:", this.command);
 if(this.button){
	this.button.domNode.style.display = "none";
 }
}

It should be changed to:

if (this.useDefaultCommand && !this.editor.queryCommandAvailable(this.command)) {
 // console.debug("hiding:", this.command);
 if(this.button){
	this.button.domNode.style.display = "none";
 }
}

[bill]

I'm not sure what useDefaultCommand is for? There aren't any comments in the code. Can you explain what that parameter is for?

[liucougar]

useDefaultCommand determines whether the function is provided by the native browser or not

so I think the suggested change should be made

[bill]

The current code and the ​1.2 code actually checks for this.command.length to see if a command is defined, so I'm not sure where you copied that code from in the description above.

Still, the this.command.length check isn't sufficient because a plugin like TabIndent defines command but doesn't use the browser's native support to execute it. So I'll make the change.

[bill]

(In [21691]) Fixing check for when to hide the plugin's button, although I'm doubtful that display:none on the button for a plugin that won't work is the right approach; for example it might mess up keyboard support in Toolbar. Fixes #7828 !strict.