Opened 3 years ago

Closed 3 years ago

#14235 closed Bug (invalid)

Autogrow not kicking when we add Embed videos

Reported by: Satya Minnekanti Owned by:
Priority: Normal Milestone:
Component: General Version: 4.5.4
Keywords: IBM Cc: Irina, chris, gui

Description

Steps to reproduce

  1. Open OEMBED Samplehttp://sdk.ckeditor.com/samples/mediaembed.htmland enable Autogrow
  2. Insert few oembed videos using Video URL's one below the other so Autogrow will kick in and editor will grow

Expected result

After minimum editor height exceeded Autogrow kicks in and editor grows automatically

Actual result

After minimum editor height exceeded Autogrow not kicking in and user not able to se content below embedded content.

Change History (3)

comment:1 Changed 3 years ago by Jakub Ś

Status: newpending

I can't confirm that.

  1. Instead of demo page I have tried default editor with embed and autogrow plugins
  2. My page configuration was:
    var editor = CKEDITOR.replace( 'editor1', {
    	extraPlugins : 'embed,embedsemantic,autoembed,autogrow'
    	//,autoGrow_minHeight : 600 // teste with and without this option
    });
    
  3. I have opened embed dialog and inserted this link https://www.youtube.com/watch?v=H08tGjXNHO4. Autogrow fired with first video inserted and with every new element that was inserted.

Could you provide HTML file which when put into server will show this issue?

comment:2 in reply to:  1 Changed 3 years ago by Satya Minnekanti

Replying to j.swiderski:

I can't confirm that.

  1. Instead of demo page I have tried default editor with embed and autogrow plugins
  2. My page configuration was:
    var editor = CKEDITOR.replace( 'editor1', {
    	extraPlugins : 'embed,embedsemantic,autoembed,autogrow'
    	//,autoGrow_minHeight : 600 // teste with and without this option
    });
    
  3. I have opened embed dialog and inserted this link https://www.youtube.com/watch?v=H08tGjXNHO4. Autogrow fired with first video inserted and with every new element that was inserted.

Could you provide HTML file which when put into server will show this issue?

@j.swiderski Thanks a lot for providing the sample. IBM found out that it's only happening in our internal builds bcoz of a patch we applied. we fixed it ourslevs. Pls close this defect

comment:3 Changed 3 years ago by Jakub Ś

Resolution: invalid
Status: pendingclosed

Thank you for the feedback.

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy