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

Docs: Improved preserving custom content conversion guide #1821

Merged
merged 1 commit into from
Feb 25, 2020
Merged

Conversation

Mgsy
Copy link
Member

@Mgsy Mgsy commented Feb 4, 2020

Suggested merge commit message (convention)

Docs: Improved preserving custom content conversion guide. Closes ckeditor/ckeditor5#6077.

@Mgsy Mgsy requested a review from Reinmar February 4, 2020 09:54
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling d09c128 on i/6077 into 0dec72d on master.

@@ -330,7 +330,9 @@ function setupCustomClassConversion( viewElementName, modelElementName, editor )
editor.conversion.for( 'upcast' ).add( upcastCustomClasses( viewElementName ), { priority: 'low' } );

// Define downcast converters for a model element with a "low" priority so they are run after the default converters.
editor.conversion.for( 'downcast' ).add( downcastCustomClasses( modelElementName ), { priority: 'low' } );
// Use `downcastCustomClassesToFigure` if you'd like to keep your classes on <figure> element or `downcastCustomClassesToChild` if you'd like to keep your classes on a <figure> child element, i.e. <img>.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
// Use `downcastCustomClassesToFigure` if you'd like to keep your classes on <figure> element or `downcastCustomClassesToChild` if you'd like to keep your classes on a <figure> child element, i.e. <img>.
// Use `downcastCustomClassesToFigure` if you'd like to keep your classes on <figure> element or
// `downcastCustomClassesToChild` if you'd like to keep your classes on a <figure> child element, i.e. <img>.

@Reinmar Reinmar merged commit 5fdb765 into master Feb 25, 2020
@Reinmar Reinmar deleted the i/6077 branch February 25, 2020 11:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants