Skip to content

Conversation

@akki-jat
Copy link

For more details: #107
Maybe also related to #87 but didn't check with v-if

Also there is one more PR open maybe for the same thing I am trying to do (but I did go with small change), also look at that: #98.

My error case

Error was coming for recreating editor because in initEvents it keeps reference of old VueComponent and that even again calls again the new instance is created but it gets _editor null because we destroyed it.

Also it can create memory leak problems because it was keeping VueComponent in init event callback.

@akki-jat
Copy link
Author

@stefanneculai can you take a look at this?

jyyan added a commit to jyyan/vue-froala-wysiwyg that referenced this pull request Jan 15, 2020
@gabaum10
Copy link

Does anyone know who's maintaining this? We'd like to get this merged/resolved. I'm tired of fielding "there is this mystery error in the console, fix it!" requests.

@rcheung9
Copy link

rcheung9 commented Feb 7, 2020

Is this going to be merged?

@akki-jat @gabaum10 @stefanneculai

@akki-jat
Copy link
Author

akki-jat commented Feb 8, 2020

I have also raised support ticket for the same.

@akki-jat
Copy link
Author

@kapil2704 can you look at this.

@paresh-accolite paresh-accolite mentioned this pull request Sep 25, 2020
@akki-jat
Copy link
Author

Closing this MR as changes are already merged with #157

@akki-jat akki-jat closed this Nov 16, 2020
@akki-jat akki-jat deleted the Fix_htmlSet_Error branch November 16, 2020 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants