Skip to content
This repository has been archived by the owner on Jun 26, 2020. It is now read-only.

Commit

Permalink
Docs: Simplified the error message.
Browse files Browse the repository at this point in the history
  • Loading branch information
Reinmar committed Sep 12, 2017
1 parent 03b0701 commit 6b8be86
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
5 changes: 2 additions & 3 deletions src/toolbar/toolbarview.js
Original file line number Diff line number Diff line change
Expand Up @@ -134,7 +134,7 @@ export default class ToolbarView extends View {
} else {
/**
* There was a problem processing the configuration of the toolbar. The item with the given
* name does not exist and it was omitted when adding toolbar items in DOM.
* name does not exist so it was omitted when rendering the toolbar.
*
* This warning usually shows up when the {@link module:core/plugin~Plugin} which is supposed
* to provide a toolbar item has not been loaded or there is a typo in the configuration.
Expand All @@ -144,11 +144,10 @@ export default class ToolbarView extends View {
*
* @error toolbarview-item-unavailable
* @param {String} name The name of the component.
* @param {module:ui/componentfactory~ComponentFactory} factory The factory that is missing the component.
*/
log.warn(
'toolbarview-item-unavailable: The requested toolbar item is unavailable.',
{ name, factory }
{ name }
);
}
} );
Expand Down
2 changes: 1 addition & 1 deletion tests/toolbar/toolbarview.js
Original file line number Diff line number Diff line change
Expand Up @@ -265,7 +265,7 @@ describe( 'ToolbarView', () => {
sinon.assert.calledOnce( log.warn );
sinon.assert.calledWithExactly( log.warn,
sinon.match( /^toolbarview-item-unavailable/ ),
{ name: 'baz', factory }
{ name: 'baz' }
);
} );
} );
Expand Down

0 comments on commit 6b8be86

Please sign in to comment.