Navigation editor: Fail loudly when attempting to save unsupported blocks

[adamziel]

Navigation editor only processes a subset of blocks in the editor, silently omitting anything that's not supported:

gutenberg/packages/edit-navigation/src/store/actions.js

Lines 207 to 209 in fa57287

	const updatedMenuItems = blocksTreeToAnnotatedList( navigationBlock )
	// Filter out unsupported blocks
	.filter( ( { block } ) => isBlockSupportedInNav( block ) )

It would be useful to log a warning and make any problems there apparent.

[anton-vlasenko]

@adamziel

It would be useful to logging a warning and make any problems there apparent.

When you say it would be helpful to log warnings, do you mean simple browser console warnings?
Or do we need something more complicated?

[adamziel]

do you mean simple browser console warnings?

@anton-vlasenko Yup, exactly! Just to let the developers know something is off. I think we could use @wordpress/warning here.